On Wed, 2012-09-19 at 13:55 +0200, Nicolas Ferre wrote: > This function has little meaning so remove it altogether and > let ethtool core fill in the fields automatically. > > Signed-off-by: Nicolas Ferre <[email protected]> Reviewed-by: Ben Hutchings <[email protected]> > --- > drivers/net/ethernet/cadence/macb.c | 11 ----------- > 1 file changed, 11 deletions(-) > > diff --git a/drivers/net/ethernet/cadence/macb.c > b/drivers/net/ethernet/cadence/macb.c > index 2948553..31f945c 100644 > --- a/drivers/net/ethernet/cadence/macb.c > +++ b/drivers/net/ethernet/cadence/macb.c > @@ -1217,20 +1217,9 @@ static int macb_set_settings(struct net_device *dev, > struct ethtool_cmd *cmd) > return phy_ethtool_sset(phydev, cmd); > } > > -static void macb_get_drvinfo(struct net_device *dev, > - struct ethtool_drvinfo *info) > -{ > - struct macb *bp = netdev_priv(dev); > - > - strcpy(info->driver, bp->pdev->dev.driver->name); > - strcpy(info->version, "$Revision: 1.14 $"); > - strcpy(info->bus_info, dev_name(&bp->pdev->dev)); > -} > - > static const struct ethtool_ops macb_ethtool_ops = { > .get_settings = macb_get_settings, > .set_settings = macb_set_settings, > - .get_drvinfo = macb_get_drvinfo, > .get_link = ethtool_op_get_link, > }; >
-- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
