On Fri, 2014-09-12 at 15:24 +0300, Daniel Baluta wrote: > On 09/11/2014 09:18 PM, Bastien Nocera wrote: > > On Tue, 2014-09-09 at 21:53 +0300, Daniel Baluta wrote: > >> Hi Bastien, > >> > >> On Tue, Sep 9, 2014 at 8:19 PM, Bastien Nocera <hadess@xxxxxxxxxx> wrote: > >>> The Onda v975w tablet contains an accelerometer that's advertised over > >>> ACPI as SMO8500. This device is however a KXCJ9 accelerometer as > >>> can be seen in the Windows driver's INF file. > >>> --- > >>> > >>> I tested the patch, and it works correctly in my testing. However, > >>> there's no GPIO pin for the "data ready" interrupt, and this throws an > >>> error that looks fatal in the logs (could this be downgraded?), and > >>> the name of the device doesn't match the hid-sensor-hub ones, making it > >>> hard for user-space to discover them. > >>> (https://github.com/hadess/iio-sensor-proxy is my attempt at something > >>> that integrates with user-space and the IIO sensors) > >>> > How did you test it? Cat on the /sys files for each of the axis. I can see the values of x/y/z changing depending on the position of the device. > There is no GpioInt associated with your device in > the the .dsl file. I know, that's because the GPIO (according to the driver) is only used to know when there's more data to be read. We just have to poll instead... > What kind of error do you see in the logs? The "acpi gpio get index failed" error when it fails to find a GPIO pin for the "data ready" interrupt. Except that it works just fine without such an interrupt. > >> How does your ACPI .dsl file looks like? > >> > >>> It will also conflict with Daniel's patch here: > >>> http://www.spinics.net/lists/linux-iio/msg14438.html > >> It shouldn't. The patch is already in togreg branch of Jonathan's tree > >> [1]. Anyhow, > >> it will conflict with this one [2]. > > Do you want me to rebase the patch on a particular tree? Do you need any > > particular changes done in my commit message or patch to get this > > merged? > > > I would suggest you wait until Jonathan merges this patch [1] in togreg > branch [2]. > Then rebase your patch on togreg branch and get feedback from Jonathan > and Srinivas. OK, I'll have to monitor those. Cheers -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
