Re: [PATCH] Fix some ARM builds due to HID brokenness

From: Jiri Kosina
Date: Fri Dec 22 2006 - 16:45:35 EST


On Fri, 22 Dec 2006, Russell King wrote:

> The new location for HID is extremely annoying:

Hi Russell,

well, the location itself was a subject to various discussions. At the
end, drivers/hid won against drivers/input/hid, because, as Marcel
correctly pointed out, in case of embedded systems there might be a need
to have an abstraction for non-input devices and then we don't want to
bundle it to tightly to input. There is no such implementation yet, but
moving this code around is quite painful, so we don't want to do it more
than once.

> 1. the help text implies that you need to enable it for any
> keyboard or mouse attached to the system. This is not
> correct.

This help text has been there for ages, it was duplicated from previous
help text from usb hid implementation, but you are right that the wording
might confuse someone. Will change it eventually, thanks.

> 2. it defaults to 'y'. When you have input deselected, this
> causes the kernel to fail to link:
> Fix the second problem by making it depend on INPUT.
> Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>

Applied, thanks.

--
Jiri Kosina
SUSE Labs
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/