All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Éric Piel" <eric.piel@tremplin-utc.net>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Darren Hart <dvhart@linux.intel.com>,
	"lkml, " <linux-kernel@vger.kernel.org>,
	Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Carmine Iascone <carmine.iascone@st.com>,
	Matteo Dameno <matteo.dameno@st.com>
Subject: Re: LIS331DLH accelerometer driver, IIO or not?
Date: Sat, 26 May 2012 13:53:06 +0200	[thread overview]
Message-ID: <4FC0C422.8080605@tremplin-utc.net> (raw)
In-Reply-To: <20120525051051.GA3135@kroah.com>

On 25-05-12 07:10, Greg Kroah-Hartman wrote:
> On Thu, May 24, 2012 at 09:29:53PM -0700, Darren Hart wrote:
>> I'm working to enable the LIS331DLH accelerometer on the Fish River
>> Island II embedded atom development kit.
>>
>> I am more interested in enabling people to do bizarre and interesting
>> things with the device, so I'm leaning toward continuing with my IIO
>> implementation.
>
> Make it an IIO driver and then we can delete the misc driver, which
> shouldn't have snuck in there in the first place :)
>

To be more fair to the misc driver, I wouldn't say it snucked in there, 
but more "it ended up there as the least worse place" ;-) Actually, the 
main problem is that there seemed to be no maintainer interested in 
taking care of accelerometer devices. Now that the IIO subsystem is out 
of staging, it might be a right place. That said, I don't know much 
about the user interface to IIO. I know that I liked the idea of having 
an joystick device created for an accelerometer because that allows to 
get many programs to access the device almost without any modifications.

I'd happy to help merge the lis3lv02d driver into IIO. IMHO, the main 
steps are:
  * make sure all the various buses are supported (e.g., I²C, SPI, and 
also "ACPI-HP")
  * ensure the various versions of the accelerometer are supported 
(there are 3 supported currently)
  * check that the driver is automatically loaded on HP laptops (via 
ACPI entry)
  * for each of the current interfaces decide if they should be ported 
or dropped (/dev/js*, /dev/freefall, sysfs...)

What do you think Jonathan?

Cheers,
Éric

  parent reply	other threads:[~2012-05-26 11:53 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-25  4:29 LIS331DLH accelerometer driver, IIO or not? Darren Hart
2012-05-25  5:10 ` Greg Kroah-Hartman
2012-05-25  7:45   ` Jonathan Cameron
2012-05-25 15:23     ` Darren Hart
2012-05-25 16:21       ` Jonathan Cameron
2012-05-26 11:53   ` Éric Piel [this message]
2012-05-26 13:59     ` Alan Cox
2012-05-27  3:16       ` Darren Hart
2012-05-26 17:40     ` Jonathan Cameron
2012-05-27  3:14       ` Darren Hart
2012-05-27  9:32         ` Jonathan Cameron
2012-05-27 16:27           ` Darren Hart

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=4FC0C422.8080605@tremplin-utc.net \
    --to=eric.piel@tremplin-utc.net \
    --cc=carmine.iascone@st.com \
    --cc=dvhart@linux.intel.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=jic23@kernel.org \
    --cc=lars@metafoo.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matteo.dameno@st.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.