> -----Original Message----- > From: Limonciello, Mario <Mario.Limonciello@xxxxxxx> > Sent: Friday, February 11, 2022 12:09 AM > To: Alex Deucher <alexdeucher@xxxxxxxxx>; Liang, Prike > <Prike.Liang@xxxxxxx> > Cc: Mahapatra, Rajib <Rajib.Mahapatra@xxxxxxx>; Deucher, Alexander > <Alexander.Deucher@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; S, Shirish > <Shirish.S@xxxxxxx> > Subject: RE: [PATCH v2] drm/amdgpu: skipping SDMA hw_init and hw_fini for > S0ix. > > [Public] > > > > VG doesn't do s0i3 right? > > > > Right. > > > > > No, YC should not take a similar fix. YC had an architectural change and > to > > > avoid a "similar" problem takes > > 26db706a6d77b9e184feb11725e97e53b7a89519. > > > > Isn't that likely just a workaround for the same issue? This seems cleaner. > > > > The SMU doesn't handle the restore of the SDMA registers for YC though, > this explicitly changed. So I don't believe we can do an identical fix there. > > @Liang, Prike comments? Yeah, in the gfx10 series looks the SMU doesn't handle SDMA save and restore in the PMFW anymore.