On Fri, Jun 22, 2012 at 02:44:14PM +0300, Kalle Valo wrote: > Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> writes: > > >> > - qCoff = qCoff & 0x7f; > >> > + qCoff &= 0x7f; > >> > >> I'm curious, how does a change like this fix anything? To me it just > >> looks same functionality, just a different operator is used. Am I > >> missing something? > > > > Though it doesn't make a difference, somehow it fixes smatch warning. > > I agree it is not the right fix. > > Ok, maybe it's just that smatch isn't clever enough with the &= operator. > Dan? Ugh... I think it's debugging code that leaked into the wild. I will remove that. regards, dan carpenter -- 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
