2009-11-24 14:14:05

by Anisse Astier

[permalink] [raw]
Subject: [PATCH] Input: i8042 - add Dell Vostro 1520 and 1720 to the reset list

These laptops may have i8042 in a bad state at very first boot or if you plug
in a non-Dell power supply.
Thus this patch is needed for proper keyboard and touchpad function.

Signed-off-by: Anisse Astier <[email protected]>
---
drivers/input/serio/i8042-x86ia64io.h | 14 ++++++++++++++
1 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index a537925..7d1cebc 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -447,6 +447,20 @@ static struct dmi_system_id __initdata i8042_dmi_reset_table[] = {
DMI_MATCH(DMI_PRODUCT_NAME, "N10"),
},
},
+ {
+ .ident = "Dell Vostro 1520",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "Vostro 1520"),
+ },
+ },
+ {
+ .ident = "Dell Vostro 1720",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "Vostro 1720"),
+ },
+ },
{ }
};

--
1.6.5.2


2009-11-25 20:17:39

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] Input: i8042 - add Dell Vostro 1520 and 1720 to the reset list

Hi Anisse,

On Tue, Nov 24, 2009 at 03:12:50PM +0100, Anisse Astier wrote:
> These laptops may have i8042 in a bad state at very first boot or if you plug
> in a non-Dell power supply.
> Thus this patch is needed for proper keyboard and touchpad function.
>

Could youplease explain in more detail what the "bad state" is. And what
exactly do you mean by "first boot"? Cold boot? The very first boot
after assembling? Something else?

Thanks.

--
Dmitry

2009-11-26 10:06:07

by Anisse Astier

[permalink] [raw]
Subject: Re: [PATCH] Input: i8042 - add Dell Vostro 1520 and 1720 to the reset list

Hi Dmitry,

Sorry for the badly threaded reply, I wasn't subscribed to the list and
didn't ask explicitely to be Cc-ed.

>On Wed, 25 Nov 2009 12:17:37 -0800, Dmitry Torokhov wrote:
>Could youplease explain in more detail what the "bad state" is.
>And what exactly do you mean by "first boot"? Cold boot? The very
>first boot after assembling? Something else?

By bad state I mean that neither touchpad nor keyboard are working. They
are both unresponsive. You can fix this by adding i8042.reset on the
kernel command line, or removing the battery and waiting long enough.
By very first boot I mean when the laptops are received from Dell and
booted in Linux.

If you cold boot with a working non-Dell power supply (with same
electrical characteristics), you can reproduce the "first boot" bug as
much as you want, and laptops won't work unless you remove the battery
and wait long enough OR add i8042.reset on the cmdline.

Regards,

--
Anisse

2009-11-30 09:35:43

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] Input: i8042 - add Dell Vostro 1520 and 1720 to the reset list

On Thu, Nov 26, 2009 at 11:05:00AM +0100, Anisse Astier wrote:
> Hi Dmitry,
>
> Sorry for the badly threaded reply, I wasn't subscribed to the list and
> didn't ask explicitely to be Cc-ed.
>
> >On Wed, 25 Nov 2009 12:17:37 -0800, Dmitry Torokhov wrote:
> >Could youplease explain in more detail what the "bad state" is.
> >And what exactly do you mean by "first boot"? Cold boot? The very
> >first boot after assembling? Something else?
>
> By bad state I mean that neither touchpad nor keyboard are working. They
> are both unresponsive. You can fix this by adding i8042.reset on the
> kernel command line, or removing the battery and waiting long enough.
> By very first boot I mean when the laptops are received from Dell and
> booted in Linux.
>
> If you cold boot with a working non-Dell power supply (with same
> electrical characteristics), you can reproduce the "first boot" bug as
> much as you want, and laptops won't work unless you remove the battery
> and wait long enough OR add i8042.reset on the cmdline.
>

Ok, I see... what about 1320? I did a quick round of googling and it
seem to have the similar issue...

--
Dmitry

2009-11-30 09:56:52

by Anisse Astier

[permalink] [raw]
Subject: Re: [PATCH] Input: i8042 - add Dell Vostro 1520 and 1720 to the reset list


On Mon, 30 Nov 2009 01:35:44 -0800, Dmitry Torokhov wrote :
>
> Ok, I see... what about 1320? I did a quick round of googling and it
> seem to have the similar issue...
>

Unfortunately I don't have the hardware, so I can't tell for sure. But
it does indeed look very alike the problem we experienced.

--
Anisse