From: Colin Ian King Date: Thu, 15 Feb 2024 23:29:44 +0000 (+0000) Subject: serial: lpc32xx_hs: remove redundant check and assignment of hsu_rate X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=1364ea8a561356760c86a38c76ddb553bb17c646;p=linux.git serial: lpc32xx_hs: remove redundant check and assignment of hsu_rate Variable hsu_rate is being checked for an upper limit and is assigned a value that is never read. The if statement and assignment are redundant and can be removed. Cleans up clang scan build warning: drivers/tty/serial/lpc32xx_hs.c:237:3: warning: Value stored to 'hsu_rate' is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King Link: https://lore.kernel.org/r/20240215232944.2075789-1-colin.i.king@gmail.com Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/tty/serial/lpc32xx_hs.c b/drivers/tty/serial/lpc32xx_hs.c index ec20329f06036..269efc5e2d51f 100644 --- a/drivers/tty/serial/lpc32xx_hs.c +++ b/drivers/tty/serial/lpc32xx_hs.c @@ -233,8 +233,6 @@ static unsigned int __serial_get_clock_div(unsigned long uartclk, hsu_rate++; } - if (hsu_rate > 0xFF) - hsu_rate = 0xFF; return goodrate; }