Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1243968 > unrolled thread
| Started by | Darren Hart <dvhart@infradead.org> |
|---|---|
| First post | 2015-10-10 20:50 +0200 |
| Last post | 2015-10-11 14:00 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] thinkpad_acpi: Add TP_HKEY_EV_SCREEN_FLIP (0x60c0) for Lenovo ThinkPad Yoga 12 Darren Hart <dvhart@infradead.org> - 2015-10-10 20:50 +0200
Re: [PATCH] thinkpad_acpi: Add TP_HKEY_EV_SCREEN_FLIP (0x60c0) for Lenovo ThinkPad Yoga 12 Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2015-10-10 23:50 +0200
Re: [PATCH] thinkpad_acpi: Add TP_HKEY_EV_SCREEN_FLIP (0x60c0) for Lenovo ThinkPad Yoga 12 Darren Hart <dvhart@infradead.org> - 2015-10-10 23:50 +0200
Re: [PATCH] thinkpad_acpi: Add TP_HKEY_EV_SCREEN_FLIP (0x60c0) for Lenovo ThinkPad Yoga 12 Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2015-10-11 14:00 +0200
| From | Darren Hart <dvhart@infradead.org> |
|---|---|
| Date | 2015-10-10 20:50 +0200 |
| Subject | Re: [PATCH] thinkpad_acpi: Add TP_HKEY_EV_SCREEN_FLIP (0x60c0) for Lenovo ThinkPad Yoga 12 |
| Message-ID | <qi7ns-Ut-7@gated-at.bofh.it> |
On Sat, Oct 10, 2015 at 11:41:47AM -0700, Darren Hart wrote:
> From: Darren Hart <dvhart@infradead.org>
>
> thinkpad_acpi reports "unhandled HKEY event 0x60c0" when switching
> between laptop and tent/tablet mode on the ThinkPad Yoga 12. This key is
> reported by atkbd serio0 as e058 and e059.
>
> Add TP_HKEY_EV_SCREEN_FLIP to the list of ignored events and silence the
> warning.
>
> Cleanup whitespace on preceeding line for TP_HKEY_EV_KEY_FN_ESC.
>
> Signed-off-by: Darren Hart <dvhart@infradead.org>
> Cc: Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br>
> Cc: ibm-acpi-devel@lists.sourceforge.net
And Cc pdx86... oi. That's embarassing.
> ---
> drivers/platform/x86/thinkpad_acpi.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
> index 131dd74..d68143e 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -197,7 +197,8 @@ enum tpacpi_hkey_event_t {
> /* Key-related user-interface events */
> TP_HKEY_EV_KEY_NUMLOCK = 0x6000, /* NumLock key pressed */
> TP_HKEY_EV_KEY_FN = 0x6005, /* Fn key pressed? E420 */
> - TP_HKEY_EV_KEY_FN_ESC = 0x6060, /* Fn+Esc key pressed X240 */
> + TP_HKEY_EV_KEY_FN_ESC = 0x6060, /* Fn+Esc key pressed X240 */
> + TP_HKEY_EV_SCREEN_FLIP = 0x60c0, /* Screen flipped Yoga 12 */
>
> /* Thermal events */
> TP_HKEY_EV_ALARM_BAT_HOT = 0x6011, /* battery too hot */
> @@ -3849,7 +3850,9 @@ static bool hotkey_notify_6xxx(const u32 hkey,
> * unplugging AC adapter, docking or undocking. */
>
> /* fallthrough */
> -
> + case TP_HKEY_EV_SCREEN_FLIP:
> + /* Yoga 12: Screen toggled between laptop and tablet mode. */
> + /* fallthrough */
> case TP_HKEY_EV_KEY_NUMLOCK:
> case TP_HKEY_EV_KEY_FN:
> case TP_HKEY_EV_KEY_FN_ESC:
> --
> 2.1.4
>
>
--
Darren Hart
Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Henrique de Moraes Holschuh <hmh@hmh.eng.br> |
|---|---|
| Date | 2015-10-10 23:50 +0200 |
| Message-ID | <qiabE-4YE-3@gated-at.bofh.it> |
| In reply to | #1243968 |
On Sat, 10 Oct 2015, Darren Hart wrote: > On Sat, Oct 10, 2015 at 11:41:47AM -0700, Darren Hart wrote: > > From: Darren Hart <dvhart@infradead.org> > > > > thinkpad_acpi reports "unhandled HKEY event 0x60c0" when switching > > between laptop and tent/tablet mode on the ThinkPad Yoga 12. This key is > > reported by atkbd serio0 as e058 and e059. > > > > Add TP_HKEY_EV_SCREEN_FLIP to the list of ignored events and silence the > > warning. > > > > Cleanup whitespace on preceeding line for TP_HKEY_EV_KEY_FN_ESC. > > > > Signed-off-by: Darren Hart <dvhart@infradead.org> > > Cc: Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br> > > Cc: ibm-acpi-devel@lists.sourceforge.net Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br> -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Darren Hart <dvhart@infradead.org> |
|---|---|
| Date | 2015-10-10 23:50 +0200 |
| Message-ID | <qiabE-4YE-15@gated-at.bofh.it> |
| In reply to | #1244020 |
On Sat, Oct 10, 2015 at 06:40:34PM -0300, Henrique de Moraes Holschuh wrote: > On Sat, 10 Oct 2015, Darren Hart wrote: > > On Sat, Oct 10, 2015 at 11:41:47AM -0700, Darren Hart wrote: > > > From: Darren Hart <dvhart@infradead.org> > > > > > > thinkpad_acpi reports "unhandled HKEY event 0x60c0" when switching > > > between laptop and tent/tablet mode on the ThinkPad Yoga 12. This key is > > > reported by atkbd serio0 as e058 and e059. > > > > > > Add TP_HKEY_EV_SCREEN_FLIP to the list of ignored events and silence the > > > warning. > > > > > > Cleanup whitespace on preceeding line for TP_HKEY_EV_KEY_FN_ESC. > > > > > > Signed-off-by: Darren Hart <dvhart@infradead.org> > > > Cc: Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br> > > > Cc: ibm-acpi-devel@lists.sourceforge.net > > Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br> Thanks Henrique, Now, as I'm digging into this I'm finding maybe there is more to this. Should I be attempting to tie this in to the tablet switch mechanism for the older x61t? Or is this something we now want to strictly hand off to a userspace helper via the atkbd event? -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Henrique de Moraes Holschuh <hmh@hmh.eng.br> |
|---|---|
| Date | 2015-10-11 14:00 +0200 |
| Message-ID | <qinsd-7rc-1@gated-at.bofh.it> |
| In reply to | #1244024 |
On Sat, 10 Oct 2015, Darren Hart wrote: > On Sat, Oct 10, 2015 at 06:40:34PM -0300, Henrique de Moraes Holschuh wrote: > > On Sat, 10 Oct 2015, Darren Hart wrote: > > > On Sat, Oct 10, 2015 at 11:41:47AM -0700, Darren Hart wrote: > > > > From: Darren Hart <dvhart@infradead.org> > > > > > > > > thinkpad_acpi reports "unhandled HKEY event 0x60c0" when switching > > > > between laptop and tent/tablet mode on the ThinkPad Yoga 12. This key is > > > > reported by atkbd serio0 as e058 and e059. > > > > > > > > Add TP_HKEY_EV_SCREEN_FLIP to the list of ignored events and silence the > > > > warning. > > > > > > > > Cleanup whitespace on preceeding line for TP_HKEY_EV_KEY_FN_ESC. > > > > > > > > Signed-off-by: Darren Hart <dvhart@infradead.org> > > > > Cc: Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br> > > > > Cc: ibm-acpi-devel@lists.sourceforge.net > > > > Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br> > > Thanks Henrique, > > Now, as I'm digging into this I'm finding maybe there is more to this. Should I > be attempting to tie this in to the tablet switch mechanism for the older x61t? I think so, provided that you can read the current mode from somewhere. And it is likelt to be exported by the EC somehow, since atkbd gets two separate keycodes... > Or is this something we now want to strictly hand off to a userspace helper via > the atkbd event? It depends on the existence of said userspace helper, I suppose. -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web