On Fri, Jul 16, 2021 at 05:12:44PM +0200, Werner Sembach wrote: > > Am 16.07.21 um 14:50 schrieb Greg Kroah-Hartman: > > On Fri, Jul 16, 2021 at 12:40:10PM +0200, Werner Sembach wrote: > >> From: Mario Limonciello <mario.limonciello@xxxxxxx> > >> > >> BugLink: https://bugs.launchpad.net/bugs/1936583 > >> > >> The XHCI controller is required to enter D3hot rather than D3cold for AMD > >> s2idle on this hardware generation. > >> > >> Otherwise, the 'Controller Not Ready' (CNR) bit is not being cleared by > >> host in resume and eventually this results in xhci resume failures during > >> the s2idle wakeup. > >> > >> Link: https://lore.kernel.org/linux-usb/1612527609-7053-1-git-send-email-Prike.Liang@xxxxxxx/ > >> Suggested-by: Prike Liang <Prike.Liang@xxxxxxx> > >> Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx> > >> Cc: stable <stable@xxxxxxxxxxxxxxx> # 5.11+ > >> Link: https://lore.kernel.org/r/20210527154534.8900-1-mario.limonciello@xxxxxxx > >> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > >> (cherry picked from commit d1658268e43980c071dbffc3d894f6f6c4b6732a) > >> Signed-off-by: Werner Sembach <wse@xxxxxxxxxxxxxxxxxxx> > >> --- > >> drivers/usb/host/xhci-pci.c | 7 ++++++- > >> drivers/usb/host/xhci.h | 1 + > >> 2 files changed, 7 insertions(+), 1 deletion(-) > >> > > Any reason you resent us a patch that is already in a stable release? > > > > And why not just use the stable kernel trees as-is? Why attempt to > > cherry-pick random portions of them? > > > > thanks, > > > > greg k-h > > I didn't add the mailing list as recipent for my last replies so here again: > > I only checked the Ubuntu 5.11 tree where the patch is actually missing. > > The 5.8 kernel has other issues because of outdated amdgpu, that's why we never checked the 5.4 kernel. > > Testing for 5.4: often hangs on boot before display manager shows up > > 5.4 + amdgpu-dkms from here: https://www.amd.com/en/support/kb/release-notes/rn-amdgpu-unified-linux-21-20 : Hang on > boot issue gone, but does not suspend anymore, and has graphic glitches. > > Should I add these findings to the SRU? What does any of this have to do with the stable@xxxxxxxxxxxxxxx developers?