All of lore.kernel.org
 help / color / mirror / Atom feed
From: Atri Bhattacharya <badshah400@aim.com>
To: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
	"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>
Subject: Re: [RFC 0/4] Intel Integrated Sensor Hub Support (ISH)
Date: Fri, 03 Jun 2016 00:57:19 +0200	[thread overview]
Message-ID: <1464908239.2722.7.camel@aim.com> (raw)
In-Reply-To: <1464907711.3970.70.camel@linux.intel.com>

On Thu, 2016-06-02 at 15:48 -0700, Srinivas Pandruvada wrote:
> On Fri, 2016-06-03 at 00:29 +0200, Atri Bhattacharya wrote:
> > Hi Srinivas,
> > 
> > On Thu, 2016-06-02 at 21:44 +0000, Pandruvada, Srinivas wrote:
> > > 
> > > On Thu, 2016-06-02 at 13:00 -0700, Srinivas Pandruvada wrote:
> > > > 
> > > > Hi,
> > > > 
> > > > Thanks for your help in evaluation.
> > > > 
> > > > 
> > > > [...]
> > > > 
> > > > The iio sensor proxy starts too soon. So I recommended to wait
> > > > for
> > > > mult-user target before start. I think developer is
> > > > considering.
> > > > 
> > > Add after "Description" in the section [Unit] in systemd unit
> > > file
> > > /usr/lib/systemd/system/iio-sensor-proxy.service
> > > 
> > > After=multi-user.target
> > > 
> > > Your rotation should work on Yoga 260. I tried on Fedora 23.
> > > 
> > It does! Thank you so much for this tip. Well, it doesn't work
> > again
> > after suspend/wakeup but I will let you know if that issue gets
> > fixed
> > after the patched kernel finishes building.
> The problem is iio_proxy keeps open the IIO file handles, but ISH FW
> is
> will powerup again after suspend, which causes issue. So may be by
> systemd magic stop the service and start again on resume.
> 
> I am working on this. So probably in RFC v2, I will have some fix.
> 

Great. Thank you very much. Just wanted to update you on what is
happening now after I applied your last patch to the kernel.
Suspend/wakeup is still very slow and journalctl still has messages
like these upon wakeup:

Jun 03 00:46:01 tp-yoga260 kernel: [drm] RC6 on
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0001: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0001: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0001: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0001: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0002: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0002: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0002: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: hid-sensor-hub 0044:8086:22D8.0002: timeout waiting for response from ISHTP device
Jun 03 00:46:01 tp-yoga260 kernel: PM: resume of devices complete after 24416.062 msecs
Jun 03 00:46:01 tp-yoga260 kernel: PM: Finishing wakeup.


But, the values even after wakeup
~> cat /sys/bus/iio/devices/iio\:device3/in_accel_?_raw
53028
-785546
-800683

now make sense (previously they were all 0 and taking a long time to
respond). Despite this, iio-sensor-proxy doesn't work after waking up
and I see messages like this on the journal when rotating my laptop:

Jun 03 00:48:38 tp-yoga260 kernel: thinkpad_acpi: unknown possible thermal alarm or keyboard event received
Jun 03 00:48:38 tp-yoga260 kernel: thinkpad_acpi: unhandled HKEY event 0x60f0
Jun 03 00:48:38 tp-yoga260 kernel: thinkpad_acpi: please report the conditions when this event happened to ibm-acpi-devel@lists.sourceforge.net
Jun 03 00:48:40 tp-yoga260 kernel: SFW2-INext-DROP-DEFLT IN=wlp4s0 OUT= MAC=33:33:00:00:00:fb:a8:bb:cf:1d:c3:da:86:dd SRC=fe80:0000:0000:0000:aabb:cfff:fe1d:c3da DST=ff02:0000:0000:0000:0000:0000:0000:00fb LEN=208 TC=0 HOPLIMIT=255 FLOWLBL=311501 PROTO=UDP SPT=5353 DPT=5353 LEN=168 

Hopefully, there is come clue there. Rotation, brightness auto
adjustment works just fine now after booting and before a first
suspend/wakeup cycle.

Thanks a lot for your follow-up. Best wishes.
-- 
Atri Bhattacharya
Fri  3 Jun 00:49:42 CEST 2016

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

  reply	other threads:[~2016-06-02 22:57 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-31  4:27 [RFC 0/4] Intel Integrated Sensor Hub Support (ISH) Srinivas Pandruvada
     [not found] ` <1464668878-17113-1-git-send-email-srinivas.pandruvada-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-05-31  4:27   ` [RFC 1/4] Documentation: hid: Intel ISH HID document Srinivas Pandruvada
2016-05-31  4:27     ` Srinivas Pandruvada
2016-05-31  4:27 ` [RFC 2/4] hid: intel_ish-hid: ISH Transport layer Srinivas Pandruvada
2016-05-31  4:27 ` [RFC 3/4] hid: intel-ish-hid: ipc layer Srinivas Pandruvada
2016-05-31  4:27 ` [RFC 4/4] hid: intel-ish-hid: ISH HID client driver Srinivas Pandruvada
2016-06-01 13:36 ` [RFC 0/4] Intel Integrated Sensor Hub Support (ISH) Atri Bhattacharya
2016-06-01 13:36   ` Atri Bhattacharya
2016-06-01 13:36   ` Atri Bhattacharya
2016-06-02  3:27   ` Srinivas Pandruvada
2016-06-02  7:48     ` Atri Bhattacharya
2016-06-02 16:49       ` Srinivas Pandruvada
2016-06-02 19:42         ` Atri Bhattacharya
2016-06-02 20:00           ` Srinivas Pandruvada
2016-06-02 21:44             ` Pandruvada, Srinivas
2016-06-02 22:29               ` Atri Bhattacharya
2016-06-02 22:48                 ` Srinivas Pandruvada
2016-06-02 22:57                   ` Atri Bhattacharya [this message]
2016-06-02 23:13                     ` Atri Bhattacharya
2016-06-03  2:50                   ` Atri Bhattacharya
2016-06-03  4:01                     ` Srinivas Pandruvada
2016-06-03  3:08             ` Atri Bhattacharya
2016-06-03  4:02               ` Srinivas Pandruvada
2016-06-06  6:47               ` Srinivas Pandruvada
2016-06-09 21:45 ` Grant Likely
2016-06-09 21:54   ` Srinivas Pandruvada
2016-06-09 21:54     ` Srinivas Pandruvada
     [not found]     ` <1465509287.24319.3.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-06-10  9:44       ` Grant Likely
2016-06-10  9:44         ` Grant Likely
2016-06-10 13:55         ` Srinivas Pandruvada
2016-06-10 14:26           ` Bastien Nocera
     [not found]             ` <1465568791.7409.38.camel-0MeiytkfxGOsTnJN9+BGXg@public.gmane.org>
2016-06-10 14:45               ` Srinivas Pandruvada
2016-06-10 14:45                 ` Srinivas Pandruvada
2016-06-10 15:04                 ` Bastien Nocera
2016-06-10 15:04                   ` Bastien Nocera
     [not found]                   ` <1465571076.7409.39.camel-0MeiytkfxGOsTnJN9+BGXg@public.gmane.org>
2016-06-10 15:23                     ` Srinivas Pandruvada
2016-06-10 15:23                       ` Srinivas Pandruvada
     [not found]                       ` <1465572227.24319.7.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-06-10 15:27                         ` Bastien Nocera
2016-06-10 15:27                           ` Bastien Nocera
     [not found]                           ` <1465572452.7409.41.camel-0MeiytkfxGOsTnJN9+BGXg@public.gmane.org>
2016-06-14 13:44                             ` Grant Likely
2016-06-14 13:44                               ` Grant Likely
2016-06-14 15:12                               ` Bastien Nocera
2016-06-14 16:08                                 ` Grant Likely
2016-06-14 17:01                                   ` Bastien Nocera
2016-06-14 21:33                                     ` Grant Likely
     [not found]         ` <CACxGe6vq-FDpjG8WtrraveQo+ZT3MHMDFzrA5-qqNLEwcbmNhw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-06-10 14:02           ` Grant Likely
2016-06-10 14:02             ` Grant Likely
  -- strict thread matches above, loose matches on Subject: below --
2016-06-02 14:35 me

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=1464908239.2722.7.camel@aim.com \
    --to=badshah400@aim.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=srinivas.pandruvada@linux.intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.