> @@ -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
--
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
