From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: Shahar Or <mightyiampresence@gmail.com>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
"John W. Linville" <linville@tuxdriver.com>,
Bob Copeland <me@bobcopeland.com>,
"ath5k-devel@lists.ath5k.org" <ath5k-devel@lists.ath5k.org>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [ath5k-devel] [PATCH] ath5k: add support for Dell Vostro A860 LED
Date: Fri, 18 Dec 2009 09:19:45 -0800 [thread overview]
Message-ID: <20091218171945.GE6231@tux> (raw)
In-Reply-To: <bf87febf0912180903x168462ffw6c9faeac0cd1596c@mail.gmail.com>
On Fri, Dec 18, 2009 at 09:03:26AM -0800, Shahar Or wrote:
> On Fri, Dec 18, 2009 at 5:37 PM, Luis R. Rodriguez
> <lrodriguez@atheros.com> wrote:
> >> If we have all the IDs from the INF file, then why do we need this in
> >> userspace? Is the list growing?
> >
> > No, but for other newer hardware/drivers it can likely grow, so my
> > comment was more of a generic driver approach to this.
>
> Is this a linux-wireless thing
You mean my suggestion? It depends, I personally am not aware of
upstream drivers keeping tabs on knobs in userspace currently
but I do think things like configs are used to tweak, for example
Oracle db stuff. configs can technically also be used to expose
knobs for example for non-generic device configuration stuff
for example but as far as I can tell no one has really used it
yet for this. It does' mean its not possible. The other question
is where do we stuff this in userspace and if it really makes
sense to start following that practice. For GPIO pin mapping,
perhaps given the number of patches I see for it but since its
the only thing I see a good use of perhaps its not worth it.
The kernel should always use default sensible values but since
this is more of database thing it makes sence to start thinking
istead of userspace for a better solution. Examples of moving
large dbs to userspace could be the preference on USB side to
not keep the USB vendor/device IDs names in the kernel, and also
the regulatory database stuff.
> or do other areas of linux keep some
> data in userspace?
I gave a few examples, not sure what else the kernel keeps in
userspace for data which is generic like that. Moving PCI device
vendor/device names might be good example.
Luis
prev parent reply other threads:[~2009-12-18 17:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-03 13:07 [PATCH] ath5k: add support for Dell Vostro A860 LED Shahar Or
2009-12-04 19:23 ` John W. Linville
2009-12-07 10:05 ` Shahar Or
2009-12-07 21:59 ` John W. Linville
2009-12-08 14:57 ` [ath5k-devel] " Bob Copeland
2009-12-08 16:01 ` Luis R. Rodriguez
2009-12-08 16:06 ` John W. Linville
2009-12-08 16:30 ` Luis R. Rodriguez
2009-12-18 11:42 ` Shahar Or
2009-12-18 15:37 ` Luis R. Rodriguez
2009-12-18 16:23 ` Luis R. Rodriguez
2009-12-18 17:03 ` Shahar Or
2009-12-18 17:19 ` Luis R. Rodriguez [this message]
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=20091218171945.GE6231@tux \
--to=lrodriguez@atheros.com \
--cc=Luis.Rodriguez@Atheros.com \
--cc=ath5k-devel@lists.ath5k.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=me@bobcopeland.com \
--cc=mightyiampresence@gmail.com \
/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).