On Fri, 26 Apr 2024 17:42:13 +0200
Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote:
> From: Nuno Sa <nuno.sa@xxxxxxxxxx>
>
> We can only access the IP core registers if the bus clock is enabled. As
> such we need to get and enable it and not rely on anyone else to do it.
>
> Note this clock is a very fundamental one that is typically enabled
> pretty early during boot. Independently of that, we should really rely on
> it to be enabled.
>
> Fixes: ef04070692a2 ("iio: adc: adi-axi-adc: add support for AXI ADC IP core")
> Signed-off-by: Nuno Sa <nuno.sa@xxxxxxxxxx>
Queued up this and the previous one.
If the rest doesn't quite make the merge window I'd like these two fixed
to do so anyway.
Also marked for stable.
Thanks,
Jonathan
> ---
> drivers/iio/adc/adi-axi-adc.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/iio/adc/adi-axi-adc.c b/drivers/iio/adc/adi-axi-adc.c
> index a543b91124b0..e3b215882941 100644
> --- a/drivers/iio/adc/adi-axi-adc.c
> +++ b/drivers/iio/adc/adi-axi-adc.c
> @@ -175,6 +175,7 @@ static int adi_axi_adc_probe(struct platform_device *pdev)
> struct adi_axi_adc_state *st;
> void __iomem *base;
> unsigned int ver;
> + struct clk *clk;
> int ret;
>
> st = devm_kzalloc(&pdev->dev, sizeof(*st), GFP_KERNEL);
> @@ -195,6 +196,10 @@ static int adi_axi_adc_probe(struct platform_device *pdev)
> if (!expected_ver)
> return -ENODEV;
>
> + clk = devm_clk_get_enabled(&pdev->dev, NULL);
> + if (IS_ERR(clk))
> + return PTR_ERR(clk);
> +
> /*
> * Force disable the core. Up to the frontend to enable us. And we can
> * still read/write registers...
>
