On Mon, 17 Apr 2006 10:17:43 +0800 Zhu Yi <[EMAIL PROTECTED]> wrote: > On Fri, 2006-04-14 at 09:33 -0700, Stephen Hemminger wrote: > > I meant get rid of CONFIG_IPW2200_DEBUG completely. Having the debug code > > isn't > > bad, and there is no reason not to have it always there. > > There are lots of IPW_DEBUG_XXX in the driver, even in TX and RX paths. > It cause extra cycles for those who really don't need it. > > Thanks, > -yi
IMHO. There is never a reason for a driver specific config option for debugging. Either it the debug option is generally useful for support (ie distro's will want it on), in which case the debugging should be runtime settable but always available; or the debug option is so meant for the driver developers in which case it should be enabled by a source modification (see #define DEBUG and pr_debug). But, those are opinions only and certainly, you can do what you want with your driver. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
