drivers/pwm/pwm-brcmstb.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
Check the clk_prepare_enable() return value and propagate it.
Suggested-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>
---
This change depends upon:
https://lore.kernel.org/all/20231004175414.1738475-1-florian.fainelli@broadcom.com/
drivers/pwm/pwm-brcmstb.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/pwm/pwm-brcmstb.c b/drivers/pwm/pwm-brcmstb.c
index a7d529bf76ad..de9cbda2e4ff 100644
--- a/drivers/pwm/pwm-brcmstb.c
+++ b/drivers/pwm/pwm-brcmstb.c
@@ -297,9 +297,7 @@ static int brcmstb_pwm_resume(struct device *dev)
{
struct brcmstb_pwm *p = dev_get_drvdata(dev);
- clk_prepare_enable(p->clk);
-
- return 0;
+ return clk_prepare_enable(p->clk);
}
#endif
--
2.34.1
On Thu, 05 Oct 2023 09:47:27 -0700, Florian Fainelli wrote:
> Check the clk_prepare_enable() return value and propagate it.
>
>
Applied, thanks!
[1/1] pwm: brcmstb: Checked clk_prepare_enable() return value
commit: d559f11591900a283b2a0b1087d68992e6c2062a
Best regards,
--
Thierry Reding <thierry.reding@gmail.com>
On Thu, Oct 05, 2023 at 09:47:27AM -0700, Florian Fainelli wrote: > Check the clk_prepare_enable() return value and propagate it. > > Suggested-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com> Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
© 2016 - 2026 Red Hat, Inc.