On Tue, Feb 03, 2015 at 01:06:31AM +0200, Imre Deak wrote: > > +static struct i915_power_well skl_power_wells[] = { > > + { > > + .name = "always-on", > > + .always_on = 1, > > + .domains = SKL_DISPLAY_ALWAYS_ON_POWER_DOMAINS, > > + .ops = &i9xx_always_on_power_well_ops, > > + }, > > + { > > + .name = "power well 1", > > + .domains = SKL_DISPLAY_POWERWELL_1_POWER_DOMAINS, > > + .ops = &skl_power_well_ops, > > + .data = SKL_DISP_PW_1, > > + }, snip > > + { > > + .name = "MISC IO power well", > > + .domains = SKL_DISPLAY_MISC_IO_POWER_DOMAINS, > > + .ops = &skl_power_well_ops, > > + .data = SKL_DISP_PW_MISC_IO, > > + } > > Again, since the recent bspec change the misc IO power well should be > enabled before anything else, so it needs to be listed before "power > well 1" on the list. So this one was causing problems. When I try to enabled MISC IO before PW1, the request times out. Enabling MISC IO just right after PW1 seems to work fine though. -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx