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

[Greg KH <gregkh@linuxfoundation.org>]

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@quicinc.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