On 2017å¹´05æ??17æ?¥ 15:55, Huang Rui wrote: > On Wed, May 17, 2017 at 03:43:47PM +0800, Zhou, David(ChunMing) wrote: >> By this change, I suggest to remove mmhub/gfxhub_v1_0_ip_funcs and their >> IP block, unify them to gmc ip block, this way we cannot lost setting >> when resume back. >> > From hw side, wo won't have real gmc since this chip, mmhub and gfxhub(gc) > instead of it. Maybe we would better to align with hw desgin. I don't see any advance, as you said, we still have gmc block in soc15, why not unify mmhub/gfxhub calls to gmc block? We can keep mmhub/gfxhub_xxx.c file, but ip_funciton isn't necessary. Regards, David Zhou > > Thanks, > Rui > >> Regards, >> David Zhou >> >> On 2017å¹´05æ??17æ?¥ 15:38, Huang Rui wrote: >>> Signed-off-by: Huang Rui <ray.huang at amd.com> >>> --- >>> drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c b/drivers/gpu/drm/amd/ >> amdgpu/gfxhub_v1_0.c >>> index 005075f..41313514 100644 >>> --- a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c >>> +++ b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c >>> @@ -368,7 +368,7 @@ static int gfxhub_v1_0_suspend(void *handle) >>> >>> static int gfxhub_v1_0_resume(void *handle) >>> { >>> - return 0; >>> + return gfxhub_v1_0_hw_init(handle); >>> } >>> >>> static bool gfxhub_v1_0_is_idle(void *handle) > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx