Dear Srinivas,
This patchset works great on my system (YOGA ThinkPad 260)! The only
issue I see is that adaptive screen-rotation/brightness via iio-sensor-
proxy does not work straightaway from boot, instead it works after a
first suspend and wake-up. This is despite

~> cat /usr/lib/systemd/system/iio-sensor-proxy.service 
[Unit]
Description=IIO Sensor Proxy service
After=multi-user.target
...

But, this might be the by-now well-known issue:
https://github.com/hadess/iio-sensor-proxy/issues/66
but as I have reported earlier that, with previous patchset versions
and the same configuration, this was working right off a cold boot.

Thanks and best wishes.

On Wed, 2016-07-06 at 11:51 -0700, Srinivas Pandruvada wrote:

Change log
v3:
- Rebased to linux-next as there are some new bus ids are added to
linux-next
- Documentation changes suggested by Jonathan
- The PCI device name for BXTP is changed to APL
- Added module parameter to enable DMA as some older FW
have issues in DMA support.
- On resume just restart to resume faster insted of reset for
S3 compliant platforms

Thanks to Grant Likely and Atri Bhattacharya for test. 
I will add Tested-by to commits for the final version of the patches.

v2:
- Overview in documentation show analogy with usbhid implementation
- sparse errors for statics. Also pointed by Jiri
- Clearly marking exported function header file. Clean up all exports
unused inteface functions
- Changed to tristate from boolean as pointed by Jiri:
this required remove/unload functions
- Prevent crash when ISH enabled on non supported platform
- Break client.c to smaller part by seprating buffer allocations
- move bus register/unregister to ishtp module
- There is only one config symbol INTEL_ISH_HID, removed silent
config for TRANSPORT and IPC


Starting from Cherrytrail, multiple generation of Intel processors
offers
on package sensor hub. Several recent tablets, 2-in-1 convertible
laptops
are using ISH instead of external sensor hubs. This resulted in lack
of
support of sensor function like device rotation and auto backlight
adjustment.
In addition, depending on the OEM implementation, support of ISH is
required
to support low power sleep states.

The support of ISH on Linux platforms is not new. Android platforms
with
Intel SoCs had this support for a while submitted by Daniel Drubin. 
This patcheset is reusing most of those changes with  clean up and
removing Android platform specific changes.

The user mode ABI is still same as external sensor hubs using Linux
IIO. So existing user mode software should still work.
This series primarily brings in new HID transport used in ISH.

Thanks to the community members who tested RFC patches and provided
feedback.

For users testing on Linux distributions using IIO sensor proxy,
a short term work around is required till we have debugged this
issue.
In systemd unit file iio-sensor-proxy.service
In the section "[Unit]" add
After=multi-user.target

Daniel Drubin (3):
  hid: intel_ish-hid: ISH Transport layer
  hid: intel-ish-hid: ipc layer
  hid: intel-ish-hid: ISH HID client driver

Srinivas Pandruvada (3):
  Documentation: hid: Intel ISH HID document
  iio: hid-sensors: use asynchronous resume
  hid: hid-sensor-hub: Add ISH quirk

 Documentation/hid/intel-ish-hid.txt                | 454 ++++++++++
 drivers/hid/Kconfig                                |   2 +
 drivers/hid/Makefile                               |   2 +
 drivers/hid/hid-sensor-hub.c                       |   4 +
 drivers/hid/intel-ish-hid/Kconfig                  |  17 +
 drivers/hid/intel-ish-hid/Makefile                 |  22 +
 drivers/hid/intel-ish-hid/ipc/hw-ish-regs.h        | 220 +++++
 drivers/hid/intel-ish-hid/ipc/hw-ish.h             |  71 ++
 drivers/hid/intel-ish-hid/ipc/ipc.c                | 719
++++++++++++++++
 drivers/hid/intel-ish-hid/ipc/pci-ish.c            | 332 ++++++++
 drivers/hid/intel-ish-hid/ipc/utils.h              |  64 ++
 drivers/hid/intel-ish-hid/ishtp-hid-client.c       | 935
+++++++++++++++++++++
 drivers/hid/intel-ish-hid/ishtp-hid.c              | 234 ++++++
 drivers/hid/intel-ish-hid/ishtp-hid.h              | 182 ++++
 drivers/hid/intel-ish-hid/ishtp/bus.c              | 794
+++++++++++++++++
 drivers/hid/intel-ish-hid/ishtp/bus.h              | 113 +++
 drivers/hid/intel-ish-hid/ishtp/client-buffers.c   | 214 +++++
 drivers/hid/intel-ish-hid/ishtp/client.c           | 935
+++++++++++++++++++++
 drivers/hid/intel-ish-hid/ishtp/client.h           | 182 ++++
 drivers/hid/intel-ish-hid/ishtp/dma-if.c           | 178 ++++
 drivers/hid/intel-ish-hid/ishtp/hbm.c              | 912
++++++++++++++++++++
 drivers/hid/intel-ish-hid/ishtp/hbm.h              | 321 +++++++
 drivers/hid/intel-ish-hid/ishtp/init.c             |  93 ++
 drivers/hid/intel-ish-hid/ishtp/ishtp-dev.h        | 277 ++++++
 .../iio/common/hid-sensors/hid-sensor-trigger.c    |  21 +-
 include/linux/hid-sensor-hub.h                     |   1 +
 include/trace/events/intel_ish.h                   |  30 +
 include/uapi/linux/input.h                         |   1 +
 28 files changed, 7329 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/hid/intel-ish-hid.txt
 create mode 100644 drivers/hid/intel-ish-hid/Kconfig
 create mode 100644 drivers/hid/intel-ish-hid/Makefile
 create mode 100644 drivers/hid/intel-ish-hid/ipc/hw-ish-regs.h
 create mode 100644 drivers/hid/intel-ish-hid/ipc/hw-ish.h
 create mode 100644 drivers/hid/intel-ish-hid/ipc/ipc.c
 create mode 100644 drivers/hid/intel-ish-hid/ipc/pci-ish.c
 create mode 100644 drivers/hid/intel-ish-hid/ipc/utils.h
 create mode 100644 drivers/hid/intel-ish-hid/ishtp-hid-client.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp-hid.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp-hid.h
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/bus.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/bus.h
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/client-buffers.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/client.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/client.h
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/dma-if.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/hbm.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/hbm.h
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/init.c
 create mode 100644 drivers/hid/intel-ish-hid/ishtp/ishtp-dev.h
 create mode 100644 include/trace/events/intel_ish.h

-- 

Atri Bhattacharya
Wed 13 Jul 21:29:30 CEST 2016

Sent from openSUSE Tumbleweed (20160710) (x86_64) on my laptop.


--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html