[PATCH 02/14] drm/i915/hdmi: convert g4x_hdmi.[ch] to struct intel_display
Jani Nikula
jani.nikula at intel.com
Thu Feb 13 09:15:09 UTC 2025
On Thu, 13 Feb 2025, "Kandpal, Suraj" <suraj.kandpal at intel.com> wrote:
>> -----Original Message-----
>> From: Intel-gfx <intel-gfx-bounces at lists.freedesktop.org> On Behalf Of Jani
>> Nikula
>> Sent: Wednesday, February 12, 2025 10:07 PM
>> To: intel-gfx at lists.freedesktop.org; intel-xe at lists.freedesktop.org
>> Cc: Nikula, Jani <jani.nikula at intel.com>
>> Subject: [PATCH 02/14] drm/i915/hdmi: convert g4x_hdmi.[ch] to struct
>> intel_display
>>
>> Going forward, struct intel_display is the main display device data pointer.
>> Convert as much as possible of g4x_hdmi.[ch] to struct intel_display.
>>
>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>> ---
>> drivers/gpu/drm/i915/display/g4x_hdmi.c | 139 +++++++++----------
>> drivers/gpu/drm/i915/display/g4x_hdmi.h | 6 +-
>> drivers/gpu/drm/i915/display/intel_display.c | 16 +--
>> 3 files changed, 79 insertions(+), 82 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/g4x_hdmi.c
>> b/drivers/gpu/drm/i915/display/g4x_hdmi.c
>> index 9e1ca7767392..6670cf101b9a 100644
>> --- a/drivers/gpu/drm/i915/display/g4x_hdmi.c
>> +++ b/drivers/gpu/drm/i915/display/g4x_hdmi.c
>> @@ -27,8 +27,8 @@
>> static void intel_hdmi_prepare(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state) {
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> + struct drm_i915_private *dev_priv = to_i915(display->drm);
>
> Nit: If we are changing having a change here why not rename it to i915 too.
> It's going to be useless in future since we want to remove drm_i915_private
> Usage altogether but in the meantime why not follow the i915 naming convention.
I'm keeping it dev_priv to avoid unrelated changes below. And as you
say, all of them will need to be removed anyway.
>
> Otherwise LGTM,
> Reviewed-by: Suraj Kandpal <suraj.kandpal at intel.com>
>
>> struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> const struct drm_display_mode *adjusted_mode = &crtc_state-
>> >hw.adjusted_mode; @@ -54,13 +54,13 @@ static void
>> intel_hdmi_prepare(struct intel_encoder *encoder,
>>
>> if (HAS_PCH_CPT(dev_priv))
>> hdmi_val |= SDVO_PIPE_SEL_CPT(crtc->pipe);
>> - else if (IS_CHERRYVIEW(dev_priv))
>> + else if (display->platform.cherryview)
>> hdmi_val |= SDVO_PIPE_SEL_CHV(crtc->pipe);
>> else
>> hdmi_val |= SDVO_PIPE_SEL(crtc->pipe);
>>
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, hdmi_val);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, hdmi_val);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> }
>>
>> static bool intel_hdmi_get_hw_state(struct intel_encoder *encoder, @@ -
>> 132,6 +132,7 @@ static int g4x_hdmi_compute_config(struct intel_encoder
>> *encoder,
>> struct intel_crtc_state *crtc_state,
>> struct drm_connector_state *conn_state) {
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_atomic_state *state = to_intel_atomic_state(crtc_state-
>> >uapi.state);
>> struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
>> struct drm_i915_private *i915 = to_i915(encoder->base.dev); @@ -
>> 142,7 +143,7 @@ static int g4x_hdmi_compute_config(struct intel_encoder
>> *encoder,
>> return -EINVAL;
>> }
>>
>> - if (IS_G4X(i915))
>> + if (display->platform.g4x)
>> crtc_state->has_hdmi_sink =
>> g4x_compute_has_hdmi_sink(state, crtc);
>> else
>> crtc_state->has_hdmi_sink =
>> @@ -154,15 +155,15 @@ static int g4x_hdmi_compute_config(struct
>> intel_encoder *encoder, static void intel_hdmi_get_config(struct
>> intel_encoder *encoder,
>> struct intel_crtc_state *pipe_config) {
>> + struct intel_display *display = to_intel_display(encoder);
>> + struct drm_i915_private *dev_priv = to_i915(display->drm);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> u32 tmp, flags = 0;
>> int dotclock;
>>
>> pipe_config->output_types |= BIT(INTEL_OUTPUT_HDMI);
>>
>> - tmp = intel_de_read(dev_priv, intel_hdmi->hdmi_reg);
>> + tmp = intel_de_read(display, intel_hdmi->hdmi_reg);
>>
>> if (tmp & SDVO_HSYNC_ACTIVE_HIGH)
>> flags |= DRM_MODE_FLAG_PHSYNC;
>> @@ -222,33 +223,32 @@ static void intel_hdmi_get_config(struct
>> intel_encoder *encoder, static void g4x_hdmi_enable_port(struct
>> intel_encoder *encoder,
>> const struct intel_crtc_state *pipe_config) {
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> u32 temp;
>>
>> - temp = intel_de_read(dev_priv, intel_hdmi->hdmi_reg);
>> + temp = intel_de_read(display, intel_hdmi->hdmi_reg);
>>
>> temp |= SDVO_ENABLE;
>>
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> }
>>
>> static void g4x_hdmi_audio_enable(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> const struct drm_connector_state
>> *conn_state) {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_hdmi *hdmi = enc_to_intel_hdmi(encoder);
>>
>> if (!crtc_state->has_audio)
>> return;
>>
>> - drm_WARN_ON(&i915->drm, !crtc_state->has_hdmi_sink);
>> + drm_WARN_ON(display->drm, !crtc_state->has_hdmi_sink);
>>
>> /* Enable audio presence detect */
>> - intel_de_rmw(i915, hdmi->hdmi_reg, 0, HDMI_AUDIO_ENABLE);
>> + intel_de_rmw(display, hdmi->hdmi_reg, 0, HDMI_AUDIO_ENABLE);
>>
>> intel_audio_codec_enable(encoder, crtc_state, conn_state); } @@ -
>> 257,7 +257,7 @@ static void g4x_hdmi_audio_disable(struct intel_encoder
>> *encoder,
>> const struct intel_crtc_state *old_crtc_state,
>> const struct drm_connector_state
>> *old_conn_state) {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_hdmi *hdmi = enc_to_intel_hdmi(encoder);
>>
>> if (!old_crtc_state->has_audio)
>> @@ -266,7 +266,7 @@ static void g4x_hdmi_audio_disable(struct
>> intel_encoder *encoder,
>> intel_audio_codec_disable(encoder, old_crtc_state, old_conn_state);
>>
>> /* Disable audio presence detect */
>> - intel_de_rmw(i915, hdmi->hdmi_reg, HDMI_AUDIO_ENABLE, 0);
>> + intel_de_rmw(display, hdmi->hdmi_reg, HDMI_AUDIO_ENABLE, 0);
>> }
>>
>> static void g4x_enable_hdmi(struct intel_atomic_state *state, @@ -282,12
>> +282,11 @@ static void ibx_enable_hdmi(struct intel_atomic_state *state,
>> const struct intel_crtc_state *pipe_config,
>> const struct drm_connector_state *conn_state) {
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> u32 temp;
>>
>> - temp = intel_de_read(dev_priv, intel_hdmi->hdmi_reg);
>> + temp = intel_de_read(display, intel_hdmi->hdmi_reg);
>>
>> temp |= SDVO_ENABLE;
>>
>> @@ -295,10 +294,10 @@ static void ibx_enable_hdmi(struct
>> intel_atomic_state *state,
>> * HW workaround, need to write this twice for issue
>> * that may result in first write getting masked.
>> */
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> /*
>> * HW workaround, need to toggle enable bit off and on @@ -
>> 309,18 +308,18 @@ static void ibx_enable_hdmi(struct intel_atomic_state
>> *state,
>> */
>> if (pipe_config->pipe_bpp > 24 &&
>> pipe_config->pixel_multiplier > 1) {
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg,
>> + intel_de_write(display, intel_hdmi->hdmi_reg,
>> temp & ~SDVO_ENABLE);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> /*
>> * HW workaround, need to write this twice for issue
>> * that may result in first write getting masked.
>> */
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> }
>> }
>>
>> @@ -329,14 +328,13 @@ static void cpt_enable_hdmi(struct
>> intel_atomic_state *state,
>> const struct intel_crtc_state *pipe_config,
>> const struct drm_connector_state *conn_state) {
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> enum pipe pipe = crtc->pipe;
>> u32 temp;
>>
>> - temp = intel_de_read(dev_priv, intel_hdmi->hdmi_reg);
>> + temp = intel_de_read(display, intel_hdmi->hdmi_reg);
>>
>> temp |= SDVO_ENABLE;
>>
>> @@ -351,24 +349,24 @@ static void cpt_enable_hdmi(struct
>> intel_atomic_state *state,
>> */
>>
>> if (pipe_config->pipe_bpp > 24) {
>> - intel_de_rmw(dev_priv, TRANS_CHICKEN1(pipe),
>> + intel_de_rmw(display, TRANS_CHICKEN1(pipe),
>> 0, TRANS_CHICKEN1_HDMIUNIT_GC_DISABLE);
>>
>> temp &= ~SDVO_COLOR_FORMAT_MASK;
>> temp |= SDVO_COLOR_FORMAT_8bpc;
>> }
>>
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> if (pipe_config->pipe_bpp > 24) {
>> temp &= ~SDVO_COLOR_FORMAT_MASK;
>> temp |= HDMI_COLOR_FORMAT_12bpc;
>>
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> - intel_de_rmw(dev_priv, TRANS_CHICKEN1(pipe),
>> + intel_de_rmw(display, TRANS_CHICKEN1(pipe),
>> TRANS_CHICKEN1_HDMIUNIT_GC_DISABLE, 0);
>> }
>> }
>> @@ -386,19 +384,18 @@ static void intel_disable_hdmi(struct
>> intel_atomic_state *state,
>> const struct drm_connector_state
>> *old_conn_state) {
>> struct intel_display *display = to_intel_display(encoder);
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct drm_i915_private *dev_priv = to_i915(display->drm);
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> struct intel_digital_port *dig_port =
>> hdmi_to_dig_port(intel_hdmi);
>> struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
>> u32 temp;
>>
>> - temp = intel_de_read(dev_priv, intel_hdmi->hdmi_reg);
>> + temp = intel_de_read(display, intel_hdmi->hdmi_reg);
>>
>> temp &= ~SDVO_ENABLE;
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> /*
>> * HW workaround for IBX, we need to move the port @@ -419,14
>> +416,14 @@ static void intel_disable_hdmi(struct intel_atomic_state *state,
>> * HW workaround, need to write this twice for issue
>> * that may result in first write getting masked.
>> */
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> temp &= ~SDVO_ENABLE;
>> - intel_de_write(dev_priv, intel_hdmi->hdmi_reg, temp);
>> - intel_de_posting_read(dev_priv, intel_hdmi->hdmi_reg);
>> + intel_de_write(display, intel_hdmi->hdmi_reg, temp);
>> + intel_de_posting_read(display, intel_hdmi->hdmi_reg);
>>
>> intel_wait_for_vblank_if_active(display, PIPE_A);
>> intel_set_cpu_fifo_underrun_reporting(dev_priv, PIPE_A,
>> true); @@ -544,8 +541,8 @@ static void chv_hdmi_post_disable(struct
>> intel_atomic_state *state,
>> const struct intel_crtc_state *old_crtc_state,
>> const struct drm_connector_state
>> *old_conn_state) {
>> - struct drm_device *dev = encoder->base.dev;
>> - struct drm_i915_private *dev_priv = to_i915(dev);
>> + struct intel_display *display = to_intel_display(encoder);
>> + struct drm_i915_private *dev_priv = to_i915(display->drm);
>>
>> vlv_dpio_get(dev_priv);
>>
>> @@ -614,7 +611,7 @@ intel_hdmi_hotplug(struct intel_encoder *encoder,
>> int g4x_hdmi_connector_atomic_check(struct drm_connector *connector,
>> struct drm_atomic_state *state) {
>> - struct drm_i915_private *i915 = to_i915(state->dev);
>> + struct intel_display *display = to_intel_display(connector->dev);
>> struct drm_connector_list_iter conn_iter;
>> struct drm_connector *conn;
>> int ret;
>> @@ -623,7 +620,7 @@ int g4x_hdmi_connector_atomic_check(struct
>> drm_connector *connector,
>> if (ret)
>> return ret;
>>
>> - if (!IS_G4X(i915))
>> + if (!display->platform.g4x)
>> return 0;
>>
>> if (!intel_connector_needs_modeset(to_intel_atomic_state(state),
>> connector)) @@ -637,7 +634,7 @@ int
>> g4x_hdmi_connector_atomic_check(struct drm_connector *connector,
>> *
>> * See also g4x_compute_has_hdmi_sink().
>> */
>> - drm_connector_list_iter_begin(&i915->drm, &conn_iter);
>> + drm_connector_list_iter_begin(display->drm, &conn_iter);
>> drm_for_each_connector_iter(conn, &conn_iter) {
>> struct drm_connector_state *conn_state;
>> struct drm_crtc_state *crtc_state;
>> @@ -646,7 +643,7 @@ int g4x_hdmi_connector_atomic_check(struct
>> drm_connector *connector,
>> if (!connector_is_hdmi(conn))
>> continue;
>>
>> - drm_dbg_kms(&i915->drm, "Adding
>> [CONNECTOR:%d:%s]\n",
>> + drm_dbg_kms(display->drm, "Adding
>> [CONNECTOR:%d:%s]\n",
>> conn->base.id, conn->name);
>>
>> conn_state = drm_atomic_get_connector_state(state, conn);
>> @@ -671,24 +668,24 @@ int g4x_hdmi_connector_atomic_check(struct
>> drm_connector *connector,
>> return ret;
>> }
>>
>> -static bool is_hdmi_port_valid(struct drm_i915_private *i915, enum port
>> port)
>> +static bool is_hdmi_port_valid(struct intel_display *display, enum port
>> +port)
>> {
>> - if (IS_G4X(i915) || IS_VALLEYVIEW(i915))
>> + if (display->platform.g4x || display->platform.valleyview)
>> return port == PORT_B || port == PORT_C;
>> else
>> return port == PORT_B || port == PORT_C || port ==
>> PORT_D; }
>>
>> -static bool assert_hdmi_port_valid(struct drm_i915_private *i915, enum
>> port port)
>> +static bool assert_hdmi_port_valid(struct intel_display *display, enum
>> +port port)
>> {
>> - return !drm_WARN(&i915->drm, !is_hdmi_port_valid(i915, port),
>> + return !drm_WARN(display->drm, !is_hdmi_port_valid(display,
>> port),
>> "Platform does not support HDMI %c\n",
>> port_name(port)); }
>>
>> -bool g4x_hdmi_init(struct drm_i915_private *dev_priv,
>> +bool g4x_hdmi_init(struct intel_display *display,
>> i915_reg_t hdmi_reg, enum port port) {
>> - struct intel_display *display = &dev_priv->display;
>> + struct drm_i915_private *dev_priv = to_i915(display->drm);
>> const struct intel_bios_encoder_data *devdata;
>> struct intel_digital_port *dig_port;
>> struct intel_encoder *intel_encoder;
>> @@ -697,14 +694,14 @@ bool g4x_hdmi_init(struct drm_i915_private
>> *dev_priv,
>> if (!assert_port_valid(dev_priv, port))
>> return false;
>>
>> - if (!assert_hdmi_port_valid(dev_priv, port))
>> + if (!assert_hdmi_port_valid(display, port))
>> return false;
>>
>> devdata = intel_bios_encoder_data_lookup(display, port);
>>
>> /* FIXME bail? */
>> if (!devdata)
>> - drm_dbg_kms(&dev_priv->drm, "No VBT child device for
>> HDMI-%c\n",
>> + drm_dbg_kms(display->drm, "No VBT child device for HDMI-
>> %c\n",
>> port_name(port));
>>
>> dig_port = kzalloc(sizeof(*dig_port), GFP_KERNEL); @@ -723,7
>> +720,7 @@ bool g4x_hdmi_init(struct drm_i915_private *dev_priv,
>>
>> mutex_init(&dig_port->hdcp_mutex);
>>
>> - if (drm_encoder_init(&dev_priv->drm, &intel_encoder->base,
>> + if (drm_encoder_init(display->drm, &intel_encoder->base,
>> &intel_hdmi_enc_funcs,
>> DRM_MODE_ENCODER_TMDS,
>> "HDMI %c", port_name(port)))
>> goto err_encoder_init;
>> @@ -738,13 +735,13 @@ bool g4x_hdmi_init(struct drm_i915_private
>> *dev_priv,
>> }
>> intel_encoder->get_hw_state = intel_hdmi_get_hw_state;
>> intel_encoder->get_config = intel_hdmi_get_config;
>> - if (IS_CHERRYVIEW(dev_priv)) {
>> + if (display->platform.cherryview) {
>> intel_encoder->pre_pll_enable = chv_hdmi_pre_pll_enable;
>> intel_encoder->pre_enable = chv_hdmi_pre_enable;
>> intel_encoder->enable = vlv_enable_hdmi;
>> intel_encoder->post_disable = chv_hdmi_post_disable;
>> intel_encoder->post_pll_disable =
>> chv_hdmi_post_pll_disable;
>> - } else if (IS_VALLEYVIEW(dev_priv)) {
>> + } else if (display->platform.valleyview) {
>> intel_encoder->pre_pll_enable = vlv_hdmi_pre_pll_enable;
>> intel_encoder->pre_enable = vlv_hdmi_pre_enable;
>> intel_encoder->enable = vlv_enable_hdmi; @@ -765,7
>> +762,7 @@ bool g4x_hdmi_init(struct drm_i915_private *dev_priv,
>> intel_encoder->type = INTEL_OUTPUT_HDMI;
>> intel_encoder->power_domain =
>> intel_display_power_ddi_lanes_domain(display, port);
>> intel_encoder->port = port;
>> - if (IS_CHERRYVIEW(dev_priv)) {
>> + if (display->platform.cherryview) {
>> if (port == PORT_D)
>> intel_encoder->pipe_mask = BIT(PIPE_C);
>> else
>> @@ -780,7 +777,7 @@ bool g4x_hdmi_init(struct drm_i915_private
>> *dev_priv,
>> * to work on real hardware. And since g4x can send infoframes to
>> * only one port anyway, nothing is lost by allowing it.
>> */
>> - if (IS_G4X(dev_priv))
>> + if (display->platform.g4x)
>> intel_encoder->cloneable |= BIT(INTEL_OUTPUT_HDMI);
>>
>> dig_port->hdmi.hdmi_reg = hdmi_reg;
>> diff --git a/drivers/gpu/drm/i915/display/g4x_hdmi.h
>> b/drivers/gpu/drm/i915/display/g4x_hdmi.h
>> index a52e8986ec7a..039d2bdba06c 100644
>> --- a/drivers/gpu/drm/i915/display/g4x_hdmi.h
>> +++ b/drivers/gpu/drm/i915/display/g4x_hdmi.h
>> @@ -13,15 +13,15 @@
>> enum port;
>> struct drm_atomic_state;
>> struct drm_connector;
>> -struct drm_i915_private;
>> +struct intel_display;
>>
>> #ifdef I915
>> -bool g4x_hdmi_init(struct drm_i915_private *dev_priv,
>> +bool g4x_hdmi_init(struct intel_display *display,
>> i915_reg_t hdmi_reg, enum port port); int
>> g4x_hdmi_connector_atomic_check(struct drm_connector *connector,
>> struct drm_atomic_state *state); #else -
>> static inline bool g4x_hdmi_init(struct drm_i915_private *dev_priv,
>> +static inline bool g4x_hdmi_init(struct intel_display *display,
>> i915_reg_t hdmi_reg, int port)
>> {
>> return false;
>> diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>> b/drivers/gpu/drm/i915/display/intel_display.c
>> index e5ceedf56335..b8c57a5d26a0 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display.c
>> +++ b/drivers/gpu/drm/i915/display/intel_display.c
>> @@ -8235,16 +8235,16 @@ void intel_setup_outputs(struct
>> drm_i915_private *dev_priv)
>> /* PCH SDVOB multiplex with HDMIB */
>> found = intel_sdvo_init(dev_priv, PCH_SDVOB,
>> PORT_B);
>> if (!found)
>> - g4x_hdmi_init(dev_priv, PCH_HDMIB,
>> PORT_B);
>> + g4x_hdmi_init(display, PCH_HDMIB,
>> PORT_B);
>> if (!found && (intel_de_read(dev_priv, PCH_DP_B) &
>> DP_DETECTED))
>> g4x_dp_init(display, PCH_DP_B, PORT_B);
>> }
>>
>> if (intel_de_read(dev_priv, PCH_HDMIC) & SDVO_DETECTED)
>> - g4x_hdmi_init(dev_priv, PCH_HDMIC, PORT_C);
>> + g4x_hdmi_init(display, PCH_HDMIC, PORT_C);
>>
>> if (!dpd_is_edp && intel_de_read(dev_priv, PCH_HDMID) &
>> SDVO_DETECTED)
>> - g4x_hdmi_init(dev_priv, PCH_HDMID, PORT_D);
>> + g4x_hdmi_init(display, PCH_HDMID, PORT_D);
>>
>> if (intel_de_read(dev_priv, PCH_DP_C) & DP_DETECTED)
>> g4x_dp_init(display, PCH_DP_C, PORT_C); @@ -
>> 8277,14 +8277,14 @@ void intel_setup_outputs(struct drm_i915_private
>> *dev_priv)
>> if (intel_de_read(dev_priv, VLV_DP_B) & DP_DETECTED ||
>> has_port)
>> has_edp &= g4x_dp_init(display, VLV_DP_B,
>> PORT_B);
>> if ((intel_de_read(dev_priv, VLV_HDMIB) & SDVO_DETECTED
>> || has_port) && !has_edp)
>> - g4x_hdmi_init(dev_priv, VLV_HDMIB, PORT_B);
>> + g4x_hdmi_init(display, VLV_HDMIB, PORT_B);
>>
>> has_edp = intel_dp_is_port_edp(display, PORT_C);
>> has_port = intel_bios_is_port_present(display, PORT_C);
>> if (intel_de_read(dev_priv, VLV_DP_C) & DP_DETECTED ||
>> has_port)
>> has_edp &= g4x_dp_init(display, VLV_DP_C,
>> PORT_C);
>> if ((intel_de_read(dev_priv, VLV_HDMIC) & SDVO_DETECTED
>> || has_port) && !has_edp)
>> - g4x_hdmi_init(dev_priv, VLV_HDMIC, PORT_C);
>> + g4x_hdmi_init(display, VLV_HDMIC, PORT_C);
>>
>> if (IS_CHERRYVIEW(dev_priv)) {
>> /*
>> @@ -8295,7 +8295,7 @@ void intel_setup_outputs(struct drm_i915_private
>> *dev_priv)
>> if (intel_de_read(dev_priv, CHV_DP_D) &
>> DP_DETECTED || has_port)
>> g4x_dp_init(display, CHV_DP_D, PORT_D);
>> if (intel_de_read(dev_priv, CHV_HDMID) &
>> SDVO_DETECTED || has_port)
>> - g4x_hdmi_init(dev_priv, CHV_HDMID,
>> PORT_D);
>> + g4x_hdmi_init(display, CHV_HDMID,
>> PORT_D);
>> }
>>
>> vlv_dsi_init(dev_priv);
>> @@ -8316,7 +8316,7 @@ void intel_setup_outputs(struct drm_i915_private
>> *dev_priv)
>> if (!found && IS_G4X(dev_priv)) {
>> drm_dbg_kms(&dev_priv->drm,
>> "probing HDMI on SDVOB\n");
>> - g4x_hdmi_init(dev_priv, GEN4_HDMIB,
>> PORT_B);
>> + g4x_hdmi_init(display, GEN4_HDMIB,
>> PORT_B);
>> }
>>
>> if (!found && IS_G4X(dev_priv))
>> @@ -8335,7 +8335,7 @@ void intel_setup_outputs(struct drm_i915_private
>> *dev_priv)
>> if (IS_G4X(dev_priv)) {
>> drm_dbg_kms(&dev_priv->drm,
>> "probing HDMI on SDVOC\n");
>> - g4x_hdmi_init(dev_priv, GEN4_HDMIC,
>> PORT_C);
>> + g4x_hdmi_init(display, GEN4_HDMIC,
>> PORT_C);
>> }
>> if (IS_G4X(dev_priv))
>> g4x_dp_init(display, DP_C, PORT_C);
>> --
>> 2.39.5
>
--
Jani Nikula, Intel
More information about the Intel-xe
mailing list