From: Guenter Roeck <guenter.roeck-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
To: Jonathan Cameron <jic23-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
Cc: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>,
"eric.piel-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org"
<eric.piel-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org>,
"linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org"
<kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>,
"lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org"
<lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org>
Subject: Re: [lm-sensors] [RFC PATCH 0/9] lis3 accelerator feature update
Date: Sat, 2 Oct 2010 06:16:06 -0700 [thread overview]
Message-ID: <20101002131606.GB27524@ericsson.com> (raw)
In-Reply-To: <4CA72519.1070600-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
On Sat, Oct 02, 2010 at 08:27:05AM -0400, Jonathan Cameron wrote:
> On 10/02/10 09:25, Jean Delvare wrote:
> > Hi Guenter,
> >
> > On Fri, 1 Oct 2010 19:53:11 -0700, Guenter Roeck wrote:
> >> On Fri, Oct 01, 2010 at 07:46:47AM -0400, Samu Onkalo wrote:
> >>> This patch set is done to top of 2.6.36-RC5
> >>>
> >>> Changes are tested only with I2C interface using 8bit sensor since I don't
> >>> have other possibilities.
> >>>
> >>> I send this as RFC since changes may affect functionalities or use cases
> >>> which I can't test or I don't know to exist.
> >>>
> >> Wonder if there is a better mailing list to get good (or even any) feedback
> >> for this patchset. Even though the driver resides in the hwmon directory,
> >> it isn't really a hardware monitoring driver. Neither i2c nor hwmon fits well.
> >>
> >> Ideas, anyone ?
> cc lkml? These are very much device specific so only people who can really
> review are those who actually have the part or don't mind spending a fair bit
> of time familiarizing themselves with a fairly complex driver.
>
> Perhaps runtime pm ones want to go to the runtime pm guys? That bit isn't
> really subsystem specific. I'm quite keen to drive that stuff into some of
> my drivers, but haven't done it yet so am not familiar enough with the
> infrastructure.
>
> >
> > In an ideal world, people interested in accelerometers would create a
> > subsystem for it, declare themselves maintainers of it, and all
> > accelerometer drivers which currently leave under drivers/hwmon (hdaps,
> > hp_accel, lis3lv02d and maybe ams) would be moved there.
> Agreed.
> >
> > The current state is very bad, because these drivers are located in a
> > maintained subsystem but they don't belong there. This is what
> > motivated my request to move these drivers away [1]. But as you can see
> > nobody replied.
Yes, I noticed.
> Dmitry often takes a little while to reply, so I wouldn't give up hope yet.
>
> Sadly, despite numerous people wading in from time to time to tell use we
> are doing it all wrong, those who actually are interested day to day in
> these sorts of devices are far too thin on the ground.
>
> Personally, if no one responds from input, I'd just push this driver
> into misc asap. Can't seen anyone objecting to that.
Makes sense to me. Drivers there seem to be maintained by individuals,
as is this driver, so it makes kind of sense. Maybe Eric has some comments ?
Guenter
prev parent reply other threads:[~2010-10-02 13:16 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-01 11:46 [RFC PATCH 0/9] lis3 accelerator feature update Samu Onkalo
2010-10-01 11:46 ` [RFC PATCH 6/9] hwmon: lis3: New parameters to platform data Samu Onkalo
[not found] ` <1285933616-16044-7-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-04 11:37 ` [lm-sensors] " Jonathan Cameron
[not found] ` <1285933616-16044-1-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-01 11:46 ` [RFC PATCH 1/9] hwmon: lis3: pm_runtime support Samu Onkalo
[not found] ` <1285933616-16044-2-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-02 17:14 ` [lm-sensors] " Jonathan Cameron
[not found] ` <4CA76875.1040508-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
2010-10-03 5:03 ` Onkalo Samu
[not found] ` <1286082228.2064.14.camel-Vo7XL3ix0D0UEupzmRo7jhl4MBrZKKet0E9HWUfgJXw@public.gmane.org>
2010-10-03 11:18 ` Jonathan Cameron
2010-10-01 11:46 ` [RFC PATCH 2/9] hwmon: lis3: regulator control Samu Onkalo
[not found] ` <1285933616-16044-3-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-02 17:33 ` [lm-sensors] " Jonathan Cameron
[not found] ` <4CA76CDA.4040803-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
2010-10-03 5:25 ` Onkalo Samu
[not found] ` <1286083558.2064.35.camel-Vo7XL3ix0D0UEupzmRo7jhl4MBrZKKet0E9HWUfgJXw@public.gmane.org>
2010-10-03 11:21 ` Jonathan Cameron
2010-10-03 11:53 ` David Lutolf
2010-10-01 11:46 ` [RFC PATCH 3/9] hwmon: lis3: Cleanup interrupt handling Samu Onkalo
2010-10-01 11:46 ` [RFC PATCH 4/9] hwmon: lis3: Update coordinates at polled device open Samu Onkalo
2010-10-01 11:46 ` [RFC PATCH 5/9] hwmon: lis3: Power on corrections Samu Onkalo
[not found] ` <1285933616-16044-6-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-02 17:43 ` [lm-sensors] " Jonathan Cameron
2010-10-01 11:46 ` [RFC PATCH 7/9] hwmon: lis3: Adjust fuzziness for 8 bit device Samu Onkalo
2010-10-01 11:46 ` [RFC PATCH 8/9] hwmon: lis3: use block read to access data registers Samu Onkalo
[not found] ` <1285933616-16044-9-git-send-email-samu.p.onkalo-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-10-04 11:41 ` [lm-sensors] " Jonathan Cameron
[not found] ` <4CA9BD6E.6040002-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
2010-10-04 13:29 ` Guenter Roeck
2010-10-01 11:46 ` [RFC PATCH 9/9] hwmon: lis3: Enhance lis3 selftest with IRQ line test Samu Onkalo
2010-10-02 2:53 ` [RFC PATCH 0/9] lis3 accelerator feature update Guenter Roeck
[not found] ` <20101002025311.GA25875-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
2010-10-02 8:25 ` Jean Delvare
[not found] ` <20101002102528.2955d95a-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org>
2010-10-02 12:27 ` [lm-sensors] " Jonathan Cameron
[not found] ` <4CA72519.1070600-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
2010-10-02 13:16 ` Guenter Roeck [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=20101002131606.GB27524@ericsson.com \
--to=guenter.roeck-izefyvvap7pwk0htik3j/w@public.gmane.org \
--cc=eric.piel-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org \
--cc=jic23-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org \
--cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
--cc=kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org \
--cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.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