[AMD Official Use Only - General] > -----Original Message----- > From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of > Stanley.Yang > Sent: Wednesday, October 18, 2023 8:22 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Yang, Stanley <Stanley.Yang@xxxxxxx> > Subject: [PATCH Review 1/1] drm/amdgpu: Enable mca debug mode mode for apu [Tao] the "mode" is duplicated. > > Enable smu_v13_0_6 mca debug mode when GFX RAS feature is enabled on APU. > > Signed-off-by: Stanley.Yang <Stanley.Yang@xxxxxxx> > --- > drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > index cab5a5569bc6..62589ba3c4a5 100644 > --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > @@ -2298,7 +2298,8 @@ static int smu_v13_0_6_post_init(struct smu_context > *smu) { > struct amdgpu_device *adev = smu->adev; > > - if (!amdgpu_sriov_vf(adev) && amdgpu_ras_is_supported(adev, > AMDGPU_RAS_BLOCK__UMC)) > + if (!amdgpu_sriov_vf(adev) && (amdgpu_ras_is_supported(adev, > AMDGPU_RAS_BLOCK__UMC) || > + (adev->is_app_apu && amdgpu_ras_is_supported(adev, > +AMDGPU_RAS_BLOCK__GFX)))) [Tao] Can "adev->is_app_apu" be dropped? > return smu_v13_0_6_mca_set_debug_mode(smu, true); > > return 0; > -- > 2.25.1