From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 496D6FDEE43 for ; Thu, 23 Apr 2026 19:06:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References:Message-Id :MIME-Version:Subject:Date:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=J11ltQ65QCJQUXu4ts2pAYuqRHQx8HQP1jRvgq2IqIM=; b=DCdnz4I57mh0Fk 4o+HOiyUv+NTgUK1In+wLxCuudFd4/FNcUcRb0jscPX66Y4A/VLPkK+CmzhpoDWWCRZa4hI0ljJJi jRoPxsHwciHJG9d4Qj2jUrQ6rpxlIcpEYHnpXd7aLYBTfJhdPYW2X2DDhDlufKQm9I9K2exb/7v/a kuf9K+E1foDN2kARp3eJ9FAybosCaoFnO/OTREDRlqhUcKLU6n2o7LyL2C/QPFoMWr1BWoKY1/N52 iVW8wLgB4BWCXEhBezVHKXtSgi/xb2j4COGwbk3oYdAMhgfkPd2tTkZy1/4ejPTKOTipGdZ4qsJfy 8k//xsxV5oLJrfKPEpRw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1wFzNZ-0000000CAvc-2MxZ; Thu, 23 Apr 2026 19:06:25 +0000 Received: from sender4-pp-f112.zoho.com ([136.143.188.112]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1wFzNN-0000000CAm4-1Uwn; Thu, 23 Apr 2026 19:06:18 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1776971142; cv=none; d=zohomail.com; s=zohoarc; b=iSOv+imv3YijAewOqHkU+tgb9dxo2zSSyeVBPrff3DOo+B+DiB7MBZDtw3jhfohNDAcJ3eMJHYONnNYQO3Q095WOMFOI4QT70I3HNphUIkHBzBZriDt25jjYUwz0ENJBzK9zRhYbcMO5wooIsZMRkMWjMXHTuUD18d/USai5e+4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1776971142; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=F0vXM/B6KFsUDgrZyWACUzMDZAG93Nob0ErRDEc0QU0=; b=MzeutdWEsNYAWd57sqVpM4E5cTTdyiniSH+tuNSyTuq3ieTO4OTI8j9uPHjfZkaCc+74t1bu6Yq+n0pvoMDO26Xh+UochZK8ADNgcuMVKA8hDS0Bh+wpf++5ZTpAxPaqauKnmyV/HA6b8cHJW5Il+PgW3QEdLUdHAYII2HoDkMo= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1776971142; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:Date:Date:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Message-Id:References:In-Reply-To:To:To:Cc:Cc:Reply-To; bh=F0vXM/B6KFsUDgrZyWACUzMDZAG93Nob0ErRDEc0QU0=; b=g5YqYQ6dctBOg4dpGGgFBa9q36tuYdVwuz7xfC0gM76YKGYZcM8OHw95oZNIMoft T7KilTXwksb4WTcVGOUkA2xctRWMLq+JjVBKL36ZzmiSMshPq1TRJnhp1Iu0p61+3+L xKte+rHRUg1s7BZtoUbyO0pBJDYzsKHmoA2Yxnkg= Received: by mx.zohomail.com with SMTPS id 1776971139745181.58237546396435; Thu, 23 Apr 2026 12:05:39 -0700 (PDT) From: Nicolas Frattaroli Date: Thu, 23 Apr 2026 21:03:37 +0200 Subject: [PATCH v14 14/28] drm/i915/dp: Implement "color format" DRM property MIME-Version: 1.0 Message-Id: <20260423-color-format-v14-14-449a419ccbd4@collabora.com> References: <20260423-color-format-v14-0-449a419ccbd4@collabora.com> In-Reply-To: <20260423-color-format-v14-0-449a419ccbd4@collabora.com> To: Harry Wentland , Leo Li , Rodrigo Siqueira , Alex Deucher , =?utf-8?q?Christian_K=C3=B6nig?= , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Sandy Huang , =?utf-8?q?Heiko_St=C3=BCbner?= , Andy Yan , Jani Nikula , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , Dmitry Baryshkov , Sascha Hauer , Rob Herring , Jonathan Corbet , Shuah Khan Cc: kernel@collabora.com, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, linux-doc@vger.kernel.org, wayland-devel@lists.freedesktop.org, Nicolas Frattaroli X-Mailer: b4 0.15.2 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260423_120613_466990_EB9D59BF X-CRM114-Status: GOOD ( 13.68 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Implement the "color format" DRM property for DP. The values of the property include RGB, YCbCr420, YCbCr444 and Auto. Auto will pick RGB, with a fallback to YCbCr420. The mask of supported formats by the source exposed by the property is an optimistic scenario, as specific DFP-related caveats can't be established before an EDID is present. Should the explicitly requested color format not be supported by the sink (or by the source in combination with the sink), then an error is returned to userspace, so that it can make a better choice. Signed-off-by: Nicolas Frattaroli --- drivers/gpu/drm/i915/display/intel_dp.c | 61 ++++++++++++++++++++++++++++++--- 1 file changed, 57 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 299b91d262c3..9334ad9b12e9 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -3414,10 +3414,10 @@ intel_dp_compute_output_format(struct intel_encoder *encoder, } static int -intel_dp_compute_formats(struct intel_encoder *encoder, - struct intel_crtc_state *crtc_state, - struct drm_connector_state *conn_state, - bool respect_downstream_limits) +intel_dp_compute_formats_auto(struct intel_encoder *encoder, + struct intel_crtc_state *crtc_state, + struct drm_connector_state *conn_state, + bool respect_downstream_limits) { struct intel_display *display = to_intel_display(encoder); struct intel_dp *intel_dp = enc_to_intel_dp(encoder); @@ -3453,6 +3453,34 @@ intel_dp_compute_formats(struct intel_encoder *encoder, return ret; } +static int +intel_dp_compute_formats(struct intel_encoder *encoder, + struct intel_crtc_state *crtc_state, + struct drm_connector_state *conn_state, + bool respect_downstream_limits) +{ + switch (conn_state->color_format) { + case DRM_CONNECTOR_COLOR_FORMAT_RGB444: + return intel_dp_compute_output_format(encoder, crtc_state, conn_state, + respect_downstream_limits, + INTEL_OUTPUT_FORMAT_RGB); + case DRM_CONNECTOR_COLOR_FORMAT_YCBCR444: + return intel_dp_compute_output_format(encoder, crtc_state, conn_state, + respect_downstream_limits, + INTEL_OUTPUT_FORMAT_YCBCR444); + case DRM_CONNECTOR_COLOR_FORMAT_YCBCR420: + return intel_dp_compute_output_format(encoder, crtc_state, conn_state, + respect_downstream_limits, + INTEL_OUTPUT_FORMAT_YCBCR420); + case DRM_CONNECTOR_COLOR_FORMAT_AUTO: + return intel_dp_compute_formats_auto(encoder, crtc_state, conn_state, + respect_downstream_limits); + default: + MISSING_CASE(conn_state->color_format); + return -EINVAL; + } +} + void intel_dp_audio_compute_config(struct intel_encoder *encoder, struct intel_crtc_state *pipe_config, @@ -7009,6 +7037,29 @@ intel_dp_has_gamut_metadata_dip(struct intel_encoder *encoder) return false; } +static void +intel_dp_attach_color_format_property(struct intel_dp *intel_dp) +{ + struct intel_connector *connector = intel_dp->attached_connector; + struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp); + unsigned long fmts = BIT(DRM_OUTPUT_COLOR_FORMAT_RGB444); + bool has_pcon = false; + + if (!connector) + return; + + if (dig_port && intel_bios_encoder_is_lspcon(dig_port->base.devdata)) + has_pcon = true; + + if (has_pcon || source_can_output(intel_dp, INTEL_OUTPUT_FORMAT_YCBCR420)) + fmts |= BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR420); + + if (has_pcon || source_can_output(intel_dp, INTEL_OUTPUT_FORMAT_YCBCR444)) + fmts |= BIT(DRM_OUTPUT_COLOR_FORMAT_YCBCR444); + + drm_connector_attach_color_format_property(&connector->base, fmts); +} + static void intel_dp_add_properties(struct intel_dp *intel_dp, struct drm_connector *_connector) { @@ -7041,6 +7092,8 @@ intel_dp_add_properties(struct intel_dp *intel_dp, struct drm_connector *_connec if (HAS_VRR(display)) drm_connector_attach_vrr_capable_property(&connector->base); + + intel_dp_attach_color_format_property(intel_dp); } static void -- 2.53.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip