All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dell-wmi: add a WMI event code for display on/off
@ 2016-06-16  3:25 Alex Hung
  2016-06-16  7:37 ` Pali Rohár
  0 siblings, 1 reply; 2+ messages in thread
From: Alex Hung @ 2016-06-16  3:25 UTC (permalink / raw)
  To: mjg59, pali.rohar, dvhart, platform-driver-x86, alex.hung

Some new Dell AIO systems have a button that generates WMI evnet
that is to be used to turn LCD on/off

Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 drivers/platform/x86/dell-wmi.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c
index 15c6f11..298aba1 100644
--- a/drivers/platform/x86/dell-wmi.c
+++ b/drivers/platform/x86/dell-wmi.c
@@ -125,6 +125,9 @@ static const struct key_entry dell_wmi_legacy_keymap[] __initconst = {
 	/* Audio panel key */
 	{ KE_IGNORE, 0xe026, { KEY_RESERVED } },
 
+	/* Monitor On/Off Control key */
+	{ KE_KEY, 0xe027, { KEY_DISPLAYTOGGLE } },
+
 	{ KE_IGNORE, 0xe02e, { KEY_VOLUMEDOWN } },
 	{ KE_IGNORE, 0xe030, { KEY_VOLUMEUP } },
 	{ KE_IGNORE, 0xe033, { KEY_KBDILLUMUP } },
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dell-wmi: add a WMI event code for display on/off
  2016-06-16  3:25 [PATCH] dell-wmi: add a WMI event code for display on/off Alex Hung
@ 2016-06-16  7:37 ` Pali Rohár
  0 siblings, 0 replies; 2+ messages in thread
From: Pali Rohár @ 2016-06-16  7:37 UTC (permalink / raw)
  To: Alex Hung; +Cc: mjg59, dvhart, platform-driver-x86

On Thursday 16 June 2016 11:25:08 Alex Hung wrote:
> Some new Dell AIO systems have a button that generates WMI evnet
> that is to be used to turn LCD on/off
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  drivers/platform/x86/dell-wmi.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c
> index 15c6f11..298aba1 100644
> --- a/drivers/platform/x86/dell-wmi.c
> +++ b/drivers/platform/x86/dell-wmi.c
> @@ -125,6 +125,9 @@ static const struct key_entry dell_wmi_legacy_keymap[] __initconst = {
>  	/* Audio panel key */
>  	{ KE_IGNORE, 0xe026, { KEY_RESERVED } },
>  
> +	/* Monitor On/Off Control key */
> +	{ KE_KEY, 0xe027, { KEY_DISPLAYTOGGLE } },
> +
>  	{ KE_IGNORE, 0xe02e, { KEY_VOLUMEDOWN } },
>  	{ KE_IGNORE, 0xe030, { KEY_VOLUMEUP } },
>  	{ KE_IGNORE, 0xe033, { KEY_KBDILLUMUP } },

Hi! Can you rebase this path on top of my "[PATCH v3 0/4] dell-wmi:
Changes in WMI event code handling" series? As it changes order of keys
in that sparse table and your change should be easy to rebase.

After rebasing needs some whitespace changes, but looks good, add my
Acked-by.

-- 
Pali Rohár
pali.rohar@gmail.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-06-16  7:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-16  3:25 [PATCH] dell-wmi: add a WMI event code for display on/off Alex Hung
2016-06-16  7:37 ` Pali Rohár

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.