> -----Original Message----- > From: Kwanghoon Son <k.son@xxxxxxxxxxx> > Sent: Wednesday, June 5, 2024 9:37 AM > To: krzk@xxxxxxxxxx; alim.akhtar@xxxxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; > jirislaby@xxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; linux- > serial@xxxxxxxxxxxxxxx > Cc: Kwanghoon Son <k.son@xxxxxxxxxxx> > Subject: [PATCH] serial: samsung: Change MAX_CLK_NAME_LENGTH to 17 > > clkname "clk_uart_baud" already 13 byte, so compiler warns > drivers/tty/serial/samsung_tty.c:1392:17: note: ‘sprintf’ output between 15 and > 17 bytes into a destination of size 15 There was a similar discussion around buffer size of clkname last year https://www.spinics.net/lists/arm-kernel/msg1039769.html and it was concluded that it’s a false warning AFAIR. > > Signed-off-by: Kwanghoon Son <k.son@xxxxxxxxxxx> > --- > drivers/tty/serial/samsung_tty.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/tty/serial/samsung_tty.c b/drivers/tty/serial/samsung_tty.c > index dc35eb77d2ef..cad838ac8aa2 100644 > --- a/drivers/tty/serial/samsung_tty.c > +++ b/drivers/tty/serial/samsung_tty.c > @@ -1339,7 +1339,7 @@ static void s3c24xx_serial_pm(struct uart_port *port, > unsigned int level, > * > */ > > -#define MAX_CLK_NAME_LENGTH 15 > +#define MAX_CLK_NAME_LENGTH 17 > > static inline u8 s3c24xx_serial_getsource(struct uart_port *port) { > -- > 2.39.2
