Re: [PATCH] iio:kxcjk-1013: Add support for SMO8500 device

[Daniel Baluta <daniel.baluta@intel.com>]

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@hadess.net> 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? There is no GpioInt associated with your device in 
the the .dsl file.
What kind of error do you see in the logs?
>> 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.

thanks,
Daniel.

[1] http://www.spinics.net/lists/linux-iio/msg14655.html
[2] https://git.kernel.org/cgit/linux/kernel/git/jic23/iio.git/log/?h=togreg