diff options
| author | Andrew Goodbody <[email protected]> | 2025-08-11 17:25:17 +0100 |
|---|---|---|
| committer | Peng Fan <[email protected]> | 2025-08-27 15:52:20 +0800 |
| commit | 4d3183723f34d58cb5decec77e8bf64b84d309d1 (patch) | |
| tree | 2f2a935d8dca61b3e0c4c7439b0c6c28ba487384 /drivers/serial | |
| parent | dc37adfc040877db359abdc7eab19adffb8fc722 (diff) | |
serial: lpuart: Return value from correct variable
In get_lpuart_clk_rate if the call to clk_get_rate returns an error then
the call to return should pass the value of the error which is in rate
rather than ret which will be 0 as its value is not affected by this
error.
This issue was found by Smatch.
Signed-off-by: Andrew Goodbody <[email protected]>
Signed-off-by: Peng Fan <[email protected]>
Diffstat (limited to 'drivers/serial')
| -rw-r--r-- | drivers/serial/serial_lpuart.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/serial/serial_lpuart.c b/drivers/serial/serial_lpuart.c index 93602797b83..9fdb6503085 100644 --- a/drivers/serial/serial_lpuart.c +++ b/drivers/serial/serial_lpuart.c @@ -129,7 +129,7 @@ static int get_lpuart_clk_rate(struct udevice *dev, u32 *clk_rate) rate = clk_get_rate(&clk); if ((long)rate <= 0) { dev_err(dev, "Failed to get clk rate: %ld\n", (long)rate); - return ret; + return rate; } *clk_rate = rate; return 0; |
