Re: [spi-devel-general] [Patch 0/4] IndustrialIO subsystem (ADCs, accelerometers etc)

[Eric Piel <eric.piel@tremplin-utc.net>]

Jonathan Cameron schreef:
:
>> * If the accelerometer is soldered on the computer, define once for all 
>> to which _physical_ movement corresponds which axis (eg: a laptop on its 
>> normal position going up has axis Z increasing).
> Agreed, though this is more documentation (and strict enforcement on drivers)
Indeed, it's more about defining conventions. A one page document saying 
to userspace developers how they can expect any accelerometer driver 
will behave is desperately needed. The more conventions, the more all 
the drivers will behave similarly, and the easier it will be for 
userspace programs to be written :-)

>> * Free fall event. Either it's hardware detected, or the accelerometer 
>> infrastructure will detect it in software.
> Hadn't thought of doing that in software - should be relatively straight
> forward, though would involve a fairly large overhead if the intention
> is to detect it fast enough to park hardware etc. (similar to that for
> the ring buffer I guess).  I'll look into getting that done for the next
> version (maybe driver specific for now).
Yes, on a second though, this is a low priority point. If userspace is 
able to know if the hardware has or not freefall detection, it should be 
possible to just implement the software detection in the userspace 
daemon. People might come up with lots of "clever" algorithms for that, 
so it might be better to not do too much in the kernel ;-)


> 
> At the moment the big missing element of the subsystem is an easy way of
> querying what is there. (proc interface similar to that for the input subsystem)
You mean /sys/class/input/, right? Indeed, something inspired by the 
input subsystem should work well.

See you,
Eric