On Fri, 8 Jan 2010 16:50:57 -0800 (PST) Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > > > On Sat, 9 Jan 2010, Rafael J. Wysocki wrote: > > > > Which is functionally equivalent to my patch, because > > i915_suspend/resume() won't be called by drm_class_suspend/resume() > > in the KMS case anyway. > > Ahh, right you are - that class suspend function does a check for > DRIVER_MODESET, and only does the suspend/resume if it's not a > MODESET driver. > > Ok, so I withdraw my objections to your original patch - it's > confusing, but that's just because DRM is such a horrible mess with > subtle things. Yeah the non-KMS paths just suck. Acked-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> Though hopefully you can get the PCI driver registration working w/o too much trouble; that would be even better. -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm