On Wed, 2010-08-25 at 14:01 -0400, John W. Linville wrote: > > "<level>dev_driver_string(&wiphy->dev) dev_name(&wiphy->dev): wiphy_name(wiphy): " fmt, ##args > > > > for instance: > > > > [ 41.261941] ieee80211 phy0: phy0: Selected rate control algorithm 'minstrel_ht' > > > > ie: redundant. > > wiphy_name(wiphy) is an inline for dev_name(&wiphy->dev) > > > > So it's appropriate to use: > > > > #define wiphy_<level>(wiphy, fmt, arg...) dev_<level>(&wiphy->dev, fmt, ##arg) > > > > and the output becomes: > > > > [ 41.261941] ieee80211 phy0: Selected rate control algorithm 'minstrel_ht' > > > > Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> > > Everyone OK w/ this version? I don't care much :) johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
