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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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? 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
--
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




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux