On Tue, Oct 19, 2021 at 05:46:33AM +1000, David Airlie wrote: > On Tue, Oct 19, 2021 at 1:35 AM Ville Syrjala > <[email protected]> wrote: > > > > From: Ville Syrjälä <[email protected]> > > > > Nuke the hsw_get_ddi_port_state() eyesore by putting the > > readout code into intel_pch_display.c, and calling it directly > > from hsw_crt_get_config(). > > > > Note that the nuked TRANS_DDI_FUNC_CTL readout from > > hsw_get_ddi_port_state() is now etirely redundant since we > > get called from the encoder->get_config() so we already know > > we're dealing with the correct DDI port. Previously the > > code was called from a place where that wasn't known so > > it had to checked manually. > > > > v2: Clarify the TRANS_DDI_FUNC_CTL change (Dave) > > Nuke the now unused *TRANS_DDI_FUNC_CTL_VAL_TO_PORT() (Dave) > > > > Cc: Dave Airlie <[email protected]> > > Cc: Jani Nikula <[email protected]> > > Signed-off-by: Ville Syrjälä <[email protected]> > > Reviewed-by: Dave Airlie <[email protected]>
Thanks for the review. Fixed up all the (known) typos and robot noises, and pushed to drm-intel-next. -- Ville Syrjälä Intel
