EljakimHerrewijnen/Openwrt
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

kernel: owl-loader: fix sparse endian warnings

Browse Source 
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 <chunkeey@gmail.com>
This commit is contained in:
Christian Lamparter 2019-02-25 15:35:23 +01:00
parent ca769e081f
commit ddece08bf4
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
target/linux/generic/files/drivers/misc/owl-loader.c
View File

@ -40,9 +40,9 @@ static int ath9k_pci_fixup(struct pci_dev *pdev, const u16 *cal_data,
void __iomem *mem; void __iomem *mem;
const void *cal_end = (void *)cal_data + cal_len; const void *cal_end = (void *)cal_data + cal_len;
const struct { const struct {
__be16 reg; u16 reg;
__be16 low_val; u16 low_val;
__be16 high_val; u16 high_val;
} __packed *data; } __packed *data;
u16 cmd; u16 cmd;
u32 bar0; 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 */ /* set pointer to first reg address */
for (data = (const void *) (cal_data + 3); 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++) { data++) {
u32 val; u32 val;
u16 reg; u16 reg;
reg = data->reg; reg = data->reg;
val = data->low_val; val = data->low_val;
val |= data->high_val << 16; val |= ((u32)data->high_val) << 16;
if (swap_needed) { if (swap_needed) {
reg = swab16(reg); reg = swab16(reg);