Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread Agrawal, Akshu
On 10/26/2018 8:34 AM, Alex Deucher wrote: > On Thu, Oct 25, 2018 at 10:52 PM Agrawal, Akshu wrote: >> >> >> >> On 10/26/2018 8:09 AM, Alex Deucher wrote: >>> On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8.

Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread Alex Deucher
On Thu, Oct 25, 2018 at 10:52 PM Agrawal, Akshu wrote: > > > > On 10/26/2018 8:09 AM, Alex Deucher wrote: > > On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: > >> > >> This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. > >> > >> Reason for revert: > >> This patch sends msg

Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread Agrawal, Akshu
On 10/26/2018 8:09 AM, Alex Deucher wrote: > On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: >> >> This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. >> >> Reason for revert: >> This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) >> in wrong of sequence to SMU which is

Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread Alex Deucher
On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: > > This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. > > Reason for revert: > This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) > in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). > This leads to

RE: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread Zhu, Rex
Patch is Acked-by: Rex Zhu Regards Rex > -Original Message- > From: S, Shirish > Sent: Friday, October 26, 2018 12:28 AM > To: Zhu, Rex ; Agrawal, Akshu > ; Deucher, Alexander > > Cc: amd-gfx@lists.freedesktop.org; S, Shirish > Subject: [PATCH] Revert &q

[PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"

2018-10-25 Thread S, Shirish
This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. Reason for revert: This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). This leads to SMU failing to service the request as it is dependent on UVD to