linux-kernel - Re: linux-next: manual merge of the tty tree with the input tree

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Mon, 28 Jan 2013 14:09:31 -0800
From:	Joe Millenbach <jmillenbach@...il.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:	Greg KH <greg@...ah.com>, Stephen Rothwell <sfr@...b.auug.org.au>,
	linux-next@...r.kernel.org, linux-kernel@...r.kernel.org,
	Brian Swetland <swetland@...gle.com>,
	"Mike A. Chan" <mikechan@...gle.com>,
	Sheng Yang <sheng@...ux.intel.com>,
	Yunhong Jiang <yunhong.jiang@...el.com>,
	Xiaohui Xin <xiaohui.xin@...el.com>,
	Jun Nakajima <jun.nakajima@...el.com>,
	Bruce Beare <bruce.j.beare@...el.com>,
	Tom Keel <thomas.keel@...el.com>,
	Alan Cox <alan@...ux.intel.com>,
	Josh Triplett <josh@...htriplett.org>,
	Jamey Sharp <jamey@...ilop.net>
Subject: Re: linux-next: manual merge of the tty tree with the input tree

On Mon, Jan 28, 2013 at 9:33 AM, Dmitry Torokhov
<dmitry.torokhov@...il.com> wrote:
> On Mon, Jan 28, 2013 at 06:46:15AM -0800, Greg KH wrote:
>> On Mon, Jan 28, 2013 at 08:44:24PM +1100, Stephen Rothwell wrote:
>> > Hi Greg,
>> >
>> > Today's linux-next merge of the tty tree got a conflict in
>> > drivers/input/keyboard/Kconfig between commit 6f2ac009f29b ("Input:
>> > goldfish - virtual input event driver") from the input tree and commit
>> > 4f73bc4dd3e8 ("tty: Added a CONFIG_TTY option to allow removal of TTY")
>> > from the tty tree.
>> >
>> > I fixed it up (see below - I am not sure if GOLDFISH_EVENTS needs TTY or
>> > not) and can carry the fix as necessary (no action is required).
>> >
>> > --
>> > Cheers,
>> > Stephen Rothwell                    sfr@...b.auug.org.au
>> >
>> > diff --cc drivers/input/keyboard/Kconfig
>> > index 078305e,008f96a..0000000
>> > --- a/drivers/input/keyboard/Kconfig
>> > +++ b/drivers/input/keyboard/Kconfig
>> > @@@ -479,16 -482,8 +482,18 @@@ config KEYBOARD_SAMSUN
>> >       To compile this driver as a module, choose M here: the
>> >       module will be called samsung-keypad.
>> >
>> > + if TTY
>> > +
>> >  +config KEYBOARD_GOLDFISH_EVENTS
>> >  +  depends on GOLDFISH
>> >  +  tristate "Generic Input Event device for Goldfish"
>> >  +  help
>> >  +    Say Y here to get an input event device for the Goldfish virtual
>>
>> Looks good, thanks.
>
> Greg,
>
> Please drop 4f73bc4dd3e8563ef4109f293a092820dff66d92, at least the parts
> related to input. As far as I know nothing except serport driver
> depends on tty and we do not need to introduce this kind of
> dependencie. Anyone needing slim config can simply try disabling
> input (or parts of it) without needing an artificial dependencies.
>
> Thanks.
>
> --
> Dmitry

Dmitry and Greg,

SERIO needs TTY, and the majority of the input changes are adding "depends on
TTY" to things that "select SERIO" as they break the dependency chain.  In other
words, enabling a component that selects SERIO will turn SERIO on even when
SERIO depends on TTY and TTY is disabled.

Joe
--
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/