Re: [PATCH] iio: adc: at91-sama5d2_adc: no devm for nvmem_cell_get

[Nuno Sá]

On Tue, 2026-03-03 at 20:18 -0800, Rosen Penev wrote:
> There is absolutely no reason to pospone cleanup of this post driver
> removal. Just do it immediately.
> 
> Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx>
> ---

I guess we do not have that many users for nvmem_cell_get(), otherwise a new __free(nvmem_cell)
could make sense. Anyways not related:

Reviewed-by: Nuno Sá <nuno.sa@xxxxxxxxxx>

>  drivers/iio/adc/at91-sama5d2_adc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/at91-sama5d2_adc.c b/drivers/iio/adc/at91-sama5d2_adc.c
> index 69bb49434f90..255970b2e747 100644
> --- a/drivers/iio/adc/at91-sama5d2_adc.c
> +++ b/drivers/iio/adc/at91-sama5d2_adc.c
> @@ -2259,7 +2259,7 @@ static int at91_adc_temp_sensor_init(struct at91_adc_state *st,
>  		return 0;
>  
>  	/* Get the calibration data from NVMEM. */
> -	temp_calib = devm_nvmem_cell_get(dev, "temperature_calib");
> +	temp_calib = nvmem_cell_get(dev, "temperature_calib");
>  	if (IS_ERR(temp_calib)) {
>  		ret = PTR_ERR(temp_calib);
>  		if (ret != -ENOENT)
> @@ -2268,6 +2268,7 @@ static int at91_adc_temp_sensor_init(struct at91_adc_state *st,
>  	}
>  
>  	buf = nvmem_cell_read(temp_calib, &len);
> +	nvmem_cell_put(temp_calib);
>  	if (IS_ERR(buf)) {
>  		dev_err(dev, "Failed to read calibration data!\n");
>  		return PTR_ERR(buf);