From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=185.246.84.56 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792687158; b=l2uDoEiwDo5bJHuShARvY2r6J2zBqEuz36BSJFcmokfdIbRwriqTbSAcMnTPOgGxpXWu Le/g5yvgE5uA1dLdKEQvSgUTHvxM3S8qQ6dfhtfUivyPmk46oIq2Ayq/Auomsc/Wx6RV8 ptaUiF4dBAmdYiqk4BTgdSvUPb9mT7C0YvpO0+h/iLgEknnoU1dtFnSs2KZ3DF6L1j+16 UyGNgDw/9nuvTYURpdULOdDDsnVVaQ6muRdN4L17PIJVGocHKav5VYMfT550+z1IhbTjX 8/BPPzi0nz7OTxn8IvStwCqfz2LXa1/7V789LArejM4obkkj2TIPdcuZmAurfCumKIg== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792687158; h=DMARC-Filter:Received:Received:Received:DKIM-Signature:From:Date: Subject:MIME-Version:Content-Type:Content-Transfer-Encoding: Message-Id:References:In-Reply-To:To:Cc:X-Mailer: X-Last-TLS-Session-Version; bh=0sHvVumUIf4TGnOLqOCT10I1Zve7awdJe3jnmpL+CjE=; b=1UtdzSToMn967pErH6RrMVGXGfBSRFKBG8JPZWBMJT3lsoySQE0n+fEccSevwngwCtFB waYsnNoaRvZzYBYUjzNAomivn0vsco5drfjbhL5iRDmFkDr6g0rp/vPt2Y0PKoKk4itmR WnjdP07QIYdMY2T2Ok30wq9gslxmRBTN/zbJaUw21NKBa+pd28rolYQm/KXjVxc/ESOd0 RUDOfVusbLs6Tjd/+ohZ2qlCbk6fh67XzgLnoyiJ93E20MmGmTVj2Z+dwXE1YerGylMqm Ak1QAbYe9cdapM5Uxoq6Gw68twnKPQ0Oi/nmYjV9DpCgyR8EXI3VEct2UapvuICBN+A== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass header.d=bootlin.com header.i=@bootlin.com header.a=rsa-sha256 header.s=dkim header.b=Z7YbT8oy; arc=none smtp.remote-ip=185.246.84.56 Received: from smtpout-02.galae.net (smtpout-02.galae.net [185.246.84.56]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E3715606A31B for ; Thu, 22 Oct 2026 16:39:16 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org E3715606A31B Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=bootlin.com header.i=@bootlin.com header.a=rsa-sha256 header.s=dkim header.b=Z7YbT8oy Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-02.galae.net (Postfix) with ESMTPS id A4EAC1A12C9; Thu, 22 Oct 2026 16:39:15 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 749ED60763; Thu, 22 Oct 2026 16:39:15 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 42F2B10328EC8; Thu, 22 Oct 2026 18:39:05 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1792687149; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=0sHvVumUIf4TGnOLqOCT10I1Zve7awdJe3jnmpL+CjE=; b=Z7YbT8oyxLmFeIR0OQTsxd9x5eU27ov/p74km/wKKCZRylODCsGLo1r1dSi/whd6bcoJ+S W2S4IBPSraaUmXfbcNuFa3aexfEE0ht5t1JJIBaqeobPzZfQl0i8eQs0hSOeu795eFFVqi +Q0F9CokJlo1sjwuji6yUINj7+nfkZVpIsowgndS5eWpx1AGpbI+4smYzQslK62sjruC9d A/xNqERZwvk8Kw2QlyQwsKiqlB1tfTOIk+V5bP3AcPdDxK+CXYDTv+nY4N18SuoEjJ5puq xSuqrZPo6EbnfNCxx3JbC2TghhulFIeFzZb58ph2GVrmvevYexc3/IePf+XNmQ== From: Kory Maincent Date: Thu, 22 Oct 2026 18:38:01 +0200 Subject: [PATCH v4 5/8] drm/i915/display/dp: Use dp_connector helpers to expose link training state X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20261022-feat_link_cap-v4-5-a4f3f6e8cad9@bootlin.com> References: <20261022-feat_link_cap-v4-0-a4f3f6e8cad9@bootlin.com> In-Reply-To: <20261022-feat_link_cap-v4-0-a4f3f6e8cad9@bootlin.com> To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Jani Nikula , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Luca Ceresoli , Chun-Kuang Hu , Philipp Zabel , Matthias Brugger , AngeloGioacchino Del Regno , Dmitry Baryshkov , Daniel Stone Cc: Thomas Petazzoni , Mark Yacoub , Sean Paul , Manasi Navare , Drew Davenport , Louis Chauvet , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Kory Maincent X-Mailer: b4 0.16.1-dev-17221 X-Last-TLS-Session-Version: TLSv1.3 Adopt the new DRM core DisplayPort helpers, which expose source and sink capabilities along with the maximum and current link parameters to userspace through the dp_link/ sysfs attributes. Keep the sink's supported link rates unclamped in the new dpcd_sink_rates/num_dpcd_sink_rates fields, rather than limiting them to the rates the source supports. Signed-off-by: Kory Maincent --- Changes in v3: - Move the call to new helpers around. - Use real sink rates instead of the modified ones. Changes in v2: - Remove voltage swing and pre emphasis properties. --- drivers/gpu/drm/i915/display/intel_display_types.h | 2 + drivers/gpu/drm/i915/display/intel_dp.c | 55 +++++++++++++++++++++- .../gpu/drm/i915/display/intel_dp_link_training.c | 14 ++++++ 3 files changed, 69 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h index 5f0fe18c0614e..025523ae58e6c 100644 --- a/drivers/gpu/drm/i915/display/intel_display_types.h +++ b/drivers/gpu/drm/i915/display/intel_display_types.h @@ -1845,6 +1845,8 @@ struct intel_dp { /* sink rates as reported by DP_MAX_LINK_RATE/DP_SUPPORTED_LINK_RATES */ int num_sink_rates; int sink_rates[DP_MAX_SUPPORTED_RATES]; + int num_dpcd_sink_rates; + int dpcd_sink_rates[DP_MAX_SUPPORTED_RATES]; bool use_rate_select; /* Max sink lane count as reported by DP_MAX_LANE_COUNT */ int max_sink_lane_count; diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index bbd76c0ff04b6..925d0ec939f55 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -167,6 +167,7 @@ static void intel_dp_set_default_sink_rates(struct intel_dp *intel_dp) { intel_dp->sink_rates[0] = 162000; intel_dp->num_sink_rates = 1; + intel_dp->num_dpcd_sink_rates = 0; } /* update sink rates from dpcd */ @@ -179,10 +180,14 @@ static void intel_dp_set_dpcd_sink_rates(struct intel_dp *intel_dp) intel_dp->lttpr_common_caps, &intel_dp->desc, intel_dp->tunnel, - intel_dp->sink_rates); + intel_dp->dpcd_sink_rates); if (num_sink_rates < 0) num_sink_rates = 0; + intel_dp->num_dpcd_sink_rates = num_sink_rates; + memcpy(intel_dp->sink_rates, intel_dp->dpcd_sink_rates, + sizeof(int) * num_sink_rates); + intel_dp->num_sink_rates = num_sink_rates; } @@ -4598,10 +4603,14 @@ intel_edp_set_sink_rates(struct intel_dp *intel_dp) num_sink_rates = drm_dp_read_edp_sink_rates(&intel_dp->aux, intel_dp->edp_dpcd[0], - intel_dp->sink_rates); + intel_dp->dpcd_sink_rates); if (num_sink_rates < 0) num_sink_rates = 0; + intel_dp->num_dpcd_sink_rates = num_sink_rates; + memcpy(intel_dp->sink_rates, intel_dp->dpcd_sink_rates, + sizeof(int) * num_sink_rates); + intel_dp->num_sink_rates = num_sink_rates; /* @@ -6141,6 +6150,20 @@ void intel_dp_dpcd_set_probe(struct intel_dp *intel_dp, bool force_on_external) intel_dp_needs_dpcd_probe(intel_dp, force_on_external)); } +static void intel_dp_set_sink_caps(struct intel_connector *connector, + struct intel_dp *intel_dp) +{ + struct drm_connector_dp_link_caps link_caps; + + link_caps.nlanes = drm_dp_get_max_lane_count(intel_dp->dpcd, + intel_dp->lttpr_common_caps); + link_caps.link_rates = intel_dp->dpcd_sink_rates; + link_caps.nlink_rates = intel_dp->num_dpcd_sink_rates; + link_caps.dsc = drm_dp_sink_supports_dsc(connector->dp.dsc_dpcd); + + drm_dp_sink_set_caps(&connector->base, &link_caps); +} + static int intel_dp_detect(struct drm_connector *_connector, struct drm_modeset_acquire_ctx *ctx, @@ -6212,6 +6235,8 @@ intel_dp_detect(struct drm_connector *_connector, intel_dp_tunnel_uhbr_lanes_wa_reset(intel_dp); + drm_dp_sink_reset_caps(_connector); + goto out_unset_edid; } @@ -6243,6 +6268,8 @@ intel_dp_detect(struct drm_connector *_connector, intel_dp_print_rates(intel_dp); + intel_dp_set_sink_caps(connector, intel_dp); + if (intel_dp->is_mst) { /* * If we are in MST mode then this connector @@ -6990,11 +7017,14 @@ intel_dp_init_connector(struct intel_digital_port *dig_port, struct intel_connector *connector) { struct intel_display *display = to_intel_display(dig_port); + struct drm_connector_dp_link_caps link_caps; struct intel_dp *intel_dp = &dig_port->dp; struct intel_encoder *encoder = &dig_port->base; struct drm_device *dev = encoder->base.dev; enum port port = encoder->port; + int *rates; int type; + int ret; if (drm_WARN(dev, dig_port->max_lanes < 1, "Not enough lanes (%d) for DP on [ENCODER:%d:%s]\n", @@ -7069,6 +7099,27 @@ intel_dp_init_connector(struct intel_digital_port *dig_port, intel_dp_set_common_link_params(intel_dp); intel_dp_reset_link_params(intel_dp); + rates = kmemdup_array(intel_dp->source_rates, + intel_dp->num_source_rates, + sizeof(*rates), + GFP_KERNEL); + if (!rates) { + intel_dp_aux_fini(intel_dp); + goto fail; + } + + link_caps.nlanes = dig_port->max_lanes; + link_caps.nlink_rates = intel_dp->num_source_rates; + link_caps.link_rates = rates; + link_caps.dsc = HAS_DSC(display); + + ret = drm_dp_source_set_caps(&connector->base, &link_caps); + if (ret) + drm_dbg_kms(display->drm, + "failed to set source link rates, skipping.\n"); + + kfree(rates); + /* init MST on ports that can support it */ intel_dp_mst_encoder_init(dig_port, connector->base.base.id); diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c b/drivers/gpu/drm/i915/display/intel_dp_link_training.c index cb92cff906146..c77cb124f1419 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c @@ -1758,6 +1758,18 @@ static void link_recovery_reset(struct intel_dp_link_training *link_training) link_training->recovery_state = INTEL_DP_LINK_RECOVERY_IDLE; } +static void intel_dp_report_link_train(struct intel_dp *intel_dp) +{ + struct intel_connector *connector = intel_dp->attached_connector; + + if (!connector) + return; + + drm_dp_set_cur_link_params(&connector->base, intel_dp->link_rate, + intel_dp->lane_count, + connector->dp.dsc_decompression_enabled); +} + /** * intel_dp_stop_link_train - stop link training * @intel_dp: DP struct @@ -1787,6 +1799,8 @@ void intel_dp_stop_link_train(struct intel_dp *intel_dp, intel_dp_program_link_training_pattern(intel_dp, crtc_state, DP_PHY_DPRX, DP_TRAINING_PATTERN_DISABLE); + intel_dp_report_link_train(intel_dp); + if (intel_dp_is_uhbr(crtc_state)) { ret = poll_timeout_us(ret = intel_dp_128b132b_intra_hop(intel_dp, crtc_state), ret == 0, -- 2.43.0