drm/i915: Group all the PPS init steps to one place
authorImre Deak <imre.deak@intel.com>
Tue, 21 Jun 2016 08:51:49 +0000 (11:51 +0300)
committerImre Deak <imre.deak@intel.com>
Wed, 22 Jun 2016 13:17:02 +0000 (16:17 +0300)
commitb4d06ede4e301c4d11bbf8fd58ebe2aad2618769
treed68357612aca5cbc37a53310362fc183c54e1393
parent0080b5da392243af673e752b5681e6388ede7da0
drm/i915: Group all the PPS init steps to one place

Move the early PPS initialization calls next to the rest of PPS
initialization steps. This allows us to forgo a duplicated call to
intel_dp_init_panel_power_sequencer_registers() on VLV/CHV.

This will swap the order of DP AUX registration wrt. PPS initialization.
There is an existing race here in case of a user space access via the
DPAUX device node after DP AUX registration and before calling
intel_dp_init_panel_power_sequencer_registers(), but this change won't
make this worse. The fix for this is to separate DP AUX initialization
and registration, that's a separate work already underway.

The order of MST wrt. PPS init as well as the order of
intel_dp_init_panel_power_sequencer_registers() wrt.
intel_edp_panel_vdd_sanitize() also swap, which is ok, there are no
dependencies between these steps.

Suggested by Ville.

CC: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1466499109-20240-4-git-send-email-imre.deak@intel.com
drivers/gpu/drm/i915/intel_dp.c