On Thu, Nov 07, 2024 at 10:32:14PM +0200, Jani Nikula wrote: > Pass the primary encoder to the primary encoder hooks. This is > pedantically correct, but intel_ddi_post_pll_disable() also works with > the fake encoder by coincidence. > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c > index 13449c85162d..9670a48586ea 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c > +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c > @@ -1113,7 +1113,7 @@ static void intel_mst_post_pll_disable_dp(struct intel_atomic_state *state, > > if (intel_dp->active_mst_links == 0 && > dig_port->base.post_pll_disable) > - dig_port->base.post_pll_disable(state, encoder, old_crtc_state, old_conn_state); > + dig_port->base.post_pll_disable(state, &dig_port->base, old_crtc_state, old_conn_state); Yes, this worked only due to enc_to_dig_port() etc., but at the least this should match the way all the other hooks for primary are called (which pass the primary encoder): Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> > } > > static void intel_mst_pre_pll_enable_dp(struct intel_atomic_state *state, > -- > 2.39.5 >
