On Tue, Jul 23, 2024 at 09:09:58AM +0200, Sascha Hauer wrote: > bss_config->key_mgmt specifies the AKM suites that are usable in > hardware. This variable is set to specific values while iterating > over the advertised AKM suites. This means the final value of the > variable depends on the order of the entries in the AKM suites > array. Instead of setting the variable, just set the relevant bits > in the key_mgmt bit field to make us independent of the order of > entries. > This behaviour is derived from the downstream driver that does the > same. > Also, set bss_config->key_mgmt to zero explicitly right before the > loop. bss_config has been zero allocated by the caller already, but > do so again to safe the reader from following the code path. > > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > --- > drivers/net/wireless/marvell/mwifiex/uap_cmd.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Acked-by: Brian Norris <briannorris@xxxxxxxxxxxx>