Kalle Valo <kvalo@xxxxxxxxxxxxxx> writes:
> <yhchuang@xxxxxxxxxxx> writes:
>
>> From: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx>
>>
>> chip files Realtek 802.11ac wireless network chips
>> 8822B & 8822C series files
>>
>> Reviewed-by: Stanislaw Gruszka <sgruszka@xxxxxxxxxx>
>> Signed-off-by: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx>
>
> [...]
>
>> +static struct rtw_pwr_seq_cmd trans_carddis_to_cardemu_8822b[] = {
>> + {0x0086,
>> + RTW_PWR_CUT_ALL_MSK,
>> + RTW_PWR_INTF_SDIO_MSK,
>> + RTW_PWR_ADDR_SDIO,
>> + RTW_PWR_CMD_WRITE, BIT(0), 0},
>
> I see lots of arrays like this which are not const. I think they should
> be const.
And I'm seeing even more of these while I review the driver. So
basically all static variables should be const to make sure you don't
accidentally between driver instances. IIRC there was something in
cfg80211 API which prevent to use const (maybe the channels? can't
remember now) but that's just an exception to the rule.
--
Kalle Valo
