On Wed, May 16, 2018 at 8:52 AM, Rex Zhu <rex....@amd.com> wrote:
> Signed-off-by: Rex Zhu <rex....@amd.com>
Please add a patch description. E.g., something like:
VCN clockgating is handled manually like VCE and UVD.
With that fixed:
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index fcd4bb2..25bee46 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -1718,6 +1718,7 @@ static int amdgpu_device_ip_late_set_cg_state(struct
> amdgpu_device *adev)
> /* skip CG for VCE/UVD, it's handled specially */
> if (adev->ip_blocks[i].version->type != AMD_IP_BLOCK_TYPE_UVD
> &&
> adev->ip_blocks[i].version->type != AMD_IP_BLOCK_TYPE_VCE
> &&
> + adev->ip_blocks[i].version->type != AMD_IP_BLOCK_TYPE_VCN
> &&
> adev->ip_blocks[i].version->funcs->set_clockgating_state)
> {
> /* enable clockgating to save power */
> r =
> adev->ip_blocks[i].version->funcs->set_clockgating_state((void *)adev,
> @@ -1817,6 +1818,7 @@ static int amdgpu_device_ip_fini(struct amdgpu_device
> *adev)
>
> if (adev->ip_blocks[i].version->type != AMD_IP_BLOCK_TYPE_UVD
> &&
> adev->ip_blocks[i].version->type !=
> AMD_IP_BLOCK_TYPE_VCE &&
> + adev->ip_blocks[i].version->type !=
> AMD_IP_BLOCK_TYPE_VCN &&
>
> adev->ip_blocks[i].version->funcs->set_clockgating_state) {
> /* ungate blocks before hw fini so that we can
> shutdown the blocks safely */
> r =
> adev->ip_blocks[i].version->funcs->set_clockgating_state((void *)adev,
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
