Since the commit in the title, a kernel built on an HP Mini 110 Netbook no
longer has keyboard input. The main difference is that CONFIG_SERIO_I8042 is no
longer defined. When this equal yes, the keyboard works. The faulty commit was
located by bisection and has been verified by reverting the patch.
Further investigation shows that the faulty hunk is
diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
index 3bfe8fa..256b9e9 100644
--- a/drivers/input/serio/Kconfig
+++ b/drivers/input/serio/Kconfig
@@ -22,7 +22,7 @@ config SERIO_I8042
tristate "i8042 PC Keyboard controller" if EMBEDDED || !X86
default y
depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && \
- (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN
+ (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN && !X86_MRST
help
i8042 is the chip over which the standard AT keyboard and PS/2
mouse are connected to the computer. If you use these devices,
Reverting this one hunk alone results in a working system. Obviously, this
Netbook does not have X86_MRST set.
On Wednesday, July 14, 2010 04:58:39 pm Larry Finger wrote:
> Since the commit in the title, a kernel built on an HP Mini 110 Netbook no
> longer has keyboard input. The main difference is that CONFIG_SERIO_I8042
> is no longer defined. When this equal yes, the keyboard works. The faulty
> commit was located by bisection and has been verified by reverting the
> patch.
>
> Further investigation shows that the faulty hunk is
>
> diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
> index 3bfe8fa..256b9e9 100644
> --- a/drivers/input/serio/Kconfig
> +++ b/drivers/input/serio/Kconfig
> @@ -22,7 +22,7 @@ config SERIO_I8042
> tristate "i8042 PC Keyboard controller" if EMBEDDED || !X86
> default y
> depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && \
> - (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN
> + (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN &&
> !X86_MRST help
> i8042 is the chip over which the standard AT keyboard and PS/2
> mouse are connected to the computer. If you use these devices,
>
> Reverting this one hunk alone results in a working system. Obviously, this
> Netbook does not have X86_MRST set.
You meant to say "does have CONFIG_X86_MRST set", right?
Thanks.
--
Dmitry
On 07/14/2010 07:07 PM, Dmitry Torokhov wrote:
> On Wednesday, July 14, 2010 04:58:39 pm Larry Finger wrote:
>> Since the commit in the title, a kernel built on an HP Mini 110 Netbook no
>> longer has keyboard input. The main difference is that CONFIG_SERIO_I8042
>> is no longer defined. When this equal yes, the keyboard works. The faulty
>> commit was located by bisection and has been verified by reverting the
>> patch.
>>
>> Further investigation shows that the faulty hunk is
>>
>> diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
>> index 3bfe8fa..256b9e9 100644
>> --- a/drivers/input/serio/Kconfig
>> +++ b/drivers/input/serio/Kconfig
>> @@ -22,7 +22,7 @@ config SERIO_I8042
>> tristate "i8042 PC Keyboard controller" if EMBEDDED || !X86
>> default y
>> depends on !PARISC&& (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST)&& \
>> - (!SUPERH || SH_CAYMAN)&& !M68K&& !BLACKFIN
>> + (!SUPERH || SH_CYes, AYMAN)&& !M68K&& !BLACKFIN&&
>> !X86_MRST help
>> i8042 is the chip over which the standard AT keyboard and PS/2
>> mouse are connected to the computer. If you use these devices,
>>
>> Reverting this one hunk alone results in a working system. Obviously, this
>> Netbook does not have X86_MRST set.
>
>
> You meant to say "does have CONFIG_X86_MRST set", right?
Yes. I got lost in the negative logic. Since !X86_MRST is false, then X86_MRST
has to be defined. In fact, .config has CONFIG_X86_MRST=y.
Larry
On Wed, Jul 14, 2010 at 07:43:57PM -0500, Larry Finger wrote:
> On 07/14/2010 07:07 PM, Dmitry Torokhov wrote:
> >On Wednesday, July 14, 2010 04:58:39 pm Larry Finger wrote:
> >>Since the commit in the title, a kernel built on an HP Mini 110 Netbook no
> >>longer has keyboard input. The main difference is that CONFIG_SERIO_I8042
> >>is no longer defined. When this equal yes, the keyboard works. The faulty
> >>commit was located by bisection and has been verified by reverting the
> >>patch.
> >>
> >>Further investigation shows that the faulty hunk is
> >>
> >>diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
> >>index 3bfe8fa..256b9e9 100644
> >>--- a/drivers/input/serio/Kconfig
> >>+++ b/drivers/input/serio/Kconfig
> >>@@ -22,7 +22,7 @@ config SERIO_I8042
> >> tristate "i8042 PC Keyboard controller" if EMBEDDED || !X86
> >> default y
> >> depends on !PARISC&& (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST)&& \
> >>- (!SUPERH || SH_CAYMAN)&& !M68K&& !BLACKFIN
> >>+ (!SUPERH || SH_CYes, AYMAN)&& !M68K&& !BLACKFIN&&
> >>!X86_MRST help
> >> i8042 is the chip over which the standard AT keyboard and PS/2
> >> mouse are connected to the computer. If you use these devices,
> >>
> >>Reverting this one hunk alone results in a working system. Obviously, this
> >>Netbook does not have X86_MRST set.
> >
> >
> >You meant to say "does have CONFIG_X86_MRST set", right?
>
> Yes. I got lost in the negative logic. Since !X86_MRST is false,
> then X86_MRST has to be defined. In fact, .config has
> CONFIG_X86_MRST=y.
>
Good. HPA posted a patch series that should fix the problem, in the mean
time just disable Moorestown support - you do not need it anyway.
--
Dmitry