else if in wnc36xx_bss_info_changed()

Eugene Krasnikov k.eugene.e at gmail.com
Wed May 22 03:59:35 EDT 2013


Agreed.

2013/5/22 Kalle Valo <kvalo at qca.qualcomm.com>:
> Hi,
>
> I noticed this in wcn36xx_bss_info_changed()
>
>         if(changed & BSS_CHANGED_BSSID) {
>                 ....
>         } else if (changed & BSS_CHANGED_BEACON_ENABLED){
>                 ...
>         }
>
> Using else here looks buggy to me. Is there any reason for it?
>
> --
> Kalle Valo
>
> _______________________________________________
> wcn36xx mailing list
> wcn36xx at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/wcn36xx



-- 
Best regards,
Eugene



More information about the wcn36xx mailing list