]> www.infradead.org Git - users/willy/xarray.git/commitdiff
drm/i915/tc: Pass intel_tc_port to internal lane mask/count helpers
authorImre Deak <imre.deak@intel.com>
Tue, 5 Aug 2025 07:36:57 +0000 (10:36 +0300)
committerImre Deak <imre.deak@intel.com>
Wed, 13 Aug 2025 12:03:22 +0000 (15:03 +0300)
Pass the intel_tc_port pointer instead of intel_digital_port to all lane
mask and count query helpers internal to intel_tc.c, to avoid the
redundant intel_digital_port -> intel_tc_port conversions.

While at it shorten the function names, keeping the intel_tc_port_
prefix only for exported functions and use the mtl_, icl_ prefixes
making it clear which platforms a given query function is specific for.

Reviewed-by: Mika Kahola <mika.kahola@intel.com>
Link: https://lore.kernel.org/r/20250805073700.642107-17-imre.deak@intel.com
Signed-off-by: Imre Deak <imre.deak@intel.com>
drivers/gpu/drm/i915/display/intel_tc.c

index 169eb61e22ea43b394a22cb28d1bfe7e00d020c6..073c434fd37870c140cca63caa557a2db475880a 100644 (file)
@@ -260,10 +260,9 @@ assert_tc_port_power_enabled(struct intel_tc_port *tc)
                    !intel_display_power_is_enabled(display, tc_port_power_domain(tc)));
 }
 
-static u32 intel_tc_port_get_lane_mask(struct intel_digital_port *dig_port)
+static u32 get_lane_mask(struct intel_tc_port *tc)
 {
-       struct intel_display *display = to_intel_display(dig_port);
-       struct intel_tc_port *tc = to_tc_port(dig_port);
+       struct intel_display *display = to_intel_display(tc->dig_port);
        intel_wakeref_t wakeref;
        u32 lane_mask;
 
@@ -322,9 +321,8 @@ get_pin_assignment(struct intel_tc_port *tc)
        return pin_assignment;
 }
 
-static int mtl_tc_port_get_max_lane_count(struct intel_digital_port *dig_port)
+static int mtl_get_max_lane_count(struct intel_tc_port *tc)
 {
-       struct intel_tc_port *tc = to_tc_port(dig_port);
        enum intel_tc_pin_assignment pin_assignment;
 
        pin_assignment = get_pin_assignment(tc);
@@ -343,11 +341,11 @@ static int mtl_tc_port_get_max_lane_count(struct intel_digital_port *dig_port)
        }
 }
 
-static int intel_tc_port_get_max_lane_count(struct intel_digital_port *dig_port)
+static int icl_get_max_lane_count(struct intel_tc_port *tc)
 {
        u32 lane_mask = 0;
 
-       lane_mask = intel_tc_port_get_lane_mask(dig_port);
+       lane_mask = get_lane_mask(tc);
 
        switch (lane_mask) {
        default:
@@ -369,15 +367,14 @@ static int intel_tc_port_get_max_lane_count(struct intel_digital_port *dig_port)
 static int get_max_lane_count(struct intel_tc_port *tc)
 {
        struct intel_display *display = to_intel_display(tc->dig_port);
-       struct intel_digital_port *dig_port = tc->dig_port;
 
        if (tc->mode != TC_PORT_DP_ALT)
                return 4;
 
        if (DISPLAY_VER(display) >= 14)
-               return mtl_tc_port_get_max_lane_count(dig_port);
+               return mtl_get_max_lane_count(tc);
 
-       return intel_tc_port_get_max_lane_count(dig_port);
+       return icl_get_max_lane_count(tc);
 }
 
 static void read_pin_configuration(struct intel_tc_port *tc)