Commit 8bcbf82f authored by Andy Shevchenko's avatar Andy Shevchenko Committed by David S. Miller

net/macb: suppress compiler warnings

This patch fixes the following warnings:
drivers/net/ethernet/cadence/macb.c: In function ‘macb_handle_link_change’:
drivers/net/ethernet/cadence/macb.c:266: warning: comparison between signed and unsigned
drivers/net/ethernet/cadence/macb.c:267: warning: comparison between signed and unsigned
drivers/net/ethernet/cadence/macb.c:291: warning: comparison between signed and unsigned
drivers/net/ethernet/cadence/macb.c: In function ‘gem_update_stats’:
drivers/net/ethernet/cadence/macb.c:1908: warning: comparison between signed and unsigned
drivers/net/ethernet/cadence/macb.c: In function ‘gem_get_ethtool_strings’:
drivers/net/ethernet/cadence/macb.c:1988: warning: comparison between signed and unsigned
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a35919e1
...@@ -303,7 +303,6 @@ static void macb_handle_link_change(struct net_device *dev) ...@@ -303,7 +303,6 @@ static void macb_handle_link_change(struct net_device *dev)
struct macb *bp = netdev_priv(dev); struct macb *bp = netdev_priv(dev);
struct phy_device *phydev = bp->phy_dev; struct phy_device *phydev = bp->phy_dev;
unsigned long flags; unsigned long flags;
int status_change = 0; int status_change = 0;
spin_lock_irqsave(&bp->lock, flags); spin_lock_irqsave(&bp->lock, flags);
...@@ -1936,7 +1935,7 @@ static int macb_change_mtu(struct net_device *dev, int new_mtu) ...@@ -1936,7 +1935,7 @@ static int macb_change_mtu(struct net_device *dev, int new_mtu)
static void gem_update_stats(struct macb *bp) static void gem_update_stats(struct macb *bp)
{ {
int i; unsigned int i;
u32 *p = &bp->hw_stats.gem.tx_octets_31_0; u32 *p = &bp->hw_stats.gem.tx_octets_31_0;
for (i = 0; i < GEM_STATS_LEN; ++i, ++p) { for (i = 0; i < GEM_STATS_LEN; ++i, ++p) {
...@@ -2015,7 +2014,7 @@ static int gem_get_sset_count(struct net_device *dev, int sset) ...@@ -2015,7 +2014,7 @@ static int gem_get_sset_count(struct net_device *dev, int sset)
static void gem_get_ethtool_strings(struct net_device *dev, u32 sset, u8 *p) static void gem_get_ethtool_strings(struct net_device *dev, u32 sset, u8 *p)
{ {
int i; unsigned int i;
switch (sset) { switch (sset) {
case ETH_SS_STATS: case ETH_SS_STATS:
......
...@@ -816,9 +816,9 @@ struct macb { ...@@ -816,9 +816,9 @@ struct macb {
struct mii_bus *mii_bus; struct mii_bus *mii_bus;
struct phy_device *phy_dev; struct phy_device *phy_dev;
unsigned int link; int link;
unsigned int speed; int speed;
unsigned int duplex; int duplex;
u32 caps; u32 caps;
unsigned int dma_burst_length; unsigned int dma_burst_length;
......
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