From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: Gertjan van Wingerde <gwingerde@kpnplanet.nl>
Cc: Ivo van Doorn <ivdoorn@gmail.com>,
Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
"John W. Linville" <linville@tuxdriver.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"rt2400-devel@lists.sourceforge.net"
<rt2400-devel@lists.sourceforge.net>
Subject: Re: [PATCH] rt2x00: Provide regulatory hint with rt2500pci/usb
Date: Tue, 13 Jan 2009 14:07:09 -0800 [thread overview]
Message-ID: <20090113220708.GS4416@tesla> (raw)
In-Reply-To: <4963D704.5030502@kpnplanet.nl>
On Tue, Jan 06, 2009 at 02:11:16PM -0800, Gertjan van Wingerde wrote:
> On 01/06/09 22:51, Ivo van Doorn wrote:
> > On Tuesday 06 January 2009, Luis R. Rodriguez wrote:
> >>>>
> >>>> Gertjan, do you know if there is any hardware with valid GEO data in the EEPROM?
> >>>>
> >>>>
> >>> As mentioned in my previous email, I do have devices that have valid GEO
> >>> data in the EEPROM. So, we should be able to use that.
> >>> Also, for the dual band cards, it does seem that at least the numerical
> >>> values are similar for both bands (although I'm not sure that same
> >>> numerical values means same regulatory domain). This is at least the
> >>> case on all the cards that I own.
> >>>
> >> If that's the case and since dual band cards will most likely have 2ghz support
> >> why not just provide regulatory_hint() based on the 2ghz band all the time?
> >>
> >
> > Sounds like the most reasonable solution to me as well.
>
> I had a *slightly* more sophisticated model in mind: Check for dual-band
> devices whether the EEPROM values match. If they do, provide the
> regulatory_hint(), if they don't then just log an error, stating that
> the EEPROM cannot make up its mind in which geography it belongs. Just
> to prevent some unexpected behavior for devices that don't have sane
> EEPROM values.
The documentation advises for the EEPROM to not be relied on though
so if you expect the EEPROM regulatory domain code to be valid you
are going against the documentation.
> However, that is all based on the assumption that we can match the
> numerical values of the two band geography domains. So far I haven't
> been able to establish that, and, moreover, the a-band field allows for
> values 0-10 where the bg-band field allows for values 0-7. So, we have
> to start matching these values against their defined allowed channels,
> and against the regulatory definitions, to see if we can actually match
> the values.
Yeah -- pretty messy...
BTW you may want to look at the patch:
[PATCH 05/12] cfg80211: add regulatory_set_custom_rd()
Which you can also use if you have very custom regulatory domains which
you must follow. We are using that for world regulatory domains which
are custom, for example. I would advise against using this for a country
regulatory hint but if you have no other option then that can be used.
I'm still inclined to believe the documentation though -- again you may
want to check with the vendor.
Luis
next prev parent reply other threads:[~2009-01-13 22:07 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-04 19:21 [PATCH] rt2x00: Provide regulatory hint with rt2500pci/usb Ivo van Doorn
2009-01-05 20:08 ` Luis R. Rodriguez
2009-01-05 21:29 ` Gertjan van Wingerde
2009-01-05 22:21 ` Ivo van Doorn
2009-01-05 23:45 ` Luis R. Rodriguez
2009-01-06 17:32 ` Ivo van Doorn
2009-01-06 17:36 ` Luis R. Rodriguez
2009-01-06 17:47 ` Ivo van Doorn
2009-01-06 20:39 ` Gertjan van Wingerde
2009-01-06 20:49 ` Luis R. Rodriguez
2009-01-06 21:51 ` Ivo van Doorn
2009-01-06 22:11 ` Gertjan van Wingerde
2009-01-13 22:07 ` Luis R. Rodriguez [this message]
2009-01-06 20:32 ` Gertjan van Wingerde
2009-01-06 20:47 ` Luis R. Rodriguez
2009-01-06 22:15 ` Gertjan van Wingerde
2009-01-06 22:23 ` Luis R. Rodriguez
2009-01-06 22:50 ` Gertjan van Wingerde
2009-01-06 20:17 ` Gertjan van Wingerde
2009-01-06 21:52 ` Ivo van Doorn
2009-01-06 22:06 ` Gertjan van Wingerde
2009-01-06 22:12 ` Ivo van Doorn
2009-01-06 22:17 ` 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=20090113220708.GS4416@tesla \
--to=lrodriguez@atheros.com \
--cc=Luis.Rodriguez@Atheros.com \
--cc=gwingerde@kpnplanet.nl \
--cc=ivdoorn@gmail.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=rt2400-devel@lists.sourceforge.net \
/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).