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

[Dave Stevenson]

On Tue, 10 Nov 2026 at 14:03, Joshua Crofts <crofts@xxxxxxxxxx> wrote:
>
> 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@xxxxxxxxxx>

Acked-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx>

> ---
>  drivers/media/i2c/imx355.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/media/i2c/imx355.c b/drivers/media/i2c/imx355.c
> index c0dafabf5dc9ad6237c9c9380d6122edfedc96f3..89e42312265822428cbf0b909627c66415b3282f 100644
> --- a/drivers/media/i2c/imx355.c
> +++ b/drivers/media/i2c/imx355.c
> @@ -1298,7 +1298,6 @@ static int imx355_probe(struct i2c_client *client)
>  error_subdev_cleanup_runtime_pm:
>         pm_runtime_disable(imx355->dev);
>         pm_runtime_set_suspended(imx355->dev);
> -       pm_runtime_dont_use_autosuspend(imx355->dev);
>         v4l2_subdev_cleanup(&imx355->sd);
>  error_media_entity_free:
>         media_entity_cleanup(&imx355->sd.entity);
> @@ -1329,7 +1328,6 @@ static void imx355_remove(struct i2c_client *client)
>                 pm_runtime_set_suspended(imx355->dev);
>         }
>
> -       pm_runtime_dont_use_autosuspend(imx355->dev);
>  }
>
>  static const struct acpi_device_id imx355_acpi_ids[] __maybe_unused = {
>
> --
> 2.47.3
>