summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Lechner <[email protected]>2026-08-07 13:59:41 -0500
committerDavid Lechner <[email protected]>2026-08-24 19:55:51 -0500
commit8a5ddf6cd349ef86eb8f3c44381cdccb8a848c66 (patch)
treec9f9363d0c25e81167851cf91f64e5b76c3067fc
parenta2ffe8d63a2b7f2326a5235cddc2ded2ec43953c (diff)
serial: mtk: return error value from clk_get_rate()
Pass the return value from clk_get_rate() instead of changing it to -EINVAL. These return values are either ignored or logged, so it should make debugging easier in the cases where it is logged. Reviewed-by: Julien Stephan <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: David Lechner <[email protected]>
-rw-r--r--drivers/serial/serial_mtk.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/serial/serial_mtk.c b/drivers/serial/serial_mtk.c
index 697f5faec1b..d6a7cf488b9 100644
--- a/drivers/serial/serial_mtk.c
+++ b/drivers/serial/serial_mtk.c
@@ -218,7 +218,7 @@ static int mtk_serial_setbrg(struct udevice *dev, int baudrate)
if (clk_valid(&priv->clk)) {
clk_rate = clk_get_rate(&priv->clk);
if (IS_ERR_VALUE(clk_rate) || clk_rate == 0)
- return -EINVAL;
+ return (int)clk_rate ?: -EINVAL;
} else {
clk_rate = priv->fixed_clk_rate;
}
@@ -290,7 +290,7 @@ static int mtk_serial_of_to_plat(struct udevice *dev)
clk_rate = clk_get_rate(&priv->clk);
if (IS_ERR_VALUE(clk_rate)) {
dev_err(dev, "invalid baud clock\n");
- return -EINVAL;
+ return clk_rate;
}
}