Re: [spi-devel-general] [Patch 0/4] IndustrialIO subsystem (ADCs,accelerometers etc)
From: Eric Piel
Date: Thu Jul 24 2008 - 08:37:55 EST
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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/