public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: Ben Dooks <ben@fluff.org>
Cc: Jonathan Cameron <Jonathan.Cameron@gmail.com>,
	spi-devel-general@lists.sourceforge.net,
	linux-kernel@vger.kernel.org,
	LM Sensors <lm-sensors@lm-sensors.org>
Subject: Re: [spi-devel-general] Accelerometer, Gyros and ADC's etc within the kernel.
Date: Tue, 27 May 2008 21:01:19 +0400	[thread overview]
Message-ID: <20080527170119.GA1938@polina.dev.rtsoft.ru> (raw)
In-Reply-To: <20080527165021.GA22585@trinity.fluff.org>

On Tue, May 27, 2008 at 05:50:21PM +0100, Ben Dooks wrote:
> On Tue, May 27, 2008 at 08:44:15PM +0400, Anton Vorontsov wrote:
> > Hi Jonathan,
> > 
> > On Tue, May 20, 2008 at 11:04:01AM +0100, Jonathan Cameron wrote:
> > > This email is basically a request for opinions on how and where such sensors
> > > should be integrated into the kernel.
> > > 
> > > To set the scene...
> > > 
> > > Increasing numbers of embedded devices are being supplied attached MEMS
> > > devices (www.xbow.com imote2 etc). Along with more traditional sensors such
> > > as ADC's not being used for hardware monitoring, these do not really 
> > > seem to
> > > fit with in an particular subsystem of the kernel.  A previous 
> > > discussion on
> > > lkml in 2006 considered the accelerometers to be found within some laptop
> > > hard drives, but I haven't been able to track down any more general 
> > > discussions
> > > of such non hardware monitoring sensors.
> > > 
> > > The obvious possibilities are:
> > > 
> > > * To place the various drivers within the spi / i2c etc subsystems as 
> > > relevant.
> > > 
> > > * To place within the hwmon subsystem as this is probably closest.
> > > (there is already at least one straight ADC driver in hwmon)
> > > 
> > > * To create a new subsystem, or perhaps merely sysfs class to contain these
> > >   elements.
> > > 
> > > Typical requirements within an application include simply polling for 
> > > current
> > > readings, and using device triggered interrupts to grab data 
> > > continuously to a
> > > ring buffer, for collection by suitable userspace code.  Obviously it 
> > > would be
> > > desirable to standardize sysfs controls for various calibration 
> > > parameters as
> > > much as possible across the various devices.
> 
> The two drivers i've seen so far use the input subsystem to report
> their data to the user. I'm working on an driver for the SMB380 which
> can be both i2c and spi.
>  
> > Also, I'd mention that most ADC devices could report in "bunched" mode,
> > i.e.
> > 
> > 1. Request ADC readings from pins X, Y, Z1, Z2.
> > 2. Wait for single IRQ
> > 3. Read all the results
> > 
> > At handhelds.org, we've wrote quite good (I think) ADC subsystem,
> > that keeps in mind ADC capabilities. It implements two interfaces:
> > in-kernel (e.g. for touchscreen drivers), and userspace interface via
> > sysfs. I was planning to implement drivers/input/ interface too.
> 
> handhelds.org's track history of getting things into the kernel
> is poor.

Yes, mostly. Because lack of human resources. But something gets done
anyway, e.g. drivers/power/ is already in the mainline. Philipp Zabel is
doing great work for the HTC Magician support (already in mainline, too).

I think there are only few of us who think that mainlining is important,
this is bad. But the thinking improves as times goes by.

So, don't be so pessimistic. :-)

-- 
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2

  reply	other threads:[~2008-05-27 17:01 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-20 10:04 Accelerometer, Gyros and ADC's etc within the kernel Jonathan Cameron
2008-05-20 11:28 ` Jean Delvare
2008-05-20 21:40   ` Hans J. Koch
2008-05-21 10:04     ` Jonathan Cameron
2008-05-21 13:20       ` Jean Delvare
2008-05-21 13:49   ` Dmitry Torokhov
2008-05-21 14:09     ` Henrique de Moraes Holschuh
2008-05-27 17:16     ` [spi-devel-general] " Ben Dooks
2008-05-27 19:01       ` Dmitry Torokhov
2008-05-22  0:52   ` David Brownell
2008-05-22  9:35     ` [spi-devel-general] " Jonathan Cameron
2008-05-26 16:23       ` Jonathan Cameron
2008-06-26 18:01   ` Accelerometer etc subsystem (Update on progress) Jonathan Cameron
2008-06-26 18:26     ` Jonathan Cameron
2008-06-27  2:39     ` Randy Dunlap
2008-06-27  3:29     ` Ben Nizette
2008-06-27  9:45       ` [lm-sensors] " Jonathan Cameron
2008-06-28  8:34         ` Ben Nizette
2008-06-28 15:34           ` Jonathan Cameron
2008-05-20 17:50 ` Accelerometer, Gyros and ADC's etc within the kernel mark gross
2008-05-21  9:40   ` [spi-devel-general] " Jonathan Cameron
2008-05-27 15:43     ` mark gross
2008-05-29 11:57       ` Jonathan Cameron
2008-05-22  0:53   ` David Brownell
2008-05-27 15:56     ` mark gross
2008-05-27 23:42       ` David Brownell
2008-05-27 16:44 ` [spi-devel-general] " Anton Vorontsov
2008-05-27 16:50   ` Ben Dooks
2008-05-27 17:01     ` Anton Vorontsov [this message]
2008-05-27 18:00     ` Jonathan Cameron
2008-05-27 18:12       ` Ben Dooks
2008-05-27 17:59   ` Jonathan Cameron

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=20080527170119.GA1938@polina.dev.rtsoft.ru \
    --to=avorontsov@ru.mvista.com \
    --cc=Jonathan.Cameron@gmail.com \
    --cc=ben@fluff.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=spi-devel-general@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