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? -- Kalle Valo -- 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