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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 22 Dec 2006 22:42:38 +0100 (CET)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Russell King <rmk+lkml@....linux.org.uk>
Cc:	Linux Kernel List <linux-kernel@...r.kernel.org>,
	linux-input@...ey.karlin.mff.cuni.cz
Subject: Re: [PATCH] Fix some ARM builds due to HID brokenness

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@....linux.org.uk>

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/