From: Atri Bhattacharya <badshah400@aim.com>
To: linux-iio@vger.kernel.org
Subject: Re: [PATCH v3 0/6] Intel Integrated Sensor Hub Support (ISH)
Date: Thu, 07 Jul 2016 20:02:22 +0200 [thread overview]
Message-ID: <1467914542.1847.2.camel@aim.com> (raw)
In-Reply-To: <1467831078-2530-1-git-send-email-srinivas.pandruvada@linux.intel.com>
Dear Srinivas,
Seems like patch number 2 (of 6) is missing from this patchset.
Cheers!
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
Thu 7 Jul 19:59:44 CEST 2016
Sent from openSUSE Tumbleweed (20160626) (x86_64) on my laptop.
next prev parent reply other threads:[~2016-07-07 18:04 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-06 18:51 [PATCH v3 0/6] Intel Integrated Sensor Hub Support (ISH) Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 1/6] Documentation: hid: Intel ISH HID document Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 2/6] hid: intel_ish-hid: ISH Transport layer Srinivas Pandruvada
2016-07-13 20:39 ` Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 3/6] hid: intel-ish-hid: ipc layer Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 4/6] hid: intel-ish-hid: ISH HID client driver Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 5/6] iio: hid-sensors: use asynchronous resume Srinivas Pandruvada
2016-07-06 18:51 ` [PATCH v3 6/6] hid: hid-sensor-hub: Add ISH quirk Srinivas Pandruvada
2016-07-07 18:02 ` Atri Bhattacharya [this message]
2016-07-10 14:21 ` [PATCH v3 0/6] Intel Integrated Sensor Hub Support (ISH) Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1467914542.1847.2.camel@aim.com \
--to=badshah400@aim.com \
--cc=linux-iio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).