SRU - LP #284066 - Input: atkbd - expand Latitude's force release quirk to other Dells

Ben Collins ben.collins at canonical.com
Wed Nov 5 17:57:01 UTC 2008


Tim Gardner wrote:
>>From 8948ecffc8c56009c4580e684d6e385b2bad96df Mon Sep 17 00:00:00 2001
> From: Matthew Garrett <mjg59 at srcf.ucam.org>
> Date: Fri, 15 Aug 2008 13:54:51 -0400
> Subject: [PATCH] Input: atkbd - expand Latitude's force release quirk to other Dells
> 
> Bug: #284066
> 
> Dell laptops fail to send key up events for several of their special
> keys. There's an existing quirk in the kernel to handle this, but it's
> limited to the Latitude range. This patch extends it to cover all
> portable Dells.

ACK

> Signed-off-by: Matthew Garrett <mjg at redhat.com>
> Signed-off-by: Dmitry Torokhov <dtor at mail.ru>
> Signed-off-by: Tim Gardner <tim.gardner at canonical.com>
> ---
>  drivers/input/keyboard/atkbd.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
> index b1ce10f..e32c24d 100644
> --- a/drivers/input/keyboard/atkbd.c
> +++ b/drivers/input/keyboard/atkbd.c
> @@ -834,10 +834,10 @@ static void atkbd_disconnect(struct serio *serio)
>  }
>  
>  /*
> - * Most special keys (Fn+F?) on Dell Latitudes do not generate release
> + * Most special keys (Fn+F?) on Dell laptops do not generate release
>   * events so we have to do it ourselves.
>   */
> -static void atkbd_latitude_keymap_fixup(struct atkbd *atkbd)
> +static void atkbd_dell_laptop_keymap_fixup(struct atkbd *atkbd)
>  {
>  	const unsigned int forced_release_keys[] = {
>  		0x85, 0x86, 0x87, 0x88, 0x89, 0x8a, 0x8b, 0x8f, 0x93,
> @@ -1461,13 +1461,13 @@ static int __init atkbd_setup_fixup(const struct dmi_system_id *id)
>  
>  static struct dmi_system_id atkbd_dmi_quirk_table[] __initdata = {
>  	{
> -		.ident = "Dell Latitude series",
> +		.ident = "Dell Laptop",
>  		.matches = {
>  			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "Latitude"),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "8"), /* Portable */
>  		},
>  		.callback = atkbd_setup_fixup,
> -		.driver_data = atkbd_latitude_keymap_fixup,
> +		.driver_data = atkbd_dell_laptop_keymap_fixup,
>  	},
>  	{
>  		.ident = "HP 2133",





More information about the kernel-team mailing list