For what it's worth this commit also breaks resume on my Tonga only system so it's not specific to just Carrizo. On 13/10/17 08:16 AM, Tom St Denis wrote: > Hi, > > This commit breaks suspend/resume on my Carrizo A12-9800 system. > > [root at carrizo linux2]# git bisect bad > 7ae4acd21e9e264afb079e23d43bcf2238c7dbea is the first bad commit > commit 7ae4acd21e9e264afb079e23d43bcf2238c7dbea > Author: Leo (Sunpeng) Li <sunpeng.li at amd.com> > Date:Â Â Thu Sep 7 17:05:38 2017 -0400 > > Â Â Â drm/amd/display: Match actual state during S3 resume. > > Â Â Â During system suspend, we: > > Â Â Â 1. Cache a duplicate of the current DRM atomic state, which calls > hooks > Â Â Â Â Â Â to duplicate our internal states. > Â Â Â 2. Call hooks to disable all functionalities. > Â Â Â 3. System suspended. > > Â Â Â During resume, we attempt to restore the cached state. However, our > Â Â Â interal states are now stale, since step 1 was done before step 2. > Â Â Â i.e. our cached state does not reflect the disabled nature of things. > > Â Â Â This change resolves that by destroying all relevant states to reflect > Â Â Â the actual state during resume. > > Â Â Â Signed-off-by: Leo (Sunpeng) Li <sunpeng.li at amd.com> > Â Â Â Reviewed-by: Harry Wentland <Harry.Wentland at amd.com> > > Tom > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
