On Thu, Jul 11, 2013 at 06:45:01PM -0300, Rodrigo Vivi wrote: > @@ -1602,6 +1611,26 @@ void intel_edp_psr_disable(struct intel_dp *intel_dp) > DRM_ERROR("Timed out waiting for PSR Idle State\n"); > } > > +void intel_edp_psr_update(struct drm_device *dev) > +{ > + struct intel_encoder *encoder; > + struct intel_dp *intel_dp = NULL; > + > + list_for_each_entry(encoder, &dev->mode_config.encoder_list, base.head) > + if (encoder->type == INTEL_OUTPUT_EDP) { How many eDP are you planning to allow on the system? We already have precedence for the presumption of a single (integrated) panel on a device, maybe we can add the logic there (i.e. stash a back pointer in this case)? We could then also do a debugfs/i915_panel_info where we can inspect various details normally associated with the panels (PSR status, backlight interface, eDP vs LVDS, etc). -Chris -- Chris Wilson, Intel Open Source Technology Centre