On Fri, Dec 20, 2019 at 07:29:54AM -0800, José Roberto de Souza wrote: > This function is only called from port sync and it is identical to > what will be executed again in intel_update_crtc() over port sync > pipes. > If it is really necessary it at least deserves a better name and a > comment, leaving it to people working on port sync.
IMO if we need special cases for port sync we should refactor the current code to accomodate that instead of copypasting the whole current implementation, as was done when port sync went it. But yeah, future work. Reviewed-by: Ville Syrjälä <[email protected]> > > Cc: Ville Syrjälä <[email protected]> > Cc: Maarten Lankhorst <[email protected]> > Cc: Manasi Navare <[email protected]> > Signed-off-by: José Roberto de Souza <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_display.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > b/drivers/gpu/drm/i915/display/intel_display.c > index 11f2c13ec23e..592c843f2f3b 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -14479,6 +14479,10 @@ static void intel_set_dp_tp_ctl_normal(struct > intel_crtc *crtc, > intel_dp_stop_link_train(intel_dp); > } > > +/* > + * TODO: This is only called from port sync and it is identical to what will > be > + * executed again in intel_update_crtc() over port sync pipes > + */ > static void intel_post_crtc_enable_updates(struct intel_crtc *crtc, > struct intel_atomic_state *state) > { > -- > 2.24.1 -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
