Re: [Patch 1/5]: DRM/I915: Add the crt save/restore function for VT switch

[Eric A. <er...@an...>]

On Tue, 2009-03-03 at 18:05 +0800, yakui_zhao wrote:
> Subject: Patch DRM/i915: Add the crt save/restore function for VT switch
> From: Zhao Yakui <yak...@in...>
> 
> Add the crt save/restore function for VT switch
> 
> Signed-off-by: Zhao Yakui <yak...@in...>

Given that these functions aren't actually hooked up to anything, I'm
not sure if there's any value here.  It sounds like we're going to go to
using the resume_force_mode function for suspend/resume, which means
we'd never use these.

jbarnes: do you like the resume_force_mode plan, or do we want to try to
fix the native suspend/resume stuff?  Should resume_force_mode be safe?
(does it turn off things as necessary for reprogramming if they're
currently on at resume time?)

airlied: do you like the resume_force_mode plan, or do you want to hook
up the save/restore functions so you have an easier time at that hack
you were talking about for user-mode backporting last week?

-- 
Eric Anholt
er...@an...                         eri...@in...