Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-08-02 Thread Alex Deucher
*To:* Zhu, Rex > *Cc:* Deucher, Alexander; Quan, Evan; amd-gfx@lists.freedesktop.org > > *Subject:* Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support > on Rv/Vega12 > > Does anything use it? Might be useful for debugging, but I guess you'd > probably use something like umr

Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-08-02 Thread Zhu, Rex
Subject: Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12 Does anything use it? Might be useful for debugging, but I guess you'd probably use something like umr in that case. Alex On Tue, Jul 31, 2018 at 9:15 PM, Zhu, Rex mailto:rex@amd.com>> wrot

Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-08-01 Thread Alex Deucher
switch, i do not think the status > reported to user is reliable. It means user was told the gfx is on but > actually it may switches to gfxoff already. > > > Regards, > > Evan > > > ------------------ > *From:* amd-gfx on behalf of Rex > Zhu > *Sent:* Sunday, July 29, 2018 7:42:27 PM

Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-07-31 Thread Zhu, Rex
n; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12 > It means user was told the gfx is on but actually it may switches to gfxoff > already. Correct. So if the gfx is off, user should disable gfx off feature before they

Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-07-29 Thread Zhu, Rex
mean it is safe to visit the registers through mmio. Best Regards Rex From: Quan, Evan Sent: Monday, July 30, 2018 9:31 AM To: Zhu, Rex; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12 Since th

Re: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-07-29 Thread Quan, Evan
29, 2018 7:42:27 PM To: amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject: [PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12 As gfx off is supported on Rv/Vega12, so this check is helpful and necessary when visit gfx regesiter via mmio. Signed-off-by: Rex Zhu --- drivers

[PATCH 3/3] drm/amd/pp: Add gfx power status check support on Rv/Vega12

2018-07-29 Thread Rex Zhu
As gfx off is supported on Rv/Vega12, so this check is helpful and necessary when visit gfx regesiter via mmio. Signed-off-by: Rex Zhu --- drivers/gpu/drm/amd/powerplay/hwmgr/smu10_hwmgr.c | 10 +++ drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c | 31 +- 2 files chan