[PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[laixiangwu]

sc16is7xx_probe() enables the clock with clk_prepare_enable() early in
the probe path. If the resulting clock frequency cannot be determined
(neither from the clock itself, nor from the "clock-frequency" property
or platform data), the driver bails out with -EINVAL, but it does so
with a plain return, leaving the clock prepared and enabled.

All the later error paths of the probe already jump to the out_clk
label to disable the clock. Do the same here.

Fixes: cb1b206cff46 ("serial: sc16is7xx: Use devm_clk_get_optional()")
Signed-off-by: laixiangwu <laixiangwu@cmss.chinamobile.com>
---
 drivers/tty/serial/sc16is7xx.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/tty/serial/sc16is7xx.c b/drivers/tty/serial/sc16is7xx.c
index fba588559..e464c6d72 100644
--- a/drivers/tty/serial/sc16is7xx.c
+++ b/drivers/tty/serial/sc16is7xx.c
@@ -1619,10 +1619,12 @@ int sc16is7xx_probe(struct device *dev, const struct sc16is7xx_devtype *devtype,
 			freq = uartclk;
 		if (pfreq)
 			freq = *pfreq;
-		if (freq)
+		if (freq) {
 			dev_dbg(dev, "Clock frequency: %luHz\n", freq);
-		else
-			return -EINVAL;
+		} else {
+			ret = -EINVAL;
+			goto out_clk;
+		}
 	}
 
 	s->devtype = devtype;
-- 

Re: [PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[sashiko-bot]

> sc16is7xx_probe() enables the clock with clk_prepare_enable() early in
> the probe path. If the resulting clock frequency cannot be determined
> (neither from the clock itself, nor from the "clock-frequency" property
> or platform data), the driver bails out with -EINVAL, but it does so
> with a plain return, leaving the clock prepared and enabled.
> 
> All the later error paths of the probe already jump to the out_clk
> label to disable the clock. Do the same here.
> 
> Fixes: cb1b206cff46 ("serial: sc16is7xx: Use devm_clk_get_optional()")
> Signed-off-by: laixiangwu <laixiangwu@cmss.chinamobile.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261030085508.1882-1-laixiangwu@cmss.chinamobile.com?part=1

Re: [PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[Xiangwu Lai]

Hi,

gentle ping.  Is there anything blocking this patch, or does it just
need more time?

Best regards,
Xiangwu Lai

Re: [PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[Greg Kroah-Hartman]

On Wed, Nov 11, 2026 at 05:52:12PM +0800, Xiangwu Lai wrote:
> Hi,
> 
> gentle ping.  Is there anything blocking this patch, or does it just
> need more time?

There is no context here :(

Re: [PATCH] serial: sc16is7xx: Fix clock leak on invalid clock frequency

[Xiangwu Lai]

Hi Greg,

sorry for the missing context.  The patch I was pinging is:

  serial: sc16is7xx: Fix clock leak on invalid clock frequency
  https://lore.kernel.org/all/20261030085508.1882-1-laixiangwu@cmss.chinamobile.com/

sc16is7xx_probe() enables the clock with clk_prepare_enable() early
in the probe path.  If the clock frequency cannot be determined
(neither from the clock itself, nor from the "clock-frequency"
property or platform data), the driver bails out with -EINVAL
without disabling the clock again, while the later error paths of
the probe already jump to the out_clk label.  The patch makes this
error path jump to out_clk as well, so the clock is disabled like
on the other error paths.

It was submitted on Oct 30.  Is there anything blocking it, or does
it just need more time?

Best regards,
Xiangwu Lai