Amitkumar Karwar <[email protected]> writes:

> From: Prameela Rani Garnepudi <[email protected]>
>
> EEPROM read frame is sent during device initialization to read mac address.
> The format of the frame is modified in firmware to include eeprom length
> and offset. This frame does not return firmware version now. Also same
> frame is sent again to read rf type and band information.
>
> Signed-off-by: Prameela Rani Garnepudi <[email protected]>
> Signed-off-by: Amitkumar Karwar <[email protected]>

[...]

> --- a/drivers/net/wireless/rsi/rsi_main.h
> +++ b/drivers/net/wireless/rsi/rsi_main.h
> @@ -37,6 +37,7 @@ enum RSI_FSM_STATES {
>       FSM_COMMON_DEV_PARAMS_SENT,
>       FSM_BOOT_PARAMS_SENT,
>       FSM_EEPROM_READ_MAC_ADDR,
> +     FSM_EEPROM_READ_RF_TYPE,
>       FSM_RESET_MAC_SENT,
>       FSM_RADIO_CAPS_SENT,
>       FSM_BB_RF_PROG_SENT,

Shouldn't you also update fsm_state array, just like Colin fixed earlier
this week:

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/?id=58828680af495bc4863af3a006f2018c71714665

I think a some kind of build time check for this is in order, should be
doable with BUILD_BUG_ON().

-- 
Kalle Valo

Reply via email to