Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> writes: > This patch fixes below smatch warnings > > drivers/net/wireless/ath/ath9k/ar9002_phy.c:275 > ar9002_hw_spur_mitigate() Error invalid range 576717 to 471859 > drivers/net/wireless/ath/ath9k/ar5008_phy.c:323 > ar5008_hw_spur_mitigate() Error invalid range 555746 to 492830 > drivers/net/wireless/ath/ath9k/ar5008_phy.c:326 > ar5008_hw_spur_mitigate() Error invalid range 587 to 481 > drivers/net/wireless/ath/ath9k/ar9003_calib.c:272 > ar9003_hw_iqcalibrate() Error invalid range 65 to 63 > > Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> [...] > - 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? -- 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
