On Mon, Sep 30, 2013 at 10:33 AM, Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote: > Anyway, I think you need to add per pipe save/restore to > i915_save_display/i915_restore_display too. I'm trying (very slowly at that) to move the register save/restore stuff out of there into more suitable places. Either we should now what should go in there (like all the w/a bits we set in e.g. init_clock_gating) or we need to have an appropriate place to store it more localized to all the other relevant code/data. People simply forget about save/restore_display when enabling new platforms or fixing up stuff since it's somewhere completely else. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx