[PATCH 3/3] drm/amdgpu: move cache window setup after power and clock resume

Christian König ckoenig.leichtzumerken at gmail.com
Tue Jul 10 07:13:31 UTC 2018


Looks like the patches should be reordered, e.g. patch #2 should be the 
last.

Apart from that Reviewed-by: Christian König <christian.koenig at amd.com>.

Regards,
Christian.

Am 09.07.2018 um 18:54 schrieb Deucher, Alexander:
>
> Series is:
>
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>
> ------------------------------------------------------------------------
> *From:* amd-gfx <amd-gfx-bounces at lists.freedesktop.org> on behalf of 
> Leo Liu <leo.liu at amd.com>
> *Sent:* Monday, July 9, 2018 12:11:38 PM
> *To:* amd-gfx at lists.freedesktop.org
> *Cc:* Liu, Leo
> *Subject:* [PATCH 3/3] drm/amdgpu: move cache window setup after power 
> and clock resume
> To make register read/write reliable. Along with the previous patch,
> VCN will work with dpm disabled case.
>
> Signed-off-by: Leo Liu <leo.liu at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c 
> b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
> index b82c92084b6f..ca4265bc10b9 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
> @@ -600,12 +600,12 @@ static int vcn_v1_0_start(struct amdgpu_device 
> *adev)
>          /* disable byte swapping */
>          lmi_swap_cntl = 0;
>
> -       vcn_v1_0_mc_resume(adev);
> -
>          vcn_1_0_disable_static_power_gating(adev);
>          /* disable clock gating */
>          vcn_v1_0_disable_clock_gating(adev);
>
> +       vcn_v1_0_mc_resume(adev);
> +
>          /* disable interupt */
>          WREG32_P(SOC15_REG_OFFSET(UVD, 0, mmUVD_MASTINT_EN), 0,
>                          ~UVD_MASTINT_EN__VCPU_EN_MASK);
> -- 
> 2.17.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180710/560084f0/attachment.html>


More information about the amd-gfx mailing list