Re: [PATCH] drm/amdgpu/vpe: power on vpe when hw_init

[Alex Deucher <alexdeucher@xxxxxxxxx>]

On Wed, Mar 13, 2024 at 7:41 AM Peyton Lee <peytolee@xxxxxxx> wrote:
>
> To fix mode2 reset failure.
> Should power on VPE when hw_init.

When does it get powered down again?  Won't this leave it powered up?

Alex

>
> Signed-off-by: Peyton Lee <peytolee@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
> index 70c5cc80ecdc..ecfe0f36e83e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
> @@ -396,6 +396,12 @@ static int vpe_hw_init(void *handle)
>         struct amdgpu_vpe *vpe = &adev->vpe;
>         int ret;
>
> +       /* Power on VPE */
> +       ret = amdgpu_device_ip_set_powergating_state(adev, AMD_IP_BLOCK_TYPE_VPE,
> +                                                    AMD_PG_STATE_UNGATE);
> +       if (ret)
> +               return ret;
> +
>         ret = vpe_load_microcode(vpe);
>         if (ret)
>                 return ret;
> --
> 2.34.1
>