login
Login
/
Register
Search
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
September
»
10
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory infrastructure
view
thread
!MAILaRCHIVE_VOTE_RePLACE
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From:
Marcel Holtmann <holtmann@...>
To: Luis R. Rodriguez <lrodriguez@...>
Cc: <linville@...>, <johannes@...>, <linux-wireless@...>, <linux-kernel@...>
Subject:
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory infrastructure
Date: Wednesday, September 10, 2008 - 6:07 pm
Hi Luis,
quoted text
> > While reading through it, I came to think about regulatory_hint(). So is > > there a use case where would give it the alpha2 code and the domain > > itself at the same time? If not, then it would make more sense to split > > this into two functions. > > Nope, you either pass an alpha2 or an rd domain which is built by you > (and in that rd structure you can set the alpha2 to your iso3166 > alpha2 or "99" if unknown). > > > Maybe something regulatory_alpha2_hint() and > > regulatory_domain_hint(). Just a thought. > > That's how I had it originally but decided to condense it to one > routine since as you could see they pretty much do the same thing > except the case where the rd is provided it calls set_regdom(). > Setting it back to use two routines if fine by me too. What is better? > Can we just get this merged and then we can flip it around if > necessary? :) I'm tired of carrying this around.
my take on this is that if from an API perspective you can only use one parameter or the other, then it should be two functions. Regards Marcel --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Marcel Holtmann
, (Wed Sep 10, 2:17 pm)
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Luis R. Rodriguez
, (Wed Sep 10, 4:18 pm)
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Marcel Holtmann
, (Wed Sep 10, 6:07 pm)
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Luis R. Rodriguez
, (Wed Sep 10, 6:18 pm)
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Marcel Holtmann
, (Wed Sep 10, 6:25 pm)
Re: [PATCH 1/2 v6] cfg80211: Add new wireless regulatory inf...
, Johannes Berg
, (Wed Sep 10, 6:30 pm)
Navigation
Create content
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Linus Torvalds
Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Greg KH
[GIT PATCH] driver core patches against 2.6.24
Paul Jackson
Re: [patch 2/2] cpusets: add interleave_over_allowed option
Linus Torvalds
Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes g...
git
:
linux-netdev
:
Mark Lord
Re: 2.6.25-rc8: FTP transfer errors
Gerrit Renker
[PATCH 0/37] dccp: Feature negotiation - last call for comments
David Miller
Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
David Miller
[GIT]: Networking
openbsd-misc
:
Colocation donated by:
Who's online
There are currently
1 user
and
781 guests
online.
Online users
strcmp
Syndicate