Re: [PATCH 1/2] drm/amdgpu/gfx8: enable cp/rlc ints after we disable clockgating

2017-04-05 Thread Alex Deucher
Ping?

On Tue, Mar 14, 2017 at 3:32 PM, Alex Deucher  wrote:
> Even if we disable clockgating, we still need to make sure the
> cp/rlc interrupts are enabled for powergating which might still
> be enabled.
>
> Signed-off-by: Alex Deucher 
> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c 
> b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> index a53e36c..c9d9913 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> @@ -6258,6 +6258,8 @@ static void 
> gfx_v8_0_update_coarse_grain_clock_gating(struct amdgpu_device *adev
>   RLC_CGCG_CGLS_CTRL__CGLS_EN_MASK);
> if (temp != data)
> WREG32(mmRLC_CGCG_CGLS_CTRL, data);
> +   /* enable interrupts again for PG */
> +   gfx_v8_0_enable_gui_idle_interrupt(adev, true);
> }
>
> gfx_v8_0_wait_for_rlc_serdes(adev);
> --
> 2.5.5
>
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


Re: [PATCH 1/2] drm/amdgpu/gfx8: enable cp/rlc ints after we disable clockgating

2017-03-16 Thread Tom St Denis

On 16/03/17 12:24 PM, Alex Deucher wrote:

On Tue, Mar 14, 2017 at 3:32 PM, Alex Deucher  wrote:

Even if we disable clockgating, we still need to make sure the
cp/rlc interrupts are enabled for powergating which might still
be enabled.

Signed-off-by: Alex Deucher 


Anyone?


Haven't had a chance to try it out (only one devel board...).

In principle that should work provided this is called when CGCG is disabled.

Tom
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


Re: [PATCH 1/2] drm/amdgpu/gfx8: enable cp/rlc ints after we disable clockgating

2017-03-16 Thread Alex Deucher
On Tue, Mar 14, 2017 at 3:32 PM, Alex Deucher  wrote:
> Even if we disable clockgating, we still need to make sure the
> cp/rlc interrupts are enabled for powergating which might still
> be enabled.
>
> Signed-off-by: Alex Deucher 

Anyone?

> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c 
> b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> index a53e36c..c9d9913 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> @@ -6258,6 +6258,8 @@ static void 
> gfx_v8_0_update_coarse_grain_clock_gating(struct amdgpu_device *adev
>   RLC_CGCG_CGLS_CTRL__CGLS_EN_MASK);
> if (temp != data)
> WREG32(mmRLC_CGCG_CGLS_CTRL, data);
> +   /* enable interrupts again for PG */
> +   gfx_v8_0_enable_gui_idle_interrupt(adev, true);
> }
>
> gfx_v8_0_wait_for_rlc_serdes(adev);
> --
> 2.5.5
>
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


[PATCH 1/2] drm/amdgpu/gfx8: enable cp/rlc ints after we disable clockgating

2017-03-14 Thread Alex Deucher
Even if we disable clockgating, we still need to make sure the
cp/rlc interrupts are enabled for powergating which might still
be enabled.

Signed-off-by: Alex Deucher 
---
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
index a53e36c..c9d9913 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
@@ -6258,6 +6258,8 @@ static void 
gfx_v8_0_update_coarse_grain_clock_gating(struct amdgpu_device *adev
  RLC_CGCG_CGLS_CTRL__CGLS_EN_MASK);
if (temp != data)
WREG32(mmRLC_CGCG_CGLS_CTRL, data);
+   /* enable interrupts again for PG */
+   gfx_v8_0_enable_gui_idle_interrupt(adev, true);
}
 
gfx_v8_0_wait_for_rlc_serdes(adev);
-- 
2.5.5

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx