lkml.org 
[lkml]   [2015]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] thinkpad_acpi: unhandled hkey event
On 2015-02-07 05:22, Darren Hart wrote:
> On Sat, Jan 31, 2015 at 07:52:03PM +0100, Xavier Naveira wrote:
>> Pressing Fn+Esc in a Lenovo Thinkpad x240 to lock the Fn keys generates
>> an unhandled hkey event
>>
>> Signed-off-by: Xavier Naveira <xnaveira@gmail.com>
>> ---
>> drivers/platform/x86/thinkpad_acpi.c | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
>> index c3d11fa..e61c43b 100644
>> --- a/drivers/platform/x86/thinkpad_acpi.c
>> +++ b/drivers/platform/x86/thinkpad_acpi.c
>> @@ -196,6 +196,7 @@ 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 */
>>
>> /* Thermal events */
>> TP_HKEY_EV_ALARM_BAT_HOT = 0x6011, /* battery too hot */
>> @@ -3717,6 +3718,12 @@ static bool hotkey_notify_6xxx(const u32 hkey,
>> *send_acpi_ev = false;
>> *ignore_acpi_ev = true;
>> return true;
>> + case TP_HKEY_EV_KEY_FN_ESC:
>> + /* key press events, we just ignore them as long as the EC
>> + * is still reporting them in the normal keyboard stream */
>> + *send_acpi_ev = false;
>> + *ignore_acpi_ev = true;
>> + return true;
>
> No need to duplicate the logic here, just add TP_HKEY_EV_KEY_FN_ESC to the list
> of fallthrough keys (right after TP_HKEY_EV_KEY_FN).
>
I don't see the list that you are referring to?


\
 
 \ /
  Last update: 2015-02-09 21:01    [W:1.130 / U:0.060 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site