<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 9, 2021 at 9:37 AM Alex Deucher <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Aug 9, 2021 at 9:59 AM Koba Ko <<a href="mailto:koba.ko@canonical.com" target="_blank">koba.ko@canonical.com</a>> wrote:<br>
><br>
> Previously, AMD had an issue about noise  with AMD-DG on the RKL platform<br>
> AMD provided a parameter.<br>
> #modprobe amdgpu ppfeaturemask=0xfff7bffb<br>
><br>
>  I thought it's better to check and assign values in amd gpu.<br>
> Have a trouble determining the type of pch(RKL or else),<br>
> search in amd drm driver and can't find any about this.<br>
> Would someone please guide me? if there's an existing function.<br>
><br>
> here's a proposal, check RKL PCH in amd driver,<br>
> the pch definitions must be splitted off from intel_pch.h in i915<br>
> folder to include/drm/intel_pch_definition.h<br>
<br>
Yes, something like that would work.<br></blockquote><div><br></div><div>Can the issue that prompted this also happen with other ASIC with the</div><div>newer SMU families?  If so, should it probably be added to all of them</div><div>or further up in the code where the mask normally gets set from module</div><div>parameters to add the extra check there.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Alex<br>
<br>
<br>
><br>
> > --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c<br>
> > +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c<br>
> > @@ -1629,7 +1629,7 @@ static void smu7_init_dpm_defaults(struct pp_hwmgr *hwmgr)<br>
> ><br>
> >         data->mclk_dpm_key_disabled = hwmgr->feature_mask & PP_MCLK_DPM_MASK ? false : true;<br>
> >         data->sclk_dpm_key_disabled = hwmgr->feature_mask & PP_SCLK_DPM_MASK ? false : true;<br>
> > -       data->pcie_dpm_key_disabled = hwmgr->feature_mask & PP_PCIE_DPM_MASK ? false : true;<br>
> > +       data->pcie_dpm_key_disabled = is_rkl_pch() || !(hwmgr->feature_mask & PP_PCIE_DPM_MASK);<br>
> >         /* need to set voltage control types before EVV patching */<br>
> >         data->voltage_control = SMU7_VOLTAGE_CONTROL_NONE;<br>
> >         data->vddci_control = SMU7_VOLTAGE_CONTROL_NONE;<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><span style="font-size:x-small"><font color="#666666">Mario Limonciello<br><a href="mailto:superm1@gmail.com" target="_blank">superm1@gmail.com</a></font></span></div></div>