[PATCH] drm/radeon: s2disk: Leave hw powered between freeze()

Oliver Winker oliver at oli1170.net
Sun Jul 19 09:21:04 PDT 2015

Just for info:

I also tested the https://patchwork.kernel.org/patch/6641371 and 6641381 
patches together with 0001-drm-radeon-rework-device-power-management- for-

It hangs on resume after image restore. 


On Sunday 19 July 2015 17:03:14 Oliver Winker wrote:
> Hello Jerome,
> Just to let you know that I'm back and a small intermediate status on this
> story:
> On Thursday 18 June 2015 23:16:57 Jerome Glisse wrote:
> > Anyway as it is your patch would break runtime suspend.
> > 
> > I recently posted patch to fix the resume side of hibernation, they might
> > also help in the freeze()/thaw() dance. Could you please test them without
> > your patch :
> > https://patchwork.kernel.org/patch/6641371/
> > https://patchwork.kernel.org/patch/6641381/
> I tested those two patches, but they (at least separately) didn't give an
> s2disk improvement.
> > If it is not enough than also try with attached patch (which i have not
> > tested myself but i think the logic is right).
> The reworked patch seemed to worked for the s2disk snapshot-taking
> freeze()->thaw(), but the resume() of the restored image didn't work then.
> Attached is a patch onto your
> 0001-drm-radeon-rework-device-power-management- for-radeon.patch, which
> basically realigns the logic to the previous flow of my initial patch. With
> this it works ok again.
> In the freeze() case it does again the "unpin the front buffers" part and in
> thaw() in again does "init dig PHYs, disp eng pll" plus what follows.
> I need to test a bit more which is the minimum delta to have it working with
> regards to your patch.
> The attached patch is just for info. It doesn't reindent the code to keep it
> more readable for the time being.
> So I'll continue "tuning" this is a bit. But I do this just on the
> high-level suspend/resume() sequence that I see in the from the existing
> code. Without deeper radeon-hw-reasoning behind.
> BR, Oliver

More information about the xorg-driver-ati mailing list