On 08/12/2011 01:59 PM, Franky Lin wrote:
If you are refering to this line >>> + } while ((e = brcmf_deq_event(cfg_priv))); The extra parentheses are added to fix a compiler warning: drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c: In function ‘brcmf_event_handler’: drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c:3576: warning: suggest parentheses around assignment used as truth value
Does checkpatch allow that assignment in the while statement? I would expect it to want you to
e = brcmf_deq_event(cfg_priv); } while (e); Larry -- 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
