* [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).