On Fri, 2014-03-07 at 20:12 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > > Hi > > This series depends on: > - [PATCH 0/6] More runtime PM fixes > - [PATCH 00/16] Merge PC8 with runtime PM, v3 > - [PATCH 00/20] ILK+ interrupt improvements, v2 > > Patches 1 and 2 are the last pieces of my runtime PM rework, and patch 1 needs > all the previous patches to be merged before it can be merged. > > Patch 3 adds runtime PM support to SNB. It's not widely tested and I haven't > tested it for a while, but at some point this was all that was needed to make my > SNB laptop work. > > Patches 4-6 are the BDW bits, but I also haven't tested them since the last > rebase, and there's the possibility that something is still missing. I wouldn't > recommend applying this to production trees yet, but I would recommend you to > apply, test the patches and report the results to me! > > More importantly, the basic reason why I sent this series to the list is that if > someone wants to add runtime PM support to some other platform (BYT anyone?), Yea, I'm trying right now to enable this on top of this patchset. > they should really try to base their patches on top of these. And they should > really look at the SNB patch to have some idea on what they need to do first. > > Q: Why don't we have IVB runtime PM support? > A: Just because it was easier for me to do development and test on a SNB > machine. I really expect that IVB runtime PM will just reuse the SNB bits. > The only thing missing for IVB is probably just a few "IS_IVYBRIDGE()" calls, > and a call to modeset_update_crtc_power_domains(). I can provide a patch for > this later. > > Thanks, > Paulo > > Paulo Zanoni (6): > drm/i915: kill dev_priv->pm.regsave > drm/i915: add gen-specific runtime suspend/resume functions > drm/i915: add SNB runtime PM support > drm/i915: remove HAS_PC8 check > drm/i915: BDW needs D_COMP writes through MCHBAR > drm/i915: add BDW runtime PM support These all have now a r-b from me, except the SNB one. That one looks ok too, but I'd need some clarification on the context save/restore semantics. --Imre > drivers/gpu/drm/i915/i915_drv.c | 53 ++++++++++++++++++++---- > drivers/gpu/drm/i915/i915_drv.h | 16 ++------ > drivers/gpu/drm/i915/i915_irq.c | 79 +++++------------------------------- > drivers/gpu/drm/i915/intel_display.c | 43 ++++++++++++-------- > drivers/gpu/drm/i915/intel_drv.h | 4 +- > 5 files changed, 87 insertions(+), 108 deletions(-) >
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
