linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] tty: serial: remove obsolete config SERIAL_SAMSUNG_UARTS_4
@ 2023-03-17  7:15 Lukas Bulwahn
  2023-03-17  7:56 ` Arnd Bergmann
  0 siblings, 1 reply; 2+ messages in thread
From: Lukas Bulwahn @ 2023-03-17  7:15 UTC (permalink / raw)
  To: Arnd Bergmann, Greg Kroah-Hartman, Jiri Slaby, linux-serial
  Cc: kernel-janitors, linux-kernel, Lukas Bulwahn

Commit 1ea35b355722 ("ARM: s3c: remove s3c24xx specific hacks"), allows
simplifying the whole config logic for SERIAL_SAMSUNG_UARTS, and did this
config simplification. However, it misses that SERIAL_SAMSUNG_UARTS_4's
effect was just to control SERIAL_SAMSUNG_UARTS, and with the commit's
change, the config SERIAL_SAMSUNG_UARTS_4 has no remaining effect and can
be deleted.

Remove this obsolete config SERIAL_SAMSUNG_UARTS_4.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
Arnd, please ack.

Greg, please pick this minor clean-up patch.

 drivers/tty/serial/Kconfig | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index 0072892ca7fc..39a0078f54f6 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -248,13 +248,6 @@ config SERIAL_SAMSUNG
 
 	  Choose Y/M here only if you build for such SoC.
 
-config SERIAL_SAMSUNG_UARTS_4
-	bool
-	depends on SERIAL_SAMSUNG
-	default y
-	help
-	  Internal node for the common case of 4 Samsung compatible UARTs
-
 config SERIAL_SAMSUNG_UARTS
 	int
 	depends on SERIAL_SAMSUNG
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] tty: serial: remove obsolete config SERIAL_SAMSUNG_UARTS_4
  2023-03-17  7:15 [PATCH] tty: serial: remove obsolete config SERIAL_SAMSUNG_UARTS_4 Lukas Bulwahn
@ 2023-03-17  7:56 ` Arnd Bergmann
  0 siblings, 0 replies; 2+ messages in thread
From: Arnd Bergmann @ 2023-03-17  7:56 UTC (permalink / raw)
  To: Lukas Bulwahn, Greg Kroah-Hartman, Jiri Slaby, linux-serial
  Cc: kernel-janitors, linux-kernel

On Fri, Mar 17, 2023, at 08:15, Lukas Bulwahn wrote:
> Commit 1ea35b355722 ("ARM: s3c: remove s3c24xx specific hacks"), allows
> simplifying the whole config logic for SERIAL_SAMSUNG_UARTS, and did this
> config simplification. However, it misses that SERIAL_SAMSUNG_UARTS_4's
> effect was just to control SERIAL_SAMSUNG_UARTS, and with the commit's
> change, the config SERIAL_SAMSUNG_UARTS_4 has no remaining effect and can
> be deleted.
>
> Remove this obsolete config SERIAL_SAMSUNG_UARTS_4.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>

Nice find!

I tried to carefully check that all Kconfig options that were
no longer selectable got removed recursively, along with the code
that they control, but I missed this one as I did not search
the other way.

Acked-by: Arnd Bergmann <arnd@arndb.de>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-03-17  7:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-17  7:15 [PATCH] tty: serial: remove obsolete config SERIAL_SAMSUNG_UARTS_4 Lukas Bulwahn
2023-03-17  7:56 ` Arnd Bergmann

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).