linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tim Gardner <tim.gardner@canonical.com>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: reinette chatre <reinette.chatre@intel.com>,
	"stable@kernel.org" <stable@kernel.org>,
	"John W. Linville" <linville@tuxdriver.com>,
	wireless <linux-wireless@vger.kernel.org>
Subject: Re: Please apply to stable: cfg80211: add support for custom firmware regulatory solutions
Date: Thu, 05 Mar 2009 15:49:30 -0700	[thread overview]
Message-ID: <49B056FA.2020605@canonical.com> (raw)
In-Reply-To: <43e72e890903051124t42ce5c7blfd741da4a2b4fa79@mail.gmail.com>

Luis R. Rodriguez wrote:
> On Thu, Mar 5, 2009 at 9:54 AM, reinette chatre
> <reinette.chatre@intel.com> wrote:
>> On Wed, 2009-03-04 at 13:35 -0800, Luis R. Rodriguez wrote:
>>> Forgot to Cc: stable@kernel.org for this patch during its submission,
>>> this is needed on 2.6.28 as otherwise there is an issue for Intel
>>> cards which get their channels 5 GHz disabled if OLD_REG is set to no
>>> (this is not the default) or the channels 12-14 are disabled if
>>> OLD_REG is set to yes (default) set to no and the ieee80211_module
>>> parameter is not used. The later issue is resolved by userspace as
>>> well but we cannot yet expect 2.6.28 kernels to have enough userspace
>>> interfaces to set the regulatory domain just yet. This is why OLD_REG
>>> is still set to default with 2.6.28.
>>>
>>> 14b9815af3f4fe0e171ee0c4325c31d2a2c1570b
>>> Author: Luis R. Rodriguez <lrodriguez@atheros.com>
>>> Date:   Wed Nov 12 14:22:03 2008 -0800
>>>
>>>    cfg80211: add support for custom firmware regulatory solutions
>>>
>>>    This adds API to cfg80211 to allow wireless drivers to inform
>>>    us if their firmware can handle regulatory considerations *and*
>>>    they cannot map these regulatory domains to an ISO / IEC 3166
>>>    alpha2. In these cases we skip the first regulatory hint instead
>>>    of expecting the driver to build their own regulatory structure,
>>>    providing us with an alpha2, or using the reg_notifier().
>>>
>>>    Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
>>>    Acked-by: Zhu Yi <yi.zhu@intel.com>
>>>    Signed-off-by: John W. Linville <linville@tuxdriver.com>
>> Could you please also add commit
>> ea4a82dceec7b5782b1259079c8de508d0afe33a? This is the commit that
>> enables the Intel cards to take advantage of the parameter introduced in
>> previous commit.
>>
>> commit ea4a82dceec7b5782b1259079c8de508d0afe33a
>> Author: Luis R. Rodriguez <lrodriguez@atheros.com>
>> Date:   Wed Nov 12 14:22:04 2008 -0800
>>
>>    iwlwifi: enable custom fw regulatory solution
>>
>>    This enables the custom firmware regulatory solution option
>>    on iwlwifi drivers. These devices are uncapable of mapping their
>>    EEPROM regulatory domain to a specific ISO / IEC alpha2.
>>    Although the new 11n devices (>= iwl 5000) have only
>>    3 regultaory SKUs -- MOW, ABG (no N) and BG -- the older
>>    devices (3945 and 4965) have a more complex SKU arrangement
>>    and therefore its not practical to move this to the driver.
>>
>>    Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
>>    Acked-by: Zhu Yi <yi.zhu@intel.com>
>>    Signed-off-by: John W. Linville <linville@tuxdriver.com>
> 
> Doh yes that would be required or it would make this pointless.
> 
>   Luis
> 

Looks like it requires a preparatory commit:

commit f3b407fba52e1b86ca286ee7c218a4fb00bd29e0
Author: Johannes Berg <johannes@sipsolutions.net>
Date:   Tue Oct 21 09:57:41 2008 +0200

    wireless: remove cfg80211_reg_mutex

    This mutex is wrong, we use cfg80211_drv_mutex (which should
    possibly be renamed to just cfg80211_mutex) everywhere except
    in one place, fix that and get rid of the extra mutex.

    Also get rid of a spurious regulatory_requests list definition.

    Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>


-- 
Tim Gardner tim.gardner@canonical.com

  reply	other threads:[~2009-03-05 23:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-04 21:35 Please apply to stable: cfg80211: add support for custom firmware regulatory solutions Luis R. Rodriguez
2009-03-05 17:54 ` reinette chatre
2009-03-05 19:24   ` Luis R. Rodriguez
2009-03-05 22:49     ` Tim Gardner [this message]
2009-03-05 23:13       ` Luis R. Rodriguez
2009-03-06  0:26         ` Tim Gardner
2009-03-06  0:33           ` Luis R. Rodriguez
2009-03-06 14:36             ` John W. Linville
2009-03-06 22:00               ` Luis R. Rodriguez

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49B056FA.2020605@canonical.com \
    --to=tim.gardner@canonical.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=mcgrof@gmail.com \
    --cc=reinette.chatre@intel.com \
    --cc=stable@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).