> @@ -3552,8 +3552,7 @@ void brcms_c_switch_shortslot(struct brcms_c_info *wlc, bool shortslot)
> if (wlc->shortslot)
> cfg->current_bss->capability |=
> WLAN_CAPABILITY_SHORT_SLOT_TIME;
> - END_FOREACH_BSS
> -
> + END_FOREACH_BSS /* this comment fixes a checkpatch warning */
I don't have strong feelings about this, but another way to do that
would be to write it like:
FOR_EACH(blah, blah) {
frob();
frob();
frob();
} END_FOREACH_BSS();
Or if you know perl, you could look into why checkpatch is printing
the wrong warning here...
regards,
dan carpenter
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
