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@xxxxxxxxxxxxx> > > --- > > [...] > > > +#define TOGGLE_BIT(_x, _m) (_x) = (_x) & (_m) ? (_x) & ~(_m) : (_x) | > > (_m) > > simple XOR :)? > > _x ^= _m; oh yeah, looks much better :) thanks! bruno - 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
