On Sun, May 23, 2010 at 12:38:48AM +0530, Kalle Valo wrote: > Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx> writes: > > > Signed-off-by: Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx> > > Please describe more, this isn't very informative (if any). Especially > try to answer in the commit log: > > o What was the original bug? How and why did it happen? > o How was it visible to the user? > o How does the fix actually fix the issue? Sure. > o In what version was the bug introduced? This helps backporters. This patch is only for ar9003 and its support is added only recently. No released kernel has the support for this chip. > > > +#define ATH9K_CHECK_AUTO_SLEEP(__sc) \ > > + (__sc->ps_enabled && \ > > + (__sc->sc_ah->caps.hw_caps & ATH9K_HW_CAP_AUTOSLEEP)) > > Like Andrew Morton says: use C instead of CPP. Meaning that this can > be an inline function instead of a macro. will do, thanks. Vasanth -- 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