From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1410289105.4077.19.camel@hadess.net> Subject: Re: [PATCH] iio:kxcjk-1013: Add support for SMO8500 device From: Bastien Nocera To: Daniel Baluta Cc: linux-iio@vger.kernel.org, Jonathan Cameron Date: Tue, 09 Sep 2014 20:58:25 +0200 In-Reply-To: References: <1410283148.4077.16.camel@hadess.net> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 List-ID: Hey Daniel, 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 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 does your ACPI .dsl file looks like? I uploaded it to the kernel bugzilla for another bug, it's at: https://bugzilla.kernel.org/attachment.cgi?id=149331 > > 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]. I used the upstream linus kernel, which I probably shouldn't have done, but hey, IIO was just one of the several class of devices that needs work on this tablet ;) > Anyhow, > it will conflict with this one [2]. Right. I'm fine to rebase it if it looks correct otherwise. Cheers