On Tue, 2013-10-29 at 23:39 +0100, Luis R. Rodriguez wrote:
> As it stands dynamic user regulatory domain support is
> only possible for a few programmed regulatory domains as
> a few countries do not allow for this.
[]
> diff --git a/drivers/net/wireless/ath/regd.c b/drivers/net/wireless/ath/regd.c
][
> @@ -296,6 +296,7 @@ static void ath_force_no_ir_freq(struct wiphy *wiphy, u16 center_freq)
>
> static void
> __ath_reg_apply_beaconing_flags(struct wiphy *wiphy,
> + struct ath_regulatory *reg,
> enum nl80211_reg_initiator initiator,
> struct ieee80211_channel *ch)
> {
> @@ -307,6 +308,9 @@ __ath_reg_apply_beaconing_flags(struct wiphy *wiphy,
> case NL80211_REGDOM_SET_BY_COUNTRY_IE:
> ath_force_clear_no_ir_chan(wiphy, ch);
> break;
> + case NL80211_REGDOM_SET_BY_USER:
> + if (ath_reg_dyn_country_user_allow(reg))
> + ath_force_clear_no_ir_chan(wiphy, ch);
Nicer to add explicit /* fallthrough */
> default:
> if (ch->beacon_found)
> ch->flags &= ~IEEE80211_CHAN_NO_IR;
> @@ -370,6 +376,11 @@ ath_reg_apply_ir_flags(struct wiphy *wiphy,
> ath_force_clear_no_ir_freq(wiphy, 2467);
> ath_force_clear_no_ir_freq(wiphy, 2472);
> break;
> + case NL80211_REGDOM_SET_BY_USER:
> + if (!ath_reg_dyn_country_user_allow(reg))
> + break;
> + ath_force_clear_no_ir_freq(wiphy, 2467);
> + ath_force_clear_no_ir_freq(wiphy, 2472);
Missing break?
> default:
> ath_force_no_ir_freq(wiphy, 2467);
> ath_force_no_ir_freq(wiphy, 2472);
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
