linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] wistron-btns: Drop Acer Aspire 3020 & 5020 support
@ 2008-02-08  1:59 Carlos Corbacho
  2008-02-08 10:20 ` Eric Piel
  0 siblings, 1 reply; 2+ messages in thread
From: Carlos Corbacho @ 2008-02-08  1:59 UTC (permalink / raw)
  To: linux-input; +Cc: Carlos Corbacho, Dmitry Torokhov, Eric Piel

With acer-wmi now merged, the last reason to keep support for the Acer
Aspire 3020 & 5020 in wistron-btns is gone.

The keys do not need polling, and are already mapped in userspace by HAL.

The other functionality (mail LED, bluetooth and wireless) is already
available in acer-wmi, and surpassed (we can read the status of all these
devices, we can export the backlight, and acer-wmi can also work on
x86-64 with the Aspire 5020, which wistron-btns cannot do).

Signed-off-by: Carlos Corbacho <carlos@strangeworlds.co.uk>
CC: Dmitry Torokhov <dmitry.torokhov@gmail.com>
CC: Eric Piel <eric.piel@tremplin-utc.net>
---

 drivers/input/misc/wistron_btns.c |   33 ---------------------------------
 1 files changed, 0 insertions(+), 33 deletions(-)


diff --git a/drivers/input/misc/wistron_btns.c b/drivers/input/misc/wistron_btns.c
index 72176f3..eaf0024 100644
--- a/drivers/input/misc/wistron_btns.c
+++ b/drivers/input/misc/wistron_btns.c
@@ -348,21 +348,6 @@ static struct key_entry keymap_acer_aspire_1600[] __initdata = {
 	{ KE_END, FE_MAIL_LED | FE_UNTESTED }
 };
 
-/* 3020 has been tested */
-static struct key_entry keymap_acer_aspire_5020[] __initdata = {
-	{ KE_KEY, 0x01, {KEY_HELP} },
-	{ KE_KEY, 0x03, {KEY_POWER} },
-	{ KE_KEY, 0x05, {KEY_SWITCHVIDEOMODE} }, /* Display selection */
-	{ KE_KEY, 0x11, {KEY_PROG1} },
-	{ KE_KEY, 0x12, {KEY_PROG2} },
-	{ KE_KEY, 0x31, {KEY_MAIL} },
-	{ KE_KEY, 0x36, {KEY_WWW} },
-	{ KE_KEY, 0x6a, {KEY_CONFIG} },
-	{ KE_WIFI, 0x30 },
-	{ KE_BLUETOOTH, 0x44 },
-	{ KE_END, FE_MAIL_LED | FE_UNTESTED }
-};
-
 static struct key_entry keymap_acer_travelmate_2410[] __initdata = {
 	{ KE_KEY, 0x01, {KEY_HELP} },
 	{ KE_KEY, 0x6d, {KEY_POWER} },
@@ -654,24 +639,6 @@ static struct dmi_system_id dmi_ids[] __initdata = {
 	},
 	{
 		.callback = dmi_matched,
-		.ident = "Acer Aspire 3020",
-		.matches = {
-			DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
-			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 3020"),
-		},
-		.driver_data = keymap_acer_aspire_5020
-	},
-	{
-		.callback = dmi_matched,
-		.ident = "Acer Aspire 5020",
-		.matches = {
-			DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
-			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5020"),
-		},
-		.driver_data = keymap_acer_aspire_5020
-	},
-	{
-		.callback = dmi_matched,
 		.ident = "Acer TravelMate 2100",
 		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Acer"),


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

* Re: [PATCH] wistron-btns: Drop Acer Aspire 3020 & 5020 support
  2008-02-08  1:59 [PATCH] wistron-btns: Drop Acer Aspire 3020 & 5020 support Carlos Corbacho
@ 2008-02-08 10:20 ` Eric Piel
  0 siblings, 0 replies; 2+ messages in thread
From: Eric Piel @ 2008-02-08 10:20 UTC (permalink / raw)
  To: Carlos Corbacho; +Cc: linux-input, Dmitry Torokhov

Carlos Corbacho wrote:
> With acer-wmi now merged, the last reason to keep support for the Acer
> Aspire 3020 & 5020 in wistron-btns is gone.
> 
> The keys do not need polling, and are already mapped in userspace by HAL.
> 
> The other functionality (mail LED, bluetooth and wireless) is already
> available in acer-wmi, and surpassed (we can read the status of all these
> devices, we can export the backlight, and acer-wmi can also work on
> x86-64 with the Aspire 5020, which wistron-btns cannot do).
> 
> Signed-off-by: Carlos Corbacho <carlos@strangeworlds.co.uk>
> CC: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> CC: Eric Piel <eric.piel@tremplin-utc.net>
It's great that acer-wmi finally arrived in the kernel :-)

Acked-by: Eric Piel <eric.piel@tremplin-utc.net>

Eric

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

end of thread, other threads:[~2008-02-08 10:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-02-08  1:59 [PATCH] wistron-btns: Drop Acer Aspire 3020 & 5020 support Carlos Corbacho
2008-02-08 10:20 ` Eric Piel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).