From ddece08bf4569d0eec865d2cbeecc551701d4467 Mon Sep 17 00:00:00 2001 From: Christian Lamparter Date: Mon, 25 Feb 2019 15:35:23 +0100 Subject: [PATCH] kernel: owl-loader: fix sparse endian warnings owl-loader.c:83:21: sparse: warning: incorrect type in assignment (different base types) owl-loader.c:83:21: sparse: expected unsigned short [usertype] reg owl-loader.c:83:21: sparse: got restricted __be16 const [usertype] reg owl-loader.c:84:21: sparse: warning: incorrect type in assignment (different base types) owl-loader.c:84:21: sparse: expected unsigned int [usertype] val owl-loader.c:84:21: sparse: got restricted __be16 const [usertype] low_val owl-loader.c:85:28: sparse: warning: restricted __be16 degrades to integer Reported-by: kbuild test robot Signed-off-by: Christian Lamparter --- target/linux/generic/files/drivers/misc/owl-loader.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/target/linux/generic/files/drivers/misc/owl-loader.c b/target/linux/generic/files/drivers/misc/owl-loader.c index f11cb2b49c5e..5cbee79740dd 100644 --- a/target/linux/generic/files/drivers/misc/owl-loader.c +++ b/target/linux/generic/files/drivers/misc/owl-loader.c @@ -40,9 +40,9 @@ static int ath9k_pci_fixup(struct pci_dev *pdev, const u16 *cal_data, void __iomem *mem; const void *cal_end = (void *)cal_data + cal_len; const struct { - __be16 reg; - __be16 low_val; - __be16 high_val; + u16 reg; + u16 low_val; + u16 high_val; } __packed *data; u16 cmd; u32 bar0; @@ -75,14 +75,14 @@ static int ath9k_pci_fixup(struct pci_dev *pdev, const u16 *cal_data, /* set pointer to first reg address */ for (data = (const void *) (cal_data + 3); - (const void *) data <= cal_end && data->reg != cpu_to_be16(~0); + (const void *) data <= cal_end && data->reg != ~0; data++) { u32 val; u16 reg; reg = data->reg; val = data->low_val; - val |= data->high_val << 16; + val |= ((u32)data->high_val) << 16; if (swap_needed) { reg = swab16(reg); -- 2.30.2