linux-kernel - Re: [PATCH 2/2] serial: qcom-geni: Make UART port count configurable via Kconfig

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] [day] [month] [year] [list]

Message-ID: <2025081249-amusement-spongy-e052@gregkh>
Date: Tue, 12 Aug 2025 08:29:56 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Zong Jiang <quic_zongjian@...cinc.com>
Cc: linux-serial@...r.kernel.org, linux-kernel@...r.kernel.org,
	quic_ztu@...cinc.com, quic_anupkulk@...cinc.com,
	quic_msavaliy@...cinc.com, quic_vdadhani@...cinc.com
Subject: Re: [PATCH 2/2] serial: qcom-geni: Make UART port count configurable
 via Kconfig

On Tue, Aug 12, 2025 at 01:48:19PM +0800, Zong Jiang wrote:
> Replace the hardcoded GENI_UART_PORTS macro with a new Kconfig option
> SERIAL_QCOM_GENI_UART_PORTS to allow platforms to configure the maximum
> number of UART ports supported by the driver at build time.
> 
> This improves flexibility for platforms that require more than the
> previously fixed number of UART ports, and avoids unnecessary allocation
> for unused ports.
> 
> Signed-off-by: Zong Jiang <quic_zongjian@...cinc.com>
> ---
>  drivers/tty/serial/Kconfig            | 8 ++++++++
>  drivers/tty/serial/qcom_geni_serial.c | 5 ++---
>  2 files changed, 10 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> index 44427415a80d..e661f5951f55 100644
> --- a/drivers/tty/serial/Kconfig
> +++ b/drivers/tty/serial/Kconfig
> @@ -928,6 +928,14 @@ config SERIAL_QCOM_GENI_CONSOLE
>  	  Serial console driver for Qualcomm Technologies Inc's GENI based
>  	  QUP hardware.
>  
> +config SERIAL_QCOM_GENI_UART_PORTS
> +	int "Maximum number of GENI UART ports"
> +	depends on SERIAL_QCOM_GENI
> +	default "8"
> +	help
> +	  Set this to the maximum number of serial ports you want the driver
> +	  to support.

Why do you need a static number of ports at all?  Why not just handle
how ever many there are in the system at the moment?  Or is this due to
some restriction in the serial core that requires this?

thanks,

greg k-h