]> git.dujemihanovic.xyz Git - linux.git/commitdiff
i2c: cp2615: Fix 'assignment to __be16' warning
authorBence Csókás <bence98@sch.bme.hu>
Mon, 30 Oct 2023 17:19:10 +0000 (17:19 +0000)
committerWolfram Sang <wsa@kernel.org>
Wed, 8 Nov 2023 09:27:17 +0000 (10:27 +0100)
While the preamble field _is_ technically big-endian, its value is always 0x2A2A,
which is the same in either endianness. However, to avoid generating a warning,
we should still call `htons()` explicitly.

Signed-off-by: Bence Csókás <bence98@sch.bme.hu>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-cp2615.c

index 20f8f7c9a8cd45a4095687938d220ea2ed687e1a..cf3747d870342e3649e290c8f254aa586c36a67a 100644 (file)
@@ -85,7 +85,7 @@ static int cp2615_init_iop_msg(struct cp2615_iop_msg *ret, enum cp2615_iop_msg_t
        if (!ret)
                return -EINVAL;
 
-       ret->preamble = 0x2A2A;
+       ret->preamble = htons(0x2A2AU);
        ret->length = htons(data_len + 6);
        ret->msg = htons(msg);
        if (data && data_len)