linux-kernel - RE: linux-next: build failure after merge of the tty tree

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

Message-ID:
 <DB9PR04MB8429CD5C790E85AEBD7A7DE292A42@DB9PR04MB8429.eurprd04.prod.outlook.com>
Date: Mon, 24 Mar 2025 09:10:01 +0000
From: Sherry Sun <sherry.sun@....com>
To: Stephen Rothwell <sfr@...b.auug.org.au>, Greg KH <greg@...ah.com>
CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Linux Kernel Mailing List
	<linux-kernel@...r.kernel.org>, Linux Next Mailing List
	<linux-next@...r.kernel.org>
Subject: RE: linux-next: build failure after merge of the tty tree



> -----Original Message-----
> From: Stephen Rothwell <sfr@...b.auug.org.au>
> Sent: Monday, March 24, 2025 4:37 PM
> To: Greg KH <greg@...ah.com>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>; Sherry Sun
> <sherry.sun@....com>; Linux Kernel Mailing List <linux-
> kernel@...r.kernel.org>; Linux Next Mailing List <linux-next@...r.kernel.org>
> Subject: linux-next: build failure after merge of the tty tree
> 
> Hi all,
> 
> After merging the tty tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
> 
> drivers/tty/serial/fsl_lpuart.c: In function 'lpuart_poll_init':
> drivers/tty/serial/fsl_lpuart.c:642:29: error: unused variable 'sport' [-
> Werror=unused-variable]
>   642 |         struct lpuart_port *sport = container_of(port,
>       |                             ^~~~~
> drivers/tty/serial/fsl_lpuart.c: In function 'lpuart32_poll_init':
> drivers/tty/serial/fsl_lpuart.c:696:29: error: unused variable 'sport' [-
> Werror=unused-variable]
>   696 |         struct lpuart_port *sport = container_of(port, struct lpuart_port,
> port);
>       |                             ^~~~~
> cc1: all warnings being treated as errors
> 

Hi Stephen,

Thanks for the quick fix, actually I sent the fix patch earlier today, please check
https://lore.kernel.org/imx/20250324021051.162676-1-sherry.sun@nxp.com/T/

Best Regards
Sherry

> Caused by commit
> 
>   3cc16ae096f1 ("tty: serial: fsl_lpuart: use port struct directly to simply code")
> 
> I have applied the following patch for today:
> 
> From: Stephen Rothwell <sfr@...b.auug.org.au>
> Date: Mon, 24 Mar 2025 19:10:57 +1100
> Subject: [PATCH] fix up for "tty: serial: fsl_lpuart: use port struct directly  to
> simply code"
> 
> Signed-off-by: Stephen Rothwell <sfr@...b.auug.org.au>
> ---
>  drivers/tty/serial/fsl_lpuart.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpuart.c index
> 33eeefa6fa8f..4470966b826c 100644
> --- a/drivers/tty/serial/fsl_lpuart.c
> +++ b/drivers/tty/serial/fsl_lpuart.c
> @@ -639,8 +639,6 @@ static void lpuart32_wait_bit_set(struct uart_port
> *port, unsigned int offset,
> 
>  static int lpuart_poll_init(struct uart_port *port)  {
> -	struct lpuart_port *sport = container_of(port,
> -					struct lpuart_port, port);
>  	unsigned long flags;
>  	u8 fifo;
> 
> @@ -693,7 +691,6 @@ static int lpuart_poll_get_char(struct uart_port *port)
> static int lpuart32_poll_init(struct uart_port *port)  {
>  	unsigned long flags;
> -	struct lpuart_port *sport = container_of(port, struct lpuart_port,
> port);
>  	u32 fifo;
> 
>  	port->fifosize = 0;
> --
> 2.45.2
> 
> --
> Cheers,
> Stephen Rothwell