2017-05-19 04:56:49

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH v2 4/5] wil6210: add option to ignore OTA regulatory hints

Maya Erez <[email protected]> writes:

> On some platforms, the regulatory domain (country) is set
> using mechanisms external to WIFI, such as cellular modem
> and GPS. In these scenarios the regulatory hints that
> are received over the air (in beacons and similar) can
> conflict and even cause an incorrect country to be set.
> Add an option to ignore the OTA regulatory hints to better
> support such scenarios.
>
> Signed-off-by: Lior David <[email protected]>
> Signed-off-by: Maya Erez <[email protected]>
> ---
> drivers/net/wireless/ath/wil6210/cfg80211.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/drivers/net/wireless/ath/wil6210/cfg80211.c b/drivers/net/wi=
reless/ath/wil6210/cfg80211.c
> index 567fe43..0f297c0 100644
> --- a/drivers/net/wireless/ath/wil6210/cfg80211.c
> +++ b/drivers/net/wireless/ath/wil6210/cfg80211.c
> @@ -26,6 +26,11 @@
> module_param(disable_ap_sme, bool, 0444);
> MODULE_PARM_DESC(disable_ap_sme, " let user space handle AP mode SME");
> =20
> +static bool ignore_reg_hints;
> +module_param(ignore_reg_hints, bool, 0444);
> +MODULE_PARM_DESC(ignore_reg_hints,
> + " Ignore OTA regulatory hints (Default: false)");
> +
> #define CHAN60G(_channel, _flags) { \
> .band =3D NL80211_BAND_60GHZ, \
> .center_freq =3D 56160 + (2160 * (_channel)), \
> @@ -1763,6 +1768,11 @@ static void wil_wiphy_init(struct wiphy *wiphy)
> =20
> wiphy->n_vendor_commands =3D ARRAY_SIZE(wil_nl80211_vendor_commands);
> wiphy->vendor_commands =3D wil_nl80211_vendor_commands;
> +
> + if (ignore_reg_hints) {
> + wiphy->regulatory_flags |=3D REGULATORY_DISABLE_BEACON_HINTS;
> + wiphy->regulatory_flags |=3D REGULATORY_COUNTRY_IE_IGNORE;
> + }

I wonder should something like this be in cfg80211? Because I doubt
wil6210 is the only driver having this problem.

--=20
Kalle Valo=


2017-05-21 20:07:43

by Maya Haim

[permalink] [raw]
Subject: RE: [PATCH v2 4/5] wil6210: add option to ignore OTA regulatory hints

[email protected] wrote:
>=20
> I wonder should something like this be in cfg80211? Because I doubt
> wil6210 is the only driver having this problem.

We will check that and share our conclusions.
Can you please drop this patch for now and apply the rest of the patches
(if there are no additional comments)?

>=20
> --
> Kalle Valo

Thanks,
Maya

2017-05-22 14:23:03

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH v2 4/5] wil6210: add option to ignore OTA regulatory hints

Maya Haim <[email protected]> writes:

> [email protected] wrote:
>>=20
>> I wonder should something like this be in cfg80211? Because I doubt
>> wil6210 is the only driver having this problem.
>
> We will check that and share our conclusions.
> Can you please drop this patch for now and apply the rest of the patches
> (if there are no additional comments)?

Sure, I have now dropped this patch. The rest are still on my queue.

--=20
Kalle Valo=