Re: [PATCH 1/4] ath5k: debug level improvements

[bruno randolf <bruno@thinktube.com>]

On Friday 18 January 2008 21:55:38 Jiri Slaby wrote:
> On 01/18/2008 01:50 PM, Bruno Randolf wrote:
> > * use only one debug level for beacon debugging: unify ATH5K_DEBUG_BEACON
> > and ATH5K_DEBUG_BEACON_PROC
> >
> > * remove debug level ATH5K_DEBUG_FATAL. doesn't make sense as a debug
> > level - if it's fatal it should be logged as an error.
> >
> > * fancier printing of debug levels. cat /debugfs/ath5k/phy0/debug
> >
> > * allow debug levels to be changed by echoing their name into
> > /debugfs/ath5k/phy0/debug. this will toggle the state, when it was off it
> > will be turned on and vice versa.
> >
> > drivers/net/wireless/ath5k/base.c:      Changes-licensed-under:
> > 3-Clause-BSD drivers/net/wireless/ath5k/debug.c:    
> > Changes-licensed-under: GPL drivers/net/wireless/ath5k/debug.h:    
> > Changes-licensed-under: GPL
> >
> > Signed-off-by: Bruno Randolf <bruno@thinktube.com>
> > ---
>
> [...]
>
> > +#define TOGGLE_BIT(_x, _m) (_x) = (_x) & (_m) ? (_x) & ~(_m) : (_x) |
> > (_m)
>
> simple XOR :)?
>
> _x ^= _m;

oh yeah, looks much better :)
thanks! 

bruno