On Thu, Jul 19, 2018 at 4:46 AM, Rex Zhu <rex.zhu at amd.com> wrote:
> if board uses AZ rather than ACP, we power down acp
> through smu to save power.
>
We also need to power it back up in hw_fini and suspend and then power
it back down in resume.
Alex
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c
> index 71efcf3..4e18a88 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c
> @@ -35,6 +35,7 @@
>
> #include "acp_gfx_if.h"
>
> +
> #define ACP_TILE_ON_MASK 0x03
> #define ACP_TILE_OFF_MASK 0x02
> #define ACP_TILE_ON_RETAIN_REG_MASK 0x1f
> @@ -289,11 +290,12 @@ static int acp_hw_init(void *handle)
> r = amd_acp_hw_init(adev->acp.cgs_device,
> ip_block->version->major, ip_block->version->minor);
> /* -ENODEV means board uses AZ rather than ACP */
> - if (r == -ENODEV)
> + if (r == -ENODEV) {
> + amdgpu_dpm_set_powergating_by_smu(adev, AMD_IP_BLOCK_TYPE_ACP, true);
> return 0;
> - else if (r)
> + } else if (r) {
> return r;
> -
> + }
> if (adev->rmmio_size == 0 || adev->rmmio_size < 0x5289)
> return -EINVAL;
>
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
