* [PATCH 2/2] tty: source all tty/*/Kconfig files from tty/Kconfig
@ 2020-03-05 23:45 Randy Dunlap
2020-03-06 7:19 ` Jiri Slaby
0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2020-03-05 23:45 UTC (permalink / raw)
To: LKML, Greg Kroah-Hartman, Arnd Bergmann, Jiri Slaby
From: Randy Dunlap <rdunlap@infradead.org>
'source' (include) all of the tty/*/Kconfig files from
drivers/tty/Kconfig instead of from drivers/char/Kconfig.
This consolidates them both in source code and in menu
presentation to the user.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Follows [PATCH] char: group some /dev configs together and un-split tty configs
as [PATCH 2/2], where [PATCH (1/2)] is here:
https://lore.kernel.org/lkml/4e90d9af-c1ec-020f-b66b-a5a02e7fbe59@infradead.org/
drivers/char/Kconfig | 5 -----
drivers/tty/Kconfig | 6 ++++++
2 files changed, 6 insertions(+), 5 deletions(-)
--- linux-next-20200304.orig/drivers/char/Kconfig
+++ linux-next-20200304/drivers/char/Kconfig
@@ -7,9 +7,6 @@ menu "Character devices"
source "drivers/tty/Kconfig"
-source "drivers/tty/serial/Kconfig"
-source "drivers/tty/serdev/Kconfig"
-
config TTY_PRINTK
tristate "TTY driver to output user messages via printk"
depends on EXPERT && TTY
@@ -94,8 +91,6 @@ config PPDEV
If unsure, say N.
-source "drivers/tty/hvc/Kconfig"
-
config VIRTIO_CONSOLE
tristate "Virtio console"
depends on VIRTIO && TTY
--- linux-next-20200304.orig/drivers/tty/Kconfig
+++ linux-next-20200304/drivers/tty/Kconfig
@@ -478,3 +478,9 @@ config LDISC_AUTOLOAD
only set the default value of this functionality.
endif # TTY
+
+source "drivers/tty/serial/Kconfig"
+
+source "drivers/tty/serdev/Kconfig"
+
+source "drivers/tty/hvc/Kconfig"
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 2/2] tty: source all tty/*/Kconfig files from tty/Kconfig
2020-03-05 23:45 [PATCH 2/2] tty: source all tty/*/Kconfig files from tty/Kconfig Randy Dunlap
@ 2020-03-06 7:19 ` Jiri Slaby
2020-03-06 19:08 ` Randy Dunlap
0 siblings, 1 reply; 3+ messages in thread
From: Jiri Slaby @ 2020-03-06 7:19 UTC (permalink / raw)
To: Randy Dunlap, LKML, Greg Kroah-Hartman, Arnd Bergmann
On 06. 03. 20, 0:45, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
>
> 'source' (include) all of the tty/*/Kconfig files from
> drivers/tty/Kconfig instead of from drivers/char/Kconfig.
> This consolidates them both in source code and in menu
> presentation to the user.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Follows [PATCH] char: group some /dev configs together and un-split tty configs
> as [PATCH 2/2], where [PATCH (1/2)] is here:
> https://lore.kernel.org/lkml/4e90d9af-c1ec-020f-b66b-a5a02e7fbe59@infradead.org/
>
>
> drivers/char/Kconfig | 5 -----
> drivers/tty/Kconfig | 6 ++++++
> 2 files changed, 6 insertions(+), 5 deletions(-)
>
> --- linux-next-20200304.orig/drivers/char/Kconfig
> +++ linux-next-20200304/drivers/char/Kconfig
> @@ -7,9 +7,6 @@ menu "Character devices"
>
> source "drivers/tty/Kconfig"
>
> -source "drivers/tty/serial/Kconfig"
> -source "drivers/tty/serdev/Kconfig"
> -
> config TTY_PRINTK
> tristate "TTY driver to output user messages via printk"
> depends on EXPERT && TTY
> @@ -94,8 +91,6 @@ config PPDEV
>
> If unsure, say N.
>
> -source "drivers/tty/hvc/Kconfig"
> -
> config VIRTIO_CONSOLE
> tristate "Virtio console"
> depends on VIRTIO && TTY
> --- linux-next-20200304.orig/drivers/tty/Kconfig
> +++ linux-next-20200304/drivers/tty/Kconfig
> @@ -478,3 +478,9 @@ config LDISC_AUTOLOAD
> only set the default value of this functionality.
>
> endif # TTY
> +
> +source "drivers/tty/serial/Kconfig"
> +
> +source "drivers/tty/serdev/Kconfig"
> +
> +source "drivers/tty/hvc/Kconfig"
Maybe sort them alphabetically? That way, you could move the hvc/Kconfig
and serial/Kconfig inside the if-endif above and remove the whole-file
if-TTYs in the 2.
thanks,
--
js
suse labs
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 2/2] tty: source all tty/*/Kconfig files from tty/Kconfig
2020-03-06 7:19 ` Jiri Slaby
@ 2020-03-06 19:08 ` Randy Dunlap
0 siblings, 0 replies; 3+ messages in thread
From: Randy Dunlap @ 2020-03-06 19:08 UTC (permalink / raw)
To: Jiri Slaby, LKML, Greg Kroah-Hartman, Arnd Bergmann
On 3/5/20 11:19 PM, Jiri Slaby wrote:
> On 06. 03. 20, 0:45, Randy Dunlap wrote:
>> From: Randy Dunlap <rdunlap@infradead.org>
>>
>> 'source' (include) all of the tty/*/Kconfig files from
>> drivers/tty/Kconfig instead of from drivers/char/Kconfig.
>> This consolidates them both in source code and in menu
>> presentation to the user.
>>
>> Suggested-by: Arnd Bergmann <arnd@arndb.de>
>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>> ---
>> Follows [PATCH] char: group some /dev configs together and un-split tty configs
>> as [PATCH 2/2], where [PATCH (1/2)] is here:
>> https://lore.kernel.org/lkml/4e90d9af-c1ec-020f-b66b-a5a02e7fbe59@infradead.org/
>>
>>
>> drivers/char/Kconfig | 5 -----
>> drivers/tty/Kconfig | 6 ++++++
>> 2 files changed, 6 insertions(+), 5 deletions(-)
>>
>> --- linux-next-20200304.orig/drivers/char/Kconfig
>> +++ linux-next-20200304/drivers/char/Kconfig
>> @@ -7,9 +7,6 @@ menu "Character devices"
>>
>> source "drivers/tty/Kconfig"
>>
>> -source "drivers/tty/serial/Kconfig"
>> -source "drivers/tty/serdev/Kconfig"
>> -
>> config TTY_PRINTK
>> tristate "TTY driver to output user messages via printk"
>> depends on EXPERT && TTY
>> @@ -94,8 +91,6 @@ config PPDEV
>>
>> If unsure, say N.
>>
>> -source "drivers/tty/hvc/Kconfig"
>> -
>> config VIRTIO_CONSOLE
>> tristate "Virtio console"
>> depends on VIRTIO && TTY
>> --- linux-next-20200304.orig/drivers/tty/Kconfig
>> +++ linux-next-20200304/drivers/tty/Kconfig
>> @@ -478,3 +478,9 @@ config LDISC_AUTOLOAD
>> only set the default value of this functionality.
>>
>> endif # TTY
>> +
>> +source "drivers/tty/serial/Kconfig"
>> +
>> +source "drivers/tty/serdev/Kconfig"
>> +
>> +source "drivers/tty/hvc/Kconfig"
>
> Maybe sort them alphabetically? That way, you could move the hvc/Kconfig
> and serial/Kconfig inside the if-endif above and remove the whole-file
> if-TTYs in the 2.
Hi Jiri,
OK, I'll send a v2 with those changes.
thanks.
--
~Randy
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-06 19:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-05 23:45 [PATCH 2/2] tty: source all tty/*/Kconfig files from tty/Kconfig Randy Dunlap
2020-03-06 7:19 ` Jiri Slaby
2020-03-06 19:08 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).