On Wed, 2009-09-30 at 21:19 -0400, John W. Linville wrote: > + if (!dev->ethtool_ops) > + dev->ethtool_ops = &cfg80211_ethtool_ops; > break; I might go so far and do it unconditionally so we get consistent functionality across things. OTOH, full-mac drivers might be able to support more. > +const struct ethtool_ops cfg80211_ethtool_ops = { > + .get_drvinfo = cfg80211_get_drvinfo, > + .get_link = ethtool_op_get_link, > +}; > + > +void cfg80211_get_drvinfo(struct net_device *dev, struct ethtool_drvinfo *info) if you change the order, you can make the latter static johannes
Attachment:
signature.asc
Description: This is a digitally signed message part