Re: [PATCH] tty: serial: fsl_lpuart: increase maximum uart_nr to 12

From: Greg KH
Date: Fri Jan 03 2025 - 04:08:38 EST


On Fri, Jan 03, 2025 at 03:11:54PM +0800, Sherry Sun wrote:
> Some SoCs like the i.MX943 have aliases for up to 12 UARTs, need to
> increase UART_NR from 8 to 12 to support lpuart9-12 to avoid
> initialization failures.
>
> Signed-off-by: Sherry Sun <sherry.sun@xxxxxxx>
> ---
> drivers/tty/serial/fsl_lpuart.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpuart.c
> index 57b0632a3db6..7cb1e36fdaab 100644
> --- a/drivers/tty/serial/fsl_lpuart.c
> +++ b/drivers/tty/serial/fsl_lpuart.c
> @@ -245,7 +245,7 @@
>
> #define DRIVER_NAME "fsl-lpuart"
> #define DEV_NAME "ttyLP"
> -#define UART_NR 8
> +#define UART_NR 12

Why not fix this properly and make this dynamic and get rid of the
static array causing all of this problem? That way when you get a
system with 13 uarts, you will be ok :)

thanks,

greg k-h