Wright Feng <Wright.Feng@xxxxxxxxxxx> writes: >>> Oh, I didn't realise ap_isolate can be -1 as struct bss_parameters >>> didn't document that. Can someone submit a patch to fix that? >>> >>> * @ap_isolate: do not forward packets between connected stations >> >> Me too. I assumed it was a boolean reading that description. >> >> Regards, >> Arend > > The ap_isolate -1 value in nl80211_set_bss means not to changed.I > intend to add a check of "params->ap_isolate >= 0" like > ath/wil6210/cfg80211.c does in brcmf_cfg80211_change_bss. And I will > submit another patch to revise the comment in cfg80211.h as below. Let > me know if you concern about it. Great, thanks. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
