Re: brcm80211: drop unreachable else case

2015-02-26 Thread Kalle Valo
> the if/elseif/else is exhaustive - there is no 4th case given the > rssi_ctrl_mask = RADIO_2055_NBRSSI_SEL | RADIO_2055_WBRSSI_G1_SEL | > RADIO_2055_WBRSSI_G2_SEL; > so this unreachable else case (dead code) can be dropped. > > Signed-off-by: Nicholas Mc Guire Thanks, applied to

Re: brcm80211: drop unreachable else case

2015-02-26 Thread Kalle Valo
the if/elseif/else is exhaustive - there is no 4th case given the rssi_ctrl_mask = RADIO_2055_NBRSSI_SEL | RADIO_2055_WBRSSI_G1_SEL | RADIO_2055_WBRSSI_G2_SEL; so this unreachable else case (dead code) can be dropped. Signed-off-by: Nicholas Mc Guire hof...@osadl.org Thanks, applied

[PATCH] brcm80211: drop unreachable else case

2015-02-06 Thread Nicholas Mc Guire
the if/elseif/else is exhaustive - there is no 4th case given the rssi_ctrl_mask = RADIO_2055_NBRSSI_SEL | RADIO_2055_WBRSSI_G1_SEL | RADIO_2055_WBRSSI_G2_SEL; so this unreachable else case (dead code) can be dropped. Signed-off-by: Nicholas Mc Guire --- It seems that there are only three

[PATCH] brcm80211: drop unreachable else case

2015-02-06 Thread Nicholas Mc Guire
the if/elseif/else is exhaustive - there is no 4th case given the rssi_ctrl_mask = RADIO_2055_NBRSSI_SEL | RADIO_2055_WBRSSI_G1_SEL | RADIO_2055_WBRSSI_G2_SEL; so this unreachable else case (dead code) can be dropped. Signed-off-by: Nicholas Mc Guire hof...@osadl.org --- It seems that