2010-02-20 11:02:32

by Alan Jenkins

Subject: [PATCH] eeepc-laptop: disable wireless hotplug for 1005PE

The wireless hotplug code is not needed on this model, and it disables
the wired ethernet card. (Like on the 1005HA and 1201N).

References: <http://lists.alioth.debian.org/pipermail/debian-eeepc-devel/2010-February/003281.html>

Signed-off-by: Alan Jenkins <[email protected]>
Reported-by: Ansgar Burchardt <[email protected]>
CC: [email protected]
---
drivers/platform/x86/eeepc-laptop.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c
index e2be6bb..6a47bb7 100644
--- a/drivers/platform/x86/eeepc-laptop.c
+++ b/drivers/platform/x86/eeepc-laptop.c
@@ -1277,7 +1277,8 @@ static void eeepc_dmi_check(struct eeepc_laptop *eeepc)
* hotplug code. In fact, current hotplug code seems to unplug another
* device...
*/
- if (strcmp(model, "1005HA") == 0 || strcmp(model, "1201N") == 0) {
+ if (strcmp(model, "1005HA") == 0 || strcmp(model, "1201N") == 0 ||
+ strcmp(model, "1005PE") == 0) {
eeepc->hotplug_disabled = true;
pr_info("wlan hotplug disabled\n");
}
--
1.6.3.3

2010-02-22 14:09:25

by Corentin Chary

Subject: Re: [PATCH] eeepc-laptop: disable wireless hotplug for 1005PE

On Sat, Feb 20, 2010 at 12:02 PM, Alan Jenkins
<[email protected]> wrote:
> The wireless hotplug code is not needed on this model, and it disables
> the wired ethernet card. ?(Like on the 1005HA and 1201N).
>
> References:
> <http://lists.alioth.debian.org/pipermail/debian-eeepc-devel/2010-February/003281.html>
>
> Signed-off-by: Alan Jenkins <[email protected]>
> Reported-by: Ansgar Burchardt <[email protected]>
> CC: [email protected]
> ---
> drivers/platform/x86/eeepc-laptop.c | ? ?3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/platform/x86/eeepc-laptop.c
> b/drivers/platform/x86/eeepc-laptop.c
> index e2be6bb..6a47bb7 100644
> --- a/drivers/platform/x86/eeepc-laptop.c
> +++ b/drivers/platform/x86/eeepc-laptop.c
> @@ -1277,7 +1277,8 @@ static void eeepc_dmi_check(struct eeepc_laptop
> *eeepc)
> ? ? ? ? * hotplug code. In fact, current hotplug code seems to unplug
> another
> ? ? ? ? * device...
> ? ? ? ? */
> - ? ? ? if (strcmp(model, "1005HA") == 0 || strcmp(model, "1201N") == 0) {
> + ? ? ? if (strcmp(model, "1005HA") == 0 || strcmp(model, "1201N") == 0 ||
> + ? ? ? ? ? strcmp(model, "1005PE") == 0) {
> ? ? ? ? ? ? ? ?eeepc->hotplug_disabled = true;
> ? ? ? ? ? ? ? ?pr_info("wlan hotplug disabled\n");
> ? ? ? ?}
> --
> 1.6.3.3
>

Ack,

Len, is it still possible merge this patch in the next rc ?
Thanks,


--
Corentin Chary
http://xf.iksaif.net