> From: Luca Coelho <[email protected]> > > It is irrelevant for the caller that the max lane count is being derived from > a FIA > register, so having "fia" in the function name is irrelevant. Rename the > function accordingly. > LGTM.
Reviewed-by: Suraj Kandpal <[email protected]> > Signed-off-by: Luca Coelho <[email protected]> > Reviewed-by: Lucas De Marchi <[email protected]> > Link: https://lore.kernel.org/r/20230721111121.369227-5- > [email protected] > --- > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 2 +- > drivers/gpu/drm/i915/display/intel_dp.c | 4 ++-- > drivers/gpu/drm/i915/display/intel_tc.c | 4 ++-- > drivers/gpu/drm/i915/display/intel_tc.h | 2 +- > 4 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c > index 26e256165b80..a5918bf30c31 100644 > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c > @@ -58,7 +58,7 @@ static u8 intel_cx0_get_owned_lane_mask(struct > drm_i915_private *i915, > * In DP-alt with pin assignment D, only PHY lane 0 is owned > * by display and lane 1 is owned by USB. > */ > - return intel_tc_port_fia_max_lane_count(dig_port) > 2 > + return intel_tc_port_max_lane_count(dig_port) > 2 > ? INTEL_CX0_BOTH_LANES : INTEL_CX0_LANE0; } > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > b/drivers/gpu/drm/i915/display/intel_dp.c > index 9f40da20e88d..84584864511b 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -306,13 +306,13 @@ static int > intel_dp_max_common_lane_count(struct intel_dp *intel_dp) > struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp); > int source_max = intel_dp_max_source_lane_count(dig_port); > int sink_max = intel_dp->max_sink_lane_count; > - int fia_max = intel_tc_port_fia_max_lane_count(dig_port); > + int port_max = intel_tc_port_max_lane_count(dig_port); > int lttpr_max = drm_dp_lttpr_max_lane_count(intel_dp- > >lttpr_common_caps); > > if (lttpr_max) > sink_max = min(sink_max, lttpr_max); > > - return min3(source_max, sink_max, fia_max); > + return min3(source_max, sink_max, port_max); > } > > int intel_dp_max_lane_count(struct intel_dp *intel_dp) diff --git > a/drivers/gpu/drm/i915/display/intel_tc.c > b/drivers/gpu/drm/i915/display/intel_tc.c > index 43b8eeba26f8..3c94bbcb5497 100644 > --- a/drivers/gpu/drm/i915/display/intel_tc.c > +++ b/drivers/gpu/drm/i915/display/intel_tc.c > @@ -337,7 +337,7 @@ static int intel_tc_port_get_max_lane_count(struct > intel_digital_port *dig_port) > } > } > > -int intel_tc_port_fia_max_lane_count(struct intel_digital_port *dig_port) > +int intel_tc_port_max_lane_count(struct intel_digital_port *dig_port) > { > struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev); > struct intel_tc_port *tc = to_tc_port(dig_port); @@ -589,7 +589,7 > @@ static bool tc_phy_verify_legacy_or_dp_alt_mode(struct intel_tc_port > *tc, > struct intel_digital_port *dig_port = tc->dig_port; > int max_lanes; > > - max_lanes = intel_tc_port_fia_max_lane_count(dig_port); > + max_lanes = intel_tc_port_max_lane_count(dig_port); > if (tc->mode == TC_PORT_LEGACY) { > drm_WARN_ON(&i915->drm, max_lanes != 4); > return true; > diff --git a/drivers/gpu/drm/i915/display/intel_tc.h > b/drivers/gpu/drm/i915/display/intel_tc.h > index ffc0e2a74e43..80a61e52850e 100644 > --- a/drivers/gpu/drm/i915/display/intel_tc.h > +++ b/drivers/gpu/drm/i915/display/intel_tc.h > @@ -20,7 +20,7 @@ bool intel_tc_port_connected(struct intel_encoder > *encoder); bool intel_tc_port_connected_locked(struct intel_encoder > *encoder); > > u32 intel_tc_port_get_pin_assignment_mask(struct intel_digital_port > *dig_port); -int intel_tc_port_fia_max_lane_count(struct intel_digital_port > *dig_port); > +int intel_tc_port_max_lane_count(struct intel_digital_port *dig_port); > void intel_tc_port_set_fia_lane_count(struct intel_digital_port *dig_port, > int required_lanes); > > -- > 2.40.1
