Re: [PATCH 0/3] net: mac80211: Neaten debugging

On Wed, 2012-05-16 at 17:30 +0200, Johannes Berg wrote:

On Wed, 2012-05-16 at 08:22 -0700, Joe Perches wrote:

quoted

On Wed, 2012-05-16 at 09:59 +0200, Johannes Berg wrote:

quoted

I wonder if it makes sense to leave these under
#ifdef though? Why #ifdef something if it's going to be invisible most
of the time anyway?

I don't understand your point.
#ifdef removal is a good thing.

Yeah, but you left a lot of them under ifdef, and I'm wondering why you
didn't remove them, or if you should, or ...

Those mostly use other different
#ifdef CONFIG_SOME_OTHER_CONTROL elements.

There are I think 3 that I left because they do
not use printk/pr_level but use wiphy_<level> or
netdev_<level>.