Re: [PATCH 4/4] media: i2c: vd56g3: remove redundant pm_runtime_dont_use_autosuspend() calls

[Benjamin Mugnier <benjamin.mugnier@foss.st.com>]

Le 10/11/2026 à 15:03, Joshua Crofts a écrit :
> Since commit d1706f9da9f4 ("PM: runtime: call pm_runtime_dont_use_autosuspend() on reinit()"),
> driver core is now responsible for undoing any
> pm_runtime_use_autosuspend() calls on driver exit.
> 
> Remove the redundant function calls.
> 
> Signed-off-by: Joshua Crofts <crofts@kernel.org>

Reviewed-by: Benjamin Mugnier <benjamin.mugnier@foss.st.com>

> ---
>  drivers/media/i2c/vd56g3.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/media/i2c/vd56g3.c b/drivers/media/i2c/vd56g3.c
> index a42210d0ab8d54a9132a5077875138c2440152bb..e4ad01973a24201a20dd43c86ac6de9817889049 100644
> --- a/drivers/media/i2c/vd56g3.c
> +++ b/drivers/media/i2c/vd56g3.c
> @@ -1538,7 +1538,6 @@ static int vd56g3_probe(struct i2c_client *client)
>  err_power_off:
>  	pm_runtime_disable(dev);
>  	pm_runtime_put_noidle(dev);
> -	pm_runtime_dont_use_autosuspend(dev);
>  	vd56g3_power_off(dev);
>  
>  	return ret;
> @@ -1555,7 +1554,6 @@ static void vd56g3_remove(struct i2c_client *client)
>  	if (!pm_runtime_status_suspended(sensor->dev))
>  		vd56g3_power_off(sensor->dev);
>  	pm_runtime_set_suspended(sensor->dev);
> -	pm_runtime_dont_use_autosuspend(sensor->dev);
>  }
>  
>  static const struct of_device_id vd56g3_dt_ids[] = {
> 

-- 
Regards,
Benjamin