[AMD Official Use Only - General]
> -----Original Message-----
> From: Quan, Evan <Evan.Quan@xxxxxxx>
> Sent: Wednesday, June 15, 2022 22:37
> To: Alex Deucher <alexdeucher@xxxxxxxxx>; Limonciello, Mario
> <Mario.Limonciello@xxxxxxx>; Feng, Kenneth <Kenneth.Feng@xxxxxxx>
> Cc: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
> Subject: RE: [PATCH] drm/amd: Don't show warning on reading vbios values
> for SMU13 3.1
>
> [AMD Official Use Only - General]
>
> Thanks for the fix. Yes, the support for APUs was missing from the relevant
> code.
> I think you can just drop the piece of code below.
> /////////////
> else {
> dev_warn(smu->adev->dev, "Unexpected and unhandled
> version: %d.%d\n",
> (uint32_t)frev, (uint32_t)crev);
> }
> /////////////
> Either-way the patch is reviewed-by: Evan Quan <evan.quan@xxxxxxx>
>
Thanks for reviewing this. I lean on leaving the warning in place and just catching
the APU case. That way if a new table is introduced it helps to make sure that it
really was updated here.
> BR
> Evan
> > -----Original Message-----
> > From: Alex Deucher <alexdeucher@xxxxxxxxx>
> > Sent: Thursday, June 16, 2022 11:12 AM
> > To: Limonciello, Mario <Mario.Limonciello@xxxxxxx>; Quan, Evan
> > <Evan.Quan@xxxxxxx>; Feng, Kenneth <Kenneth.Feng@xxxxxxx>
> > Cc: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
> > Subject: Re: [PATCH] drm/amd: Don't show warning on reading vbios
> values
> > for SMU13 3.1
> >
> > On Tue, Jun 14, 2022 at 10:22 AM Mario Limonciello
> > <mario.limonciello@xxxxxxx> wrote:
> > >
> > > Some APUs with SMU13 are showing the following message:
> > > `amdgpu 0000:63:00.0: amdgpu: Unexpected and unhandled version: 3.1`
> > >
> > > This warning isn't relevant for smu info 3.1, as no bootup information
> > > is present in the table.
> > >
> > > Fixes: 6a73454350c0 ("drm/amd/pm: correct the way for retrieving
> > > bootup clocks")
> > > Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx>
> > > ---
> > > drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c | 2 ++
> > > 1 file changed, 2 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> > > b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> > > index f18f9605e586..0e59ab2192bf 100644
> > > --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> > > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> > > @@ -731,6 +731,8 @@ int smu_v13_0_get_vbios_bootup_values(struct
> > smu_context *smu)
> > > smu->smu_table.boot_values.vclk = smu_info_v3_6-
> > >bootup_vclk_10khz;
> > > smu->smu_table.boot_values.dclk = smu_info_v3_6-
> > >bootup_dclk_10khz;
> > > smu->smu_table.boot_values.fclk =
> > > smu_info_v3_6->bootup_fclk_10khz;
> > > + } else if ((frev == 3) && (crev == 1)) {
> > > + return 0;
> >
> > I don't know if Evan has any comments, but it might be better to just return
> if
> > ((frev == 3) && (crev < 2)). I'm not sure off hand which smu_info table
> > versions we might run into. Evan or Kenneth would know which smu_info
> > table versions are valid on which SMU 13.x asics.
> >
> > A;ex
> >
> >
> > > } else if ((frev == 4) && (crev == 0)) {
> > > smu_info_v4_0 = (struct atom_smu_info_v4_0
> > > *)header;
> > >
> > > --
> > > 2.25.1
> > >
