Commit ce3f9521 authored by Jakub Kicinski's avatar Jakub Kicinski

Merge branch 'davicom-w-1-fixes'

Andrew Lunn says:

====================
davicom W=1 fixes

Fixup various W=1 warnings, and then add COMPILE_TEST support, which
explains why these where missed on the previous pass.
====================

Link: https://lore.kernel.org/r/20201031005833.1060316-1-andrew@lunn.chSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parents 7812e105 1bcb32c7
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
config DM9000 config DM9000
tristate "DM9000 support" tristate "DM9000 support"
depends on ARM || MIPS || COLDFIRE || NIOS2 depends on ARM || MIPS || COLDFIRE || NIOS2 || COMPILE_TEST
select CRC32 select CRC32
select MII select MII
help help
......
...@@ -232,32 +232,29 @@ static void dm9000_inblk_32bit(void __iomem *reg, void *data, int count) ...@@ -232,32 +232,29 @@ static void dm9000_inblk_32bit(void __iomem *reg, void *data, int count)
static void dm9000_dumpblk_8bit(void __iomem *reg, int count) static void dm9000_dumpblk_8bit(void __iomem *reg, int count)
{ {
int i; int i;
int tmp;
for (i = 0; i < count; i++) for (i = 0; i < count; i++)
tmp = readb(reg); readb(reg);
} }
static void dm9000_dumpblk_16bit(void __iomem *reg, int count) static void dm9000_dumpblk_16bit(void __iomem *reg, int count)
{ {
int i; int i;
int tmp;
count = (count + 1) >> 1; count = (count + 1) >> 1;
for (i = 0; i < count; i++) for (i = 0; i < count; i++)
tmp = readw(reg); readw(reg);
} }
static void dm9000_dumpblk_32bit(void __iomem *reg, int count) static void dm9000_dumpblk_32bit(void __iomem *reg, int count)
{ {
int i; int i;
int tmp;
count = (count + 3) >> 2; count = (count + 3) >> 2;
for (i = 0; i < count; i++) for (i = 0; i < count; i++)
tmp = readl(reg); readl(reg);
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment