diff options
author | Hannes Eder <hannes@hanneseder.net> | 2008-12-26 02:55:35 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-12-26 02:55:35 -0500 |
commit | e4c3c13cb4c4985cb62cf28677fc0ace69a8d69f (patch) | |
tree | 3e944ccdee5669f517803ed3ab7475c75a35e69a /drivers/net/starfire.c | |
parent | 2705d4f87c068552f45e5d6feaa5c468a312f761 (diff) |
drivers/net: fix sparse warnings: make do-while a compound statement
While at it insert some extra curly braces and fix formatting.
Fix this sparse warnings:
drivers/net/atp.c:811:8: warning: do-while statement is not a compound statement
drivers/net/atp.c:813:8: warning: do-while statement is not a compound statement
drivers/net/atp.c:815:11: warning: do-while statement is not a compound statement
drivers/net/atp.c:817:11: warning: do-while statement is not a compound statement
drivers/net/plip.c:642:4: warning: do-while statement is not a compound statement
drivers/net/plip.c:647:4: warning: do-while statement is not a compound statement
drivers/net/plip.c:820:4: warning: do-while statement is not a compound statement
drivers/net/plip.c:825:4: warning: do-while statement is not a compound statement
drivers/net/starfire.c:886:3: warning: do-while statement is not a compound statement
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/starfire.c')
-rw-r--r-- | drivers/net/starfire.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/starfire.c b/drivers/net/starfire.c index 1902f4fabde1..f54ac2389da2 100644 --- a/drivers/net/starfire.c +++ b/drivers/net/starfire.c | |||
@@ -880,9 +880,9 @@ static int mdio_read(struct net_device *dev, int phy_id, int location) | |||
880 | void __iomem *mdio_addr = np->base + MIICtrl + (phy_id<<7) + (location<<2); | 880 | void __iomem *mdio_addr = np->base + MIICtrl + (phy_id<<7) + (location<<2); |
881 | int result, boguscnt=1000; | 881 | int result, boguscnt=1000; |
882 | /* ??? Should we add a busy-wait here? */ | 882 | /* ??? Should we add a busy-wait here? */ |
883 | do | 883 | do { |
884 | result = readl(mdio_addr); | 884 | result = readl(mdio_addr); |
885 | while ((result & 0xC0000000) != 0x80000000 && --boguscnt > 0); | 885 | } while ((result & 0xC0000000) != 0x80000000 && --boguscnt > 0); |
886 | if (boguscnt == 0) | 886 | if (boguscnt == 0) |
887 | return 0; | 887 | return 0; |
888 | if ((result & 0xffff) == 0xffff) | 888 | if ((result & 0xffff) == 0xffff) |