* [PATCH] iio: adc: rockchip: Fix clock initialization sequence
@ 2025-03-12 6:20 Simon Xue
2025-03-12 7:00 ` Heiko Stuebner
0 siblings, 1 reply; 3+ messages in thread
From: Simon Xue @ 2025-03-12 6:20 UTC (permalink / raw)
To: Jonathan Cameron, Heiko Stuebner
Cc: Lars-Peter Clausen, Andy Shevchenko, Shreeya Patel, linux-iio,
linux-arm-kernel, linux-rockchip, linux-kernel, Simon Xue
clock_set_rate should be executed after devm_clk_get_enabled.
Fixes: 97ad10bb2901 ("iio: adc: rockchip_saradc: Make use of devm_clk_get_enabled")
Signed-off-by: Simon Xue <xxm@rock-chips.com>
---
drivers/iio/adc/rockchip_saradc.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/iio/adc/rockchip_saradc.c b/drivers/iio/adc/rockchip_saradc.c
index a29e54754c8f..ab4de67fb135 100644
--- a/drivers/iio/adc/rockchip_saradc.c
+++ b/drivers/iio/adc/rockchip_saradc.c
@@ -480,15 +480,6 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
if (info->reset)
rockchip_saradc_reset_controller(info->reset);
- /*
- * Use a default value for the converter clock.
- * This may become user-configurable in the future.
- */
- ret = clk_set_rate(info->clk, info->data->clk_rate);
- if (ret < 0)
- return dev_err_probe(&pdev->dev, ret,
- "failed to set adc clk rate\n");
-
ret = regulator_enable(info->vref);
if (ret < 0)
return dev_err_probe(&pdev->dev, ret,
@@ -515,6 +506,14 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
if (IS_ERR(info->clk))
return dev_err_probe(&pdev->dev, PTR_ERR(info->clk),
"failed to get adc clock\n");
+ /*
+ * Use a default value for the converter clock.
+ * This may become user-configurable in the future.
+ */
+ ret = clk_set_rate(info->clk, info->data->clk_rate);
+ if (ret < 0)
+ return dev_err_probe(&pdev->dev, ret,
+ "failed to set adc clk rate\n");
platform_set_drvdata(pdev, indio_dev);
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] iio: adc: rockchip: Fix clock initialization sequence
2025-03-12 6:20 [PATCH] iio: adc: rockchip: Fix clock initialization sequence Simon Xue
@ 2025-03-12 7:00 ` Heiko Stuebner
2025-03-15 18:40 ` Jonathan Cameron
0 siblings, 1 reply; 3+ messages in thread
From: Heiko Stuebner @ 2025-03-12 7:00 UTC (permalink / raw)
To: Jonathan Cameron, Simon Xue
Cc: Lars-Peter Clausen, Andy Shevchenko, Shreeya Patel, linux-iio,
linux-arm-kernel, linux-rockchip, linux-kernel, Simon Xue
Am Mittwoch, 12. März 2025, 07:20:16 MEZ schrieb Simon Xue:
> clock_set_rate should be executed after devm_clk_get_enabled.
>
> Fixes: 97ad10bb2901 ("iio: adc: rockchip_saradc: Make use of devm_clk_get_enabled")
> Signed-off-by: Simon Xue <xxm@rock-chips.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---
> drivers/iio/adc/rockchip_saradc.c | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/iio/adc/rockchip_saradc.c b/drivers/iio/adc/rockchip_saradc.c
> index a29e54754c8f..ab4de67fb135 100644
> --- a/drivers/iio/adc/rockchip_saradc.c
> +++ b/drivers/iio/adc/rockchip_saradc.c
> @@ -480,15 +480,6 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
> if (info->reset)
> rockchip_saradc_reset_controller(info->reset);
>
> - /*
> - * Use a default value for the converter clock.
> - * This may become user-configurable in the future.
> - */
> - ret = clk_set_rate(info->clk, info->data->clk_rate);
> - if (ret < 0)
> - return dev_err_probe(&pdev->dev, ret,
> - "failed to set adc clk rate\n");
> -
> ret = regulator_enable(info->vref);
> if (ret < 0)
> return dev_err_probe(&pdev->dev, ret,
> @@ -515,6 +506,14 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
> if (IS_ERR(info->clk))
> return dev_err_probe(&pdev->dev, PTR_ERR(info->clk),
> "failed to get adc clock\n");
> + /*
> + * Use a default value for the converter clock.
> + * This may become user-configurable in the future.
> + */
> + ret = clk_set_rate(info->clk, info->data->clk_rate);
> + if (ret < 0)
> + return dev_err_probe(&pdev->dev, ret,
> + "failed to set adc clk rate\n");
>
> platform_set_drvdata(pdev, indio_dev);
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] iio: adc: rockchip: Fix clock initialization sequence
2025-03-12 7:00 ` Heiko Stuebner
@ 2025-03-15 18:40 ` Jonathan Cameron
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Cameron @ 2025-03-15 18:40 UTC (permalink / raw)
To: Heiko Stuebner
Cc: Simon Xue, Lars-Peter Clausen, Andy Shevchenko, Shreeya Patel,
linux-iio, linux-arm-kernel, linux-rockchip, linux-kernel
On Wed, 12 Mar 2025 08:00:54 +0100
Heiko Stuebner <heiko@sntech.de> wrote:
> Am Mittwoch, 12. März 2025, 07:20:16 MEZ schrieb Simon Xue:
> > clock_set_rate should be executed after devm_clk_get_enabled.
> >
> > Fixes: 97ad10bb2901 ("iio: adc: rockchip_saradc: Make use of devm_clk_get_enabled")
> > Signed-off-by: Simon Xue <xxm@rock-chips.com>
>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
ouch.
Applied and marked for stable.
Thanks,
Jonathan
>
> > ---
> > drivers/iio/adc/rockchip_saradc.c | 17 ++++++++---------
> > 1 file changed, 8 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/iio/adc/rockchip_saradc.c b/drivers/iio/adc/rockchip_saradc.c
> > index a29e54754c8f..ab4de67fb135 100644
> > --- a/drivers/iio/adc/rockchip_saradc.c
> > +++ b/drivers/iio/adc/rockchip_saradc.c
> > @@ -480,15 +480,6 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
> > if (info->reset)
> > rockchip_saradc_reset_controller(info->reset);
> >
> > - /*
> > - * Use a default value for the converter clock.
> > - * This may become user-configurable in the future.
> > - */
> > - ret = clk_set_rate(info->clk, info->data->clk_rate);
> > - if (ret < 0)
> > - return dev_err_probe(&pdev->dev, ret,
> > - "failed to set adc clk rate\n");
> > -
> > ret = regulator_enable(info->vref);
> > if (ret < 0)
> > return dev_err_probe(&pdev->dev, ret,
> > @@ -515,6 +506,14 @@ static int rockchip_saradc_probe(struct platform_device *pdev)
> > if (IS_ERR(info->clk))
> > return dev_err_probe(&pdev->dev, PTR_ERR(info->clk),
> > "failed to get adc clock\n");
> > + /*
> > + * Use a default value for the converter clock.
> > + * This may become user-configurable in the future.
> > + */
> > + ret = clk_set_rate(info->clk, info->data->clk_rate);
> > + if (ret < 0)
> > + return dev_err_probe(&pdev->dev, ret,
> > + "failed to set adc clk rate\n");
> >
> > platform_set_drvdata(pdev, indio_dev);
> >
> >
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-03-15 18:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-12 6:20 [PATCH] iio: adc: rockchip: Fix clock initialization sequence Simon Xue
2025-03-12 7:00 ` Heiko Stuebner
2025-03-15 18:40 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).