net: l2: wifi: fix AP sets band failed with channel 0

[MaochenWang1]

Should support setting band for both STA and SAP mode.

[jukkar]

Perhaps the commit subject
fix AP sets band failed with channel 0
should be clarified, it does not seem to reflect what is being changed.
The change is to support both SAP and STA mode for this command, why there is mention to "channel 0" in the first place.

[MaochenWang1]

Perhaps the commit subject fix AP sets band failed with channel 0 should be clarified, it does not seem to reflect what is being changed. The change is to support both SAP and STA mode for this command, why there is mention to "channel 0" in the first place.

Hi @jukkar , as there is following check below, and this commit can fix the issue that AP sets band failed with channel 0
if (params->channel == 0 && params->band == WIFI_FREQ_BAND_UNKNOWN) {
PR_ERROR("Band not provided when channel is 0\n");
return -EINVAL;
}

[krish2718]

Perhaps the commit subject fix AP sets band failed with channel 0 should be clarified, it does not seem to reflect what is being changed. The change is to support both SAP and STA mode for this command, why there is mention to "channel 0" in the first place.

Hi @jukkar , as there is following check below, and this commit can fix the issue that AP sets band failed with channel 0 if (params->channel == 0 && params->band == WIFI_FREQ_BAND_UNKNOWN) { PR_ERROR("Band not provided when channel is 0\n"); return -EINVAL; }

BTW, to convery any channel, there is already a macro in use WIFI_CHANNEL_ANY please switch to that, a non-zero is used to convey explicit intention than default value of 0 (memset or static etc).