On Wed, Oct 17, 2012 at 04:52:28PM -0300, Paulo Zanoni wrote: > Hi > > 2012/10/17 Daniel Vetter <daniel.vetter at ffwll.ch>: > > We completely compute these anew in each modeset, hence we don't rely > > on them containing anything valid after resume. > > > > To avoid breaking any ums setup due to reordering of the reads/writes > > simply don't reorder anything, but bracket the reads/writes into if > > (!kms) conditionals. More churn, but safer. > > > > v2: Fixup the logic, noticed by Paulo Zanoni. > > > > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > > For the 3 patches: > Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com> > Tested-by: Paulo Zanoni <paulo.r.zanoni at intel.com> > > Tested only on HSW: S3 and S4 suspend still works. All three merged, thanks for the review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
