[AMD Official Use Only] > -----Original Message----- > From: Alex Deucher <alexdeucher@xxxxxxxxx> > Sent: Friday, August 13, 2021 9:56 PM > To: Quan, Evan <Evan.Quan@xxxxxxx> > Cc: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>; Deucher, Alexander > <Alexander.Deucher@xxxxxxx>; Chen, Guchun <Guchun.Chen@xxxxxxx> > Subject: Re: [PATCH] drm/amdgpu: disable BACO support for 699F:C7 > polaris12 SKU temporarily > > On Fri, Aug 13, 2021 at 9:54 AM Alex Deucher <alexdeucher@xxxxxxxxx> > wrote: > > > > Acked-by: Alex Deucher <alexander.deucher@xxxxxxx> > > > > On Fri, Aug 13, 2021 at 4:09 AM Evan Quan <evan.quan@xxxxxxx> wrote: > > > > > > We have a S3 issue on that SKU with BACO enabled. Will bring back > > > this when that root caused. > > > > > Actually it might be worth checking the SSIDs as well assuming this is platform > specific. [Quan, Evan] Can you share more how to perform that check? Where to get the SSID? BR Evan > > Alex > > > > Change-Id: I56d4830e6275e20a415808896eecbadfe944070b > > > Signed-off-by: Evan Quan <evan.quan@xxxxxxx> > > > --- > > > drivers/gpu/drm/amd/amdgpu/vi.c | 7 ++++++- > > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c > > > b/drivers/gpu/drm/amd/amdgpu/vi.c index fe9a7cc8d9eb..7210f80815b8 > > > 100644 > > > --- a/drivers/gpu/drm/amd/amdgpu/vi.c > > > +++ b/drivers/gpu/drm/amd/amdgpu/vi.c > > > @@ -904,7 +904,12 @@ static bool vi_asic_supports_baco(struct > amdgpu_device *adev) > > > case CHIP_POLARIS11: > > > case CHIP_POLARIS12: > > > case CHIP_TOPAZ: > > > - return amdgpu_dpm_is_baco_supported(adev); > > > + /* Disable BACO support for the specific polaris12 SKU > temporarily */ > > > + if ((adev->pdev->device == 0x699F) && > > > + (adev->pdev->revision == 0xC7)) > > > + return false; > > > + else > > > + return amdgpu_dpm_is_baco_supported(adev); > > > default: > > > return false; > > > } > > > -- > > > 2.29.0 > > >
