From: "Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>
To: Jani Nikula <jani.nikula@intel.com>, <intel-gfx@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 04/13] drm/i915/dp: stop caching has_hdmi_sink in struct intel_dp
Date: Mon, 29 May 2023 11:02:25 +0530 [thread overview]
Message-ID: <9db28b67-fe6a-0eee-dbc9-de7af886560b@intel.com> (raw)
In-Reply-To: <904309a6-2aa0-ce72-cb35-10447d7108ce@intel.com>
On 5/29/2023 10:52 AM, Nautiyal, Ankit K wrote:
> LGTM.
>
> Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
(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@intel.com>
>> ---
>> 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);
next prev parent reply other threads:[~2023-05-29 5:32 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-21 11:47 [Intel-gfx] [PATCH 00/13] drm/display & drm/i915: more struct drm_edid conversions Jani Nikula
2023-04-21 11:47 ` [Intel-gfx] [PATCH 01/13] drm/edid: parse display info has_audio similar to is_hdmi Jani Nikula
2023-05-29 4:56 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 02/13] drm/display/dp_mst: drop has_audio from struct drm_dp_mst_port Jani Nikula
2023-05-29 5:10 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 03/13] drm/i915/dp: stop caching has_audio in struct intel_dp Jani Nikula
2023-05-29 5:15 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 04/13] drm/i915/dp: stop caching has_hdmi_sink " Jani Nikula
2023-05-29 5:22 ` Nautiyal, Ankit K
2023-05-29 5:32 ` Nautiyal, Ankit K [this message]
2023-04-21 11:47 ` [Intel-gfx] [PATCH 05/13] drm/i915/hdmi: stop caching has_audio in struct intel_hdmi Jani Nikula
2023-05-29 5:43 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 06/13] drm/i915/hdmi: stop caching has_hdmi_sink " Jani Nikula
2023-05-29 5:49 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 07/13] drm/i915/sdvo: stop caching has_hdmi_monitor in struct intel_sdvo Jani Nikula
2023-05-29 6:05 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 08/13] drm/i915/sdvo: stop caching has_hdmi_audio " Jani Nikula
2023-05-29 6:06 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 09/13] drm/edid: add drm_edid_read_switcheroo() Jani Nikula
2023-05-30 4:09 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 10/13] drm/i915/lvds: switch to drm_edid_read_switcheroo() Jani Nikula
2023-04-23 8:10 ` Lukas Wunner
2023-04-21 11:47 ` [Intel-gfx] [PATCH 11/13] drm/edid: make drm_edid_duplicate() safe to call with NULL parameter Jani Nikula
2023-05-29 13:11 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 12/13] drm/display/dp_mst: convert to struct drm_edid Jani Nikula
2023-05-29 13:08 ` Nautiyal, Ankit K
2023-04-21 11:47 ` [Intel-gfx] [PATCH 13/13] drm/i915/display: switch the rest of the connectors " Jani Nikula
2023-05-30 4:38 ` Nautiyal, Ankit K
2023-04-21 14:11 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/display & drm/i915: more struct drm_edid conversions Patchwork
2023-04-21 14:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-04-21 22:34 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9db28b67-fe6a-0eee-dbc9-de7af886560b@intel.com \
--to=ankit.k.nautiyal@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox