RE: [PATCH 2/2] drm/amdgpu: disable runpm if other reset method not runpm capable is chosen

2021-03-22 Thread Quan, Evan
[AMD Public Use] Thanks Alex. Yes, you are right. BACO is used as a power saving feature for runpm. I missed that. I wrongly treated BACO as a reset method only which cannot be used with mode1 reset at the same time. Please ignore this patch. BR Evan From: Deucher, Alexander Sent: Saturday,

Re: [PATCH 2/2] drm/amdgpu: disable runpm if other reset method not runpm capable is chosen

2021-03-19 Thread Deucher, Alexander
[AMD Public Use] Yeah, the reset method is independent of whether we can use BACO for runtime PM. We can use BACO for power saving and use mode1 for reset. Is there some issue I'm missing? Alex From: amd-gfx on behalf of Lazar, Lijo Sent: Friday, March

RE: [PATCH 2/2] drm/amdgpu: disable runpm if other reset method not runpm capable is chosen

2021-03-19 Thread Lazar, Lijo
[AMD Public Use] -Original Message- From: amd-gfx On Behalf Of Evan Quan Sent: Friday, March 19, 2021 2:42 PM To: amd-gfx@lists.freedesktop.org Cc: Quan, Evan Subject: [PATCH 2/2] drm/amdgpu: disable runpm if other reset method not runpm capable is chosen Otherwise, the runpm will

[PATCH 2/2] drm/amdgpu: disable runpm if other reset method not runpm capable is chosen

2021-03-19 Thread Evan Quan
Otherwise, the runpm will be always enabled on a BACO capable target even the reset method was forced as like mode1. Change-Id: If6bf55c533e91470c9c83383788466161608f68d Signed-off-by: Evan Quan --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +-