On Wed, 2012-04-04 at 13:00 -0700, Pedersen, Thomas wrote: > > > + ath6kl_dbg(ATH6KL_DBG_WLAN_CFG, "channel switch notify nw_type %d" > > > + "freq %d mode %d", vif->nw_type, channel, mode); > > > > Please coalesce formats. > > If you had, you'd've seen a problem with this format. > > Add a newline "\n" terminator to it too. > > I'm not sure what you mean by "coalesce formats". The freq / channel > semantic mismatch? I'll change that and add a newline as well. There's currently no space between a "%d" and "freq". There should be one. It'd be better and less error prone like this: ath6kl_dbg(ATH6KL_DBG_WLAN_CFG, "channel switch notify nw_type %d freq %d mode %d\n", vif->nw_type, channel, mode); cheers, Joe -- 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