All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ritesh Raj Sarraf <rrs@debian.org>
To: Jiri Kosina <jikos@kernel.org>
Cc: linux-kernel@vger.kernel.org,
	Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Subject: Re: [PATCH] Add quirk for Lenovo Yoga 2 with ITE Chips
Date: Mon, 05 Oct 2015 17:12:03 +0530	[thread overview]
Message-ID: <1444045323.24055.23.camel@debian.org> (raw)
In-Reply-To: <alpine.LNX.2.00.1510051327050.2758@pobox.suse.cz>

[-- Attachment #1: Type: text/plain, Size: 1807 bytes --]

On Mon, 2015-10-05 at 13:28 +0200, Jiri Kosina wrote:
> On Mon, 5 Oct 2015, Ritesh Raj Sarraf wrote:
> 
> Please insert a little bit more verbose changelog here (what exactly
> this 
> device is, why does it need such quirk, etc).
> 
> > Signed-off-by: Ritesh Raj Sarraf <rrs@debian.org>
> 
> Adding Srinivas to CC as well.

Dropping stable

This patch is a follow-up to the one below. The USB ID is different for
the Lenovo Yoga 2 13. What Guilhem sent you was verified against Yoga 2
11 and Yoga 3 14. I guess this product line comes with a couple of
variants.

Without the quirk, no data is received from the accelerometer. I have
verified the patch, testing this on 4.3-rc4 (and 4.2 stable). With this
patch, proper orientation data is received.

rrs@learner:~/Community/UpstreamSources/linux-upstream_GIT (stable-42)$ 
monitor-sensor 
** Message: Accelerometer orientation changed: bottom-up
** Message: Light changed: 0.000000 (lux)
±** Message: Accelerometer orientation changed: left-up
** Message: Accelerometer orientation changed: bottom-up
** Message: Accelerometer orientation changed: left-up
** Message: Accelerometer orientation changed: normal
** Message: Light changed: 29.999999 (lux)


monitor-sensor can be found in the iio-sensor-proxy tool.


commit 47eeca8a4838714e582c2a3058458408a328ed60
Author: Guilhem Lettron <guilhem@lettron.fr>
Date:   Sat Jun 27 17:02:23 2015 +0200

    HID: sensor-hub: Add in quirk for Lenovo Yogas with ITE
    
    Like yogas with TEXAS_INSTRUMENTS, yogas with ITE chips needs
    to be initialized with enumeration quirks.
    
    Signed-off-by: Jiri Kosina <jkosina@suse.com>


-- 
Ritesh Raj Sarraf | http://people.debian.org/~rrs
Debian - The Universal Operating System

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  reply	other threads:[~2015-10-05 11:42 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-05  6:37 [PATCH] Add quirk for Lenovo Yoga 2 with ITE Chips Ritesh Raj Sarraf
2015-10-05  9:51 ` Greg KH
2015-10-05 11:28 ` Jiri Kosina
2015-10-05 11:42   ` Ritesh Raj Sarraf [this message]
2015-10-05 16:12     ` Srinivas Pandruvada
  -- strict thread matches above, loose matches on Subject: below --
2015-10-05 16:32 Ritesh Raj Sarraf
2015-10-05 16:42 ` Srinivas Pandruvada
2015-10-05 18:40 ` Jiri Kosina

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=1444045323.24055.23.camel@debian.org \
    --to=rrs@debian.org \
    --cc=jikos@kernel.org \
    --cc=linux-kernel@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.