Re: [PATCH 04/13] drm/i915/dp: stop caching has_hdmi_sink in struct intel_dp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 5/29/2023 10:52 AM, Nautiyal, Ankit K wrote:
LGTM.

Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx>

(This patch would require a rebas.e)

Regards,

Ankit


On 4/21/2023 5:17 PM, Jani Nikula wrote:
Use the information stored in display info. Add intel_dp_has_hdmi_sink()
helper to access it.

Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
---
  drivers/gpu/drm/i915/display/intel_ddi.c      |  6 +++---
  .../drm/i915/display/intel_display_types.h    |  1 -
  drivers/gpu/drm/i915/display/intel_dp.c       | 21 ++++++++++---------
  drivers/gpu/drm/i915/display/intel_dp.h       |  1 +
  4 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 29e4bfab4635..9b462dd8bb14 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -2800,7 +2800,7 @@ static void intel_ddi_pre_enable(struct intel_atomic_state *state,
            /* FIXME precompute everything properly */
          /* FIXME how do we turn infoframes off again? */
-        if (dig_port->lspcon.active && dig_port->dp.has_hdmi_sink)
+        if (dig_port->lspcon.active && intel_dp_has_hdmi_sink(&dig_port->dp))
              dig_port->set_infoframes(encoder,
                           crtc_state->has_infoframe,
                           crtc_state, conn_state);
@@ -3109,7 +3109,7 @@ static void intel_enable_ddi_dp(struct intel_atomic_state *state,
      drm_connector_update_privacy_screen(conn_state);
      intel_edp_backlight_on(crtc_state, conn_state);
  -    if (!dig_port->lspcon.active || dig_port->dp.has_hdmi_sink)
+    if (!dig_port->lspcon.active || intel_dp_has_hdmi_sink(&dig_port->dp))           intel_dp_set_infoframes(encoder, true, crtc_state, conn_state);
        intel_audio_codec_enable(encoder, crtc_state, conn_state);
@@ -3736,7 +3736,7 @@ static void intel_ddi_read_func_ctl(struct intel_encoder *encoder,
                      pipe_config->fec_enable);
          }
  -        if (dig_port->lspcon.active && dig_port->dp.has_hdmi_sink)
+        if (dig_port->lspcon.active && intel_dp_has_hdmi_sink(&dig_port->dp))
              pipe_config->infoframes.enable |=
                  intel_lspcon_infoframes_enabled(encoder, pipe_config);
          else
diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h
index 9c9b771cbcbd..17681a8e745c 100644
--- a/drivers/gpu/drm/i915/display/intel_display_types.h
+++ b/drivers/gpu/drm/i915/display/intel_display_types.h
@@ -1669,7 +1669,6 @@ struct intel_dp {
      u8 lane_count;
      u8 sink_count;
      bool link_trained;
-    bool has_hdmi_sink;
      bool reset_link_params;
      bool use_max_params;
      u8 dpcd[DP_RECEIVER_CAP_SIZE];
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index cda69861b00a..1708277547f3 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -1201,6 +1201,13 @@ void intel_dp_compute_rate(struct intel_dp *intel_dp, int port_clock,
      }
  }
  +bool intel_dp_has_hdmi_sink(struct intel_dp *intel_dp)
+{
+    struct intel_connector *connector = intel_dp->attached_connector;
+
+    return connector->base.display_info.is_hdmi;
+}
+
  static bool intel_dp_source_supports_fec(struct intel_dp *intel_dp,
                       const struct intel_crtc_state *pipe_config)
  {
@@ -1265,7 +1272,7 @@ static int intel_dp_hdmi_compute_bpc(struct intel_dp *intel_dp,
        for (; bpc >= 8; bpc -= 2) {
          if (intel_hdmi_bpc_possible(crtc_state, bpc,
-                        intel_dp->has_hdmi_sink, ycbcr420_output) &&
+                        intel_dp_has_hdmi_sink(intel_dp), ycbcr420_output) &&               intel_dp_tmds_clock_valid(intel_dp, clock, bpc, ycbcr420_output,
                            respect_downstream_limits) == MODE_OK)
              return bpc;
@@ -2646,7 +2653,7 @@ static int intel_dp_pcon_start_frl_training(struct intel_dp *intel_dp)
  static bool intel_dp_is_hdmi_2_1_sink(struct intel_dp *intel_dp)
  {
      if (drm_dp_is_branch(intel_dp->dpcd) &&
-        intel_dp->has_hdmi_sink &&
+        intel_dp_has_hdmi_sink(intel_dp) &&
          intel_dp_hdmi_sink_max_frl(intel_dp) > 0)
          return true;
  @@ -2812,13 +2819,12 @@ void intel_dp_configure_protocol_converter(struct intel_dp *intel_dp,
      if (!drm_dp_is_branch(intel_dp->dpcd))
          return;
  -    tmp = intel_dp->has_hdmi_sink ?
-        DP_HDMI_DVI_OUTPUT_CONFIG : 0;
+    tmp = intel_dp_has_hdmi_sink(intel_dp) ? DP_HDMI_DVI_OUTPUT_CONFIG : 0;
        if (drm_dp_dpcd_writeb(&intel_dp->aux,
                     DP_PROTOCOL_CONVERTER_CONTROL_0, tmp) != 1)
          drm_dbg_kms(&i915->drm, "Failed to %s protocol converter HDMI mode\n",
- str_enable_disable(intel_dp->has_hdmi_sink));
+ str_enable_disable(intel_dp_has_hdmi_sink(intel_dp)));
        tmp = crtc_state->output_format == INTEL_OUTPUT_FORMAT_YCBCR444 &&           intel_dp->dfp.ycbcr_444_to_420 ? DP_CONVERSION_TO_YCBCR420_ENABLE : 0;
@@ -4711,9 +4717,6 @@ intel_dp_set_edid(struct intel_dp *intel_dp)
        /* FIXME: Get rid of drm_edid_raw() */
      edid = drm_edid_raw(drm_edid);
-    if (edid && edid->input & DRM_EDID_INPUT_DIGITAL) {
-        intel_dp->has_hdmi_sink = drm_detect_hdmi_monitor(edid);
-    }
        drm_dp_cec_set_edid(&intel_dp->aux, edid);
  }
@@ -4727,8 +4730,6 @@ intel_dp_unset_edid(struct intel_dp *intel_dp)
      drm_edid_free(connector->detect_edid);
      connector->detect_edid = NULL;
  -    intel_dp->has_hdmi_sink = false;
-
      intel_dp->dfp.max_bpc = 0;
      intel_dp->dfp.max_dotclock = 0;
      intel_dp->dfp.min_tmds_clock = 0;
diff --git a/drivers/gpu/drm/i915/display/intel_dp.h b/drivers/gpu/drm/i915/display/intel_dp.h
index ef39e4f7a329..803b0822c661 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.h
+++ b/drivers/gpu/drm/i915/display/intel_dp.h
@@ -62,6 +62,7 @@ int intel_dp_dsc_compute_config(struct intel_dp *intel_dp,
                  struct link_config_limits *limits,
                  int timeslots,
                  bool recompute_pipe_bpp);
+bool intel_dp_has_hdmi_sink(struct intel_dp *intel_dp);
  bool intel_dp_is_edp(struct intel_dp *intel_dp);
  bool intel_dp_is_uhbr(const struct intel_crtc_state *crtc_state);
  bool intel_dp_is_port_edp(struct drm_i915_private *dev_priv, enum port port);



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux