Please combine the 2 close "if (adev->asic_type != CHIP_ARCTURUS) {".With that fixed: Reviewed-by: Le Ma <[email protected]> Regards, Ma Le -----Original Message----- From: amd-gfx <[email protected]> On Behalf Of Evan Quan Sent: Monday, August 05, 2019 3:13 PM To: [email protected] Cc: Quan, Evan <[email protected]> Subject: [PATCH] drm/amd/powerplay: skip pcie params override on Arcturus This is not supported on Arcturus. Affected ASIC: Arcturus Change-Id: I62a8bce17a070ce4eda5fa22f4b12a7ffa1201cd Signed-off-by: Evan Quan <[email protected]> --- drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c index 5ba038260091..7c2c24a291b0 100644 --- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c +++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c @@ -1109,9 +1109,11 @@ static int smu_smc_table_hw_init(struct smu_context *smu, if (ret) return ret; - ret = smu_override_pcie_parameters(smu); - if (ret) - return ret; + if (adev->asic_type != CHIP_ARCTURUS) { + ret = smu_override_pcie_parameters(smu); + if (ret) + return ret; + } if (adev->asic_type != CHIP_ARCTURUS) { ret = smu_notify_display_change(smu); -- 2.22.0 _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
