[PATCH] drm/amdgpu/vcn: drop extra cancel_delayed_work_sync()

Alex Deucher alexdeucher at gmail.com
Wed Aug 20 18:36:13 UTC 2025


Ping?

On Wed, Aug 13, 2025 at 3:37 PM Alex Deucher <alexander.deucher at amd.com> wrote:
>
> We already call this in the hw_fini() methods for all
> VCN instances, so no need to call it again in
> amdgpu_vcn_suspend().
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
> index 9a76e11d1c184..fd8ebf4b5a824 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
> @@ -357,8 +357,6 @@ int amdgpu_vcn_suspend(struct amdgpu_device *adev, int i)
>         if (adev->vcn.harvest_config & (1 << i))
>                 return 0;
>
> -       cancel_delayed_work_sync(&adev->vcn.inst[i].idle_work);
> -
>         /* err_event_athub and dpc recovery will corrupt VCPU buffer, so we need to
>          * restore fw data and clear buffer in amdgpu_vcn_resume() */
>         if (in_ras_intr || adev->pcie_reset_ctx.in_link_reset)
> --
> 2.50.1
>


More information about the amd-gfx mailing list