drm/i915/dpio: un-inline the vlv phy/channel mapping functions
authorJani Nikula <jani.nikula@intel.com>
Wed, 2 Nov 2022 10:08:13 +0000 (12:08 +0200)
committerJani Nikula <jani.nikula@intel.com>
Thu, 3 Nov 2022 14:20:53 +0000 (16:20 +0200)
Simplify the heavy intel_display_types.h header.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/943dd3e9812138b1cf3ddcfde896cfec006f3847.1667383630.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_display.c
drivers/gpu/drm/i915/display/intel_display_types.h
drivers/gpu/drm/i915/display/intel_dpio_phy.c
drivers/gpu/drm/i915/display/intel_dpio_phy.h
drivers/gpu/drm/i915/display/intel_dpll.c
drivers/gpu/drm/i915/display/intel_pps.c

index 0691b49..31e91db 100644 (file)
@@ -90,6 +90,7 @@
 #include "intel_display_types.h"
 #include "intel_dmc.h"
 #include "intel_dp_link_training.h"
+#include "intel_dpio_phy.h"
 #include "intel_dpt.h"
 #include "intel_dsb.h"
 #include "intel_fbc.h"
index 5b38937..6431427 100644 (file)
@@ -1803,51 +1803,6 @@ struct intel_dp_mst_encoder {
        struct intel_connector *connector;
 };
 
-static inline enum dpio_channel
-vlv_dig_port_to_channel(struct intel_digital_port *dig_port)
-{
-       switch (dig_port->base.port) {
-       default:
-               MISSING_CASE(dig_port->base.port);
-               fallthrough;
-       case PORT_B:
-       case PORT_D:
-               return DPIO_CH0;
-       case PORT_C:
-               return DPIO_CH1;
-       }
-}
-
-static inline enum dpio_phy
-vlv_dig_port_to_phy(struct intel_digital_port *dig_port)
-{
-       switch (dig_port->base.port) {
-       default:
-               MISSING_CASE(dig_port->base.port);
-               fallthrough;
-       case PORT_B:
-       case PORT_C:
-               return DPIO_PHY0;
-       case PORT_D:
-               return DPIO_PHY1;
-       }
-}
-
-static inline enum dpio_channel
-vlv_pipe_to_channel(enum pipe pipe)
-{
-       switch (pipe) {
-       default:
-               MISSING_CASE(pipe);
-               fallthrough;
-       case PIPE_A:
-       case PIPE_C:
-               return DPIO_CH0;
-       case PIPE_B:
-               return DPIO_CH1;
-       }
-}
-
 struct intel_load_detect_pipe {
        struct drm_atomic_state *restore_state;
 };
index 8732b87..6fc5b9e 100644 (file)
@@ -655,6 +655,48 @@ bxt_ddi_phy_get_lane_lat_optim_mask(struct intel_encoder *encoder)
        return mask;
 }
 
+enum dpio_channel vlv_dig_port_to_channel(struct intel_digital_port *dig_port)
+{
+       switch (dig_port->base.port) {
+       default:
+               MISSING_CASE(dig_port->base.port);
+               fallthrough;
+       case PORT_B:
+       case PORT_D:
+               return DPIO_CH0;
+       case PORT_C:
+               return DPIO_CH1;
+       }
+}
+
+enum dpio_phy vlv_dig_port_to_phy(struct intel_digital_port *dig_port)
+{
+       switch (dig_port->base.port) {
+       default:
+               MISSING_CASE(dig_port->base.port);
+               fallthrough;
+       case PORT_B:
+       case PORT_C:
+               return DPIO_PHY0;
+       case PORT_D:
+               return DPIO_PHY1;
+       }
+}
+
+enum dpio_channel vlv_pipe_to_channel(enum pipe pipe)
+{
+       switch (pipe) {
+       default:
+               MISSING_CASE(pipe);
+               fallthrough;
+       case PIPE_A:
+       case PIPE_C:
+               return DPIO_CH0;
+       case PIPE_B:
+               return DPIO_CH1;
+       }
+}
+
 void chv_set_phy_signal_level(struct intel_encoder *encoder,
                              const struct intel_crtc_state *crtc_state,
                              u32 deemph_reg_value, u32 margin_reg_value,
index 9c3d008..828ad58 100644 (file)
 
 enum dpio_channel;
 enum dpio_phy;
+enum pipe;
 enum port;
 struct drm_i915_private;
 struct intel_crtc_state;
+struct intel_digital_port;
 struct intel_encoder;
 
 void bxt_port_to_phy_channel(struct drm_i915_private *dev_priv, enum port port,
@@ -30,6 +32,10 @@ void bxt_ddi_phy_set_lane_optim_mask(struct intel_encoder *encoder,
                                     u8 lane_lat_optim_mask);
 u8 bxt_ddi_phy_get_lane_lat_optim_mask(struct intel_encoder *encoder);
 
+enum dpio_channel vlv_dig_port_to_channel(struct intel_digital_port *dig_port);
+enum dpio_phy vlv_dig_port_to_phy(struct intel_digital_port *dig_port);
+enum dpio_channel vlv_pipe_to_channel(enum pipe pipe);
+
 void chv_set_phy_signal_level(struct intel_encoder *encoder,
                              const struct intel_crtc_state *crtc_state,
                              u32 deemph_reg_value, u32 margin_reg_value,
index b15ba78..fdc6fa4 100644 (file)
@@ -10,6 +10,7 @@
 #include "intel_de.h"
 #include "intel_display.h"
 #include "intel_display_types.h"
+#include "intel_dpio_phy.h"
 #include "intel_dpll.h"
 #include "intel_lvds.h"
 #include "intel_panel.h"
index 21944f5..22f5e08 100644 (file)
@@ -9,6 +9,7 @@
 #include "intel_display_power_well.h"
 #include "intel_display_types.h"
 #include "intel_dp.h"
+#include "intel_dpio_phy.h"
 #include "intel_dpll.h"
 #include "intel_lvds.h"
 #include "intel_pps.h"