[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150306182102.GG19001@vmdeb7>
Date: Fri, 6 Mar 2015 10:21:02 -0800
From: Darren Hart <dvhart@...radead.org>
To: Azael Avalos <coproscefalo@...il.com>
Cc: platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] toshiba_acpi: Add Hotkey Event Type function and
definitions
On Thu, Feb 26, 2015 at 10:57:12AM -0700, Azael Avalos wrote:
Hi Azael,
I'm pretty behind on this one, apologies.
> This patch adds support to query the "Hotkey Event Type" the system
> supports.
>
> There are two main event types (so far), 0x10 and 0x11, with the
> first beign all those laptops that have the old keyboard layout, and
being
> the latter all those new laptops with the new keyboard layout.
>
Some concern about this binary "new" and "old". What happens in 2 years when
they decide to break^W change our keyboards again?
> Signed-off-by: Azael Avalos <coproscefalo@...il.com>
> ---
> drivers/platform/x86/toshiba_acpi.c | 26 ++++++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
> index dbcb7a8..e6aa8f9 100644
> --- a/drivers/platform/x86/toshiba_acpi.c
> +++ b/drivers/platform/x86/toshiba_acpi.c
> @@ -116,6 +116,7 @@ MODULE_LICENSE("GPL");
> #define HCI_KBD_ILLUMINATION 0x0095
> #define HCI_ECO_MODE 0x0097
> #define HCI_ACCELEROMETER2 0x00a6
> +#define HCI_HOTKEY_EVENT_TYPE 0xc000
> #define SCI_PANEL_POWER_ON 0x010d
> #define SCI_ILLUMINATION 0x014e
> #define SCI_USB_SLEEP_CHARGE 0x0150
> @@ -127,8 +128,11 @@ MODULE_LICENSE("GPL");
>
> /* field definitions */
> #define HCI_ACCEL_MASK 0x7fff
> +#define HCI_HOTKEY_EVENT_NORMAL 0x10
> +#define HCI_HOTKEY_EVENT_SPECIAL 0x11
NORMAL and SPECIAL? Hrm... what comes after special? EXTRASPECIAL? ;-)
Or am I not looking at this right? Is there reason to expect these two to be
sufficient?
> #define HCI_HOTKEY_DISABLE 0x0b
> #define HCI_HOTKEY_ENABLE 0x09
> +#define HCI_HOTKEY_ENABLE_SPECIAL 0x10
> #define HCI_LCD_BRIGHTNESS_BITS 3
> #define HCI_LCD_BRIGHTNESS_SHIFT (16-HCI_LCD_BRIGHTNESS_BITS)
> #define HCI_LCD_BRIGHTNESS_LEVELS (1 << HCI_LCD_BRIGHTNESS_BITS)
> @@ -1149,6 +1153,28 @@ static int toshiba_usb_three_set(struct toshiba_acpi_dev *dev, u32 state)
> return 0;
> }
>
> +/* Hotkey event type */
> +static int toshiba_hotkey_event_type_get(struct toshiba_acpi_dev *dev,
> + u32 *hotkey_event_type)
> +{
> + u32 val1 = 0x03;
> + u32 val2 = 0;
> + u32 result;
> +
> + result = hci_read2(dev, HCI_HOTKEY_EVENT_TYPE, &val1, &val2);
> + if (result == TOS_FAILURE) {
> + pr_err("ACPI callto get Hotkey Event type failed\n");
call to
Thanks,
--
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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists