From nobody Thu Apr 2 05:48:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7530A3AD50F for ; Mon, 30 Mar 2026 08:33:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774859630; cv=pass; b=QctcxfUyUQZhuHiTek3i8rOfm32oteRs2PUrfA+ZYO0D5GnGTaqXE249//3cgboX7ihvQNuumY8L6qsYYqOW7HuwADc8MclCgAovK4edcsD5AvwPdBtqQ4TDiQFa3nUxdAQiihjmfjYHDmCUkYWfMNYtOM9Sy68GiluAYNEA7do= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774859630; c=relaxed/simple; bh=EiyQkV+/KFrtk6IsyaPeCXH6QSPrNQ/+an9jXMGJCHk=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=knnzRd0QdGP5vUQFUgnb0mF1p8nBmbJeIy6vudK/DOKZAObOn6P1SWwaev+o8wkDrF4UWCJ68YrQQ/lUvRJJOJR+RzCkS3HOy+mJzdcSdL0O/i/ZHRdRrUDDocXMyI7u1oEhDLbaXjKp3BDiGsWckvOBASAn6d42733fSnxbzLI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=OIAbij6L; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="OIAbij6L" ARC-Seal: i=1; a=rsa-sha256; t=1774859609; cv=none; d=zohomail.com; s=zohoarc; b=EhPX/2klAC77SnZYQ+t3cz/FzK0i3fqqlhrF/HNk1GXwOYn85aUVQaTeLFTN94RbtqLwgKBMkJWsXWt1v5rbk4QS61sOZ2MhZlCyaIwGU8MIAA4+YC8f/FbyTCqty9SjtX7/txXPfO2khURxFy9NJEQqAcje62BCQDtSTpIIu2c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1774859609; 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=EPu4ovEn8Oz48DgygDWvYKO+4spfzpRrt/6V788UXTc=; b=FPhYRHHDNTyHM7VewRKPIg6gaEq4YL4Amb2SOacS5umDlQouUU8MIo6izKZPMMbQ1BIikKfSgveg3ePhAWUV7HpBQEb7Se+oUvVnrq/ERwVFLr3dOPuJilh0vabFkGIUTMV/lLJeAr1B4uvt835wOm4qKiPdt+mhePrllSftmso= 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=1774859609; 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=EPu4ovEn8Oz48DgygDWvYKO+4spfzpRrt/6V788UXTc=; b=OIAbij6L17XisiEvb/dARdCwBnP0wpTtq0Jr0MOm4AY8Bh2RfkkOuV/dkr4hqnRQ NOBPYBxlcoGBzRXoM0o/YxKIBz1taLAwnXU5zQ88xJ1RwD2fuwbcy9N0P112Yukb0p1 sVDBI/tJTyhISYBtrvK2H3ozOjRVr0XV6lVAgZyk= Received: by mx.zohomail.com with SMTPS id 1774859608515839.6302977846477; Mon, 30 Mar 2026 01:33:28 -0700 (PDT) From: Nicolas Frattaroli Date: Mon, 30 Mar 2026 10:33:09 +0200 Subject: [PATCH v6 1/2] drm/connector: Add a 'link bpc' property Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260330-link-bpc-v6-1-ff124af93e48@collabora.com> References: <20260330-link-bpc-v6-0-ff124af93e48@collabora.com> In-Reply-To: <20260330-link-bpc-v6-0-ff124af93e48@collabora.com> To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , =?utf-8?q?Ville_Syrj=C3=A4l=C3=A4?= , Daniel Stone , Dmitry Baryshkov Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel@collabora.com, Nicolas Frattaroli , Derek Foreman , Marius Vlad X-Mailer: b4 0.15.1 Display drivers may degrade from higher bit depths to lower bit depths for reasons such as bandwidth constraints. Userspace applications, such as compositors, may wish to know that this has occurred transparently. Introduce a new immutable DRM property called "link bpc" that reflects the current display link's bits-per-component without considering a lower "on the wire" bit depth caused by display protocol features such as compression or dithering. An uevent is fired when the link bpc value changes on an atomic commit. Acked-by: Maxime Ripard Co-developed-by: Derek Foreman Signed-off-by: Derek Foreman Co-developed-by: Marius Vlad Signed-off-by: Marius Vlad Signed-off-by: Nicolas Frattaroli --- drivers/gpu/drm/drm_atomic_helper.c | 9 ++++ drivers/gpu/drm/drm_atomic_uapi.c | 2 + drivers/gpu/drm/drm_connector.c | 84 +++++++++++++++++++++++++++++++++= ++++ include/drm/drm_connector.h | 16 +++++++ 4 files changed, 111 insertions(+) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atom= ic_helper.c index 26953ed6b53e..4a932f543ac7 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c @@ -2033,9 +2033,11 @@ EXPORT_SYMBOL(drm_atomic_helper_commit_tail_rpm); =20 static void commit_tail(struct drm_atomic_state *state) { + struct drm_connector_state *old_conn_state, *new_conn_state; struct drm_device *dev =3D state->dev; const struct drm_mode_config_helper_funcs *funcs; struct drm_crtc_state *new_crtc_state; + struct drm_connector *connector; struct drm_crtc *crtc; ktime_t start; s64 commit_time_ms; @@ -2059,6 +2061,13 @@ static void commit_tail(struct drm_atomic_state *sta= te) =20 drm_atomic_helper_wait_for_dependencies(state); =20 + for_each_oldnew_connector_in_state(state, connector, old_conn_state, + new_conn_state, i) { + if (old_conn_state->link_bpc !=3D new_conn_state->link_bpc) + drm_connector_update_link_bpc_property(connector, + new_conn_state); + } + /* * We cannot safely access new_crtc_state after * drm_atomic_helper_commit_hw_done() so figure out which crtc's have diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic= _uapi.c index 5bd5bf6661df..12e7a5ab2299 100644 --- a/drivers/gpu/drm/drm_atomic_uapi.c +++ b/drivers/gpu/drm/drm_atomic_uapi.c @@ -1020,6 +1020,8 @@ drm_atomic_connector_get_property(struct drm_connecto= r *connector, *val =3D state->privacy_screen_sw_state; } else if (property =3D=3D connector->broadcast_rgb_property) { *val =3D state->hdmi.broadcast_rgb; + } else if (property =3D=3D connector->link_bpc_property) { + *val =3D state->link_bpc; } else if (connector->funcs->atomic_get_property) { return connector->funcs->atomic_get_property(connector, state, property, val); diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connecto= r.c index 47dc53c4a738..85557106218b 100644 --- a/drivers/gpu/drm/drm_connector.c +++ b/drivers/gpu/drm/drm_connector.c @@ -542,6 +542,75 @@ int drmm_connector_init(struct drm_device *dev, } EXPORT_SYMBOL(drmm_connector_init); =20 +/** + * drm_connector_attach_link_bpc_property - create and attach 'link bpc' p= roperty + * @connector: drm connector + * @max_bpc: specify the upper limit, matching that of 'max bpc' property + * + * Create and attach the 'link bpc' DRM property on @connector with an upp= er + * limit of @max_bpc. + * + * Returns: + * 0 on success, negative errno on failure. + */ +int +drm_connector_attach_link_bpc_property(struct drm_connector *connector, + unsigned int max_bpc) +{ + struct drm_device *dev =3D connector->dev; + struct drm_property *prop; + + if (connector->link_bpc_property) + return -EBUSY; + + if (max_bpc < 8 || max_bpc > U8_MAX) + return -EINVAL; + + prop =3D drm_property_create_range(dev, DRM_MODE_PROP_IMMUTABLE, + "link bpc", 8, max_bpc); + if (!prop) + return -ENOMEM; + + connector->link_bpc_property =3D prop; + + drm_object_attach_property(&connector->base, prop, max_bpc); + + return 0; +} +EXPORT_SYMBOL(drm_connector_attach_link_bpc_property); + +/** + * drm_connector_update_link_bpc_property - update the 'link bpc' property= of a + * connector and fire uevent + * @connector: pointer to the &struct drm_connector + * @state: pointer to the &struct drm_connector_state with the new value + * + * Update the 'link bpc' property of the given @connector to the + * &drm_connector_state.link_bpc member's value of @state and fire a ueven= t. + */ +void +drm_connector_update_link_bpc_property(struct drm_connector *connector, + struct drm_connector_state *state) +{ + u8 bpc =3D clamp(state->link_bpc, 8, state->max_bpc); + + if (!connector->link_bpc_property) + return; + + if (bpc !=3D state->link_bpc) + drm_dbg_kms(connector->dev, "[CONNECTOR:%d:%s] Clamping link bpc from %u= to %u\n", + connector->base.id, connector->name, state->link_bpc, bpc); + + drm_dbg_kms(connector->dev, "[CONNECTOR:%d:%s] Setting state link bpc %u\= n", + connector->base.id, connector->name, bpc); + drm_object_property_set_value(&connector->base, connector->link_bpc_prope= rty, + bpc); + + drm_sysfs_connector_property_event(connector, + connector->link_bpc_property); +} +EXPORT_SYMBOL(drm_connector_update_link_bpc_property); + /** * drmm_connector_hdmi_init - Init a preallocated HDMI connector * @dev: DRM device @@ -1713,6 +1782,21 @@ EXPORT_SYMBOL(drm_hdmi_connector_get_output_format_n= ame); * drm_connector_attach_max_bpc_property() to create and attach the * property to the connector during initialization. * + * link bpc: + * This immutable range property can be used by userspace to determine the + * current display link's bit depth. This bit depth is not reduced by + * display protocol features such as compression or dithering. Drivers can + * use drm_connector_attach_link_bpc_property() to create and attach the + * property to the connector during initialization. They can then set the + * &drm_connector_state.link_bpc member to the actual output bit depth + * after any degradation. The drm property will be updated to this member's + * value on the next atomic commit, and if it changed, a uevent will be + * fired. + * Userspace can listen to the uevent to be notified of link bpc changes, + * and compare the property's value to what userspace would've hoped for, + * so that userspace gets a view into bit depth degradation of its + * submitted framebuffers that may be happening on the output. + * * Connectors also have one standardized atomic property: * * CRTC_ID: diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h index af8b92d2d5b7..6ca5ebf7a6d3 100644 --- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -1154,6 +1154,11 @@ struct drm_connector_state { */ u8 max_bpc; =20 + /** + * @link_bpc: Current display link's bits-per-component. + */ + u8 link_bpc; + /** * @privacy_screen_sw_state: See :ref:`Standard Connector * Properties` @@ -2146,6 +2151,12 @@ struct drm_connector { */ struct drm_property *max_bpc_property; =20 + /** + * @link_bpc_property: Connector property that reflects the current + * output bits per component. + */ + struct drm_property *link_bpc_property; + /** @privacy_screen: drm_privacy_screen for this connector, or NULL. */ struct drm_privacy_screen *privacy_screen; =20 @@ -2556,6 +2567,11 @@ void drm_connector_attach_privacy_screen_provider( struct drm_connector *connector, struct drm_privacy_screen *priv); void drm_connector_update_privacy_screen(const struct drm_connector_state = *connector_state); =20 +int drm_connector_attach_link_bpc_property(struct drm_connector *connector, + unsigned int max_bpc); +void drm_connector_update_link_bpc_property(struct drm_connector *connecto= r, + struct drm_connector_state *state); + /** * struct drm_tile_group - Tile group metadata * @refcount: reference count --=20 2.53.0 From nobody Thu Apr 2 05:48:26 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2FBDD3AD512 for ; Mon, 30 Mar 2026 08:33:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774859632; cv=pass; b=kAfMvHuVTlsUWMEcfs2askFV/0nFjx2HpuWE1oUheUDCWZS9Qj+Hd99DqmFv8yVGSISSLJuOXe23pFwgsn1IwpJ0A8bOB1wRyFAc1IxDkYDE6XSm4eY7X7xyimEk2ARQr5s6yAF4mSEdArBIKYnHCkcfl7WK4poXOJQ7+EotvAE= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774859632; c=relaxed/simple; bh=WGL4WQF5S6UMJBmLUhu4ObLf7cWQPXQqiwpMExcq/3k=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=sBn1kyCFbxyLKVmxr2wyYUUtZRzXeC0hJ45SRuMSg98491pU156a8Md4GVy/Fb1dBf4Gi6xeAjfbKS7WNqJO/fVkWk+2gFZIshmwRNsy+bAFSUYMUrP9i4jt9urKndsKl1Wm9U2IOvU+p3WGbXBDenj+O8zybTP7RtuS13W1myk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=I847uVWR; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="I847uVWR" ARC-Seal: i=1; a=rsa-sha256; t=1774859613; cv=none; d=zohomail.com; s=zohoarc; b=G67CFC/fZxIhMnI1JBF8ku7caUmaH5gN6/1Yf8KuG9wUqIgpBfsIwOIfEG/wtqiJJUZZJ3sC2mEF7SJYMFGdl6yV3wN/l9vJFN5l/f+Ey0L6CbaE9tpHbGSx8MdEpu/gnaau/fP4ASDtbh/wHcVy3t1dOvHRpMn2cslcVDekv2E= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1774859613; 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=aU6Hkzh2ZGMxnXb53gyh1Js8KrUAKoWYrd8TXgFTFIA=; b=AttcPcpFotZM8uizTBoIE8OAswyDAehLe1MhMD9TSBhTHhd3DzfoxlhkQLXfEHQUnPepwIrz/zgYz6O5RSDaqIO4mCTAa2lRkqneWvwPj2hBoOwmNV06n/fITDOs0St3DKX8nhrfW/y5WBKqxDEhLFjR13uBbJ6GgdlPR6r4Su4= 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=1774859613; 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=aU6Hkzh2ZGMxnXb53gyh1Js8KrUAKoWYrd8TXgFTFIA=; b=I847uVWRN8mccEOB+4RvXfePx2feCM3d3wM1JsYcvLw5ZKPeOG8Ltbxg+3ojXKCN +ZrHfRHWT7/S8jVbgmhFfMd1ZoicxSAvWNmZsOsUMkttsEFuoOQl8/CL06EOT3K5qP9 lhO+YhgDXJjbr2mRRsg577oiGsBqP4t0Zqic0rGY= Received: by mx.zohomail.com with SMTPS id 1774859611665808.5760822952676; Mon, 30 Mar 2026 01:33:31 -0700 (PDT) From: Nicolas Frattaroli Date: Mon, 30 Mar 2026 10:33:10 +0200 Subject: [PATCH v6 2/2] drm/connector: hdmi: Add support for 'link bpc' property Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260330-link-bpc-v6-2-ff124af93e48@collabora.com> References: <20260330-link-bpc-v6-0-ff124af93e48@collabora.com> In-Reply-To: <20260330-link-bpc-v6-0-ff124af93e48@collabora.com> To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , =?utf-8?q?Ville_Syrj=C3=A4l=C3=A4?= , Daniel Stone , Dmitry Baryshkov Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel@collabora.com, Nicolas Frattaroli X-Mailer: b4 0.15.1 Create and attach the 'link bpc' DRM property for every HDMI connector that's created through drmm_connector_hdmi_init. Then, set the connector state's link_bpc member in the HDMI atomic check state helper to the connector state's HDMI output bpc. Acked-by: Maxime Ripard Signed-off-by: Nicolas Frattaroli --- drivers/gpu/drm/display/drm_hdmi_state_helper.c | 2 ++ drivers/gpu/drm/drm_connector.c | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/drivers/gpu/drm/display/drm_hdmi_state_helper.c b/drivers/gpu/= drm/display/drm_hdmi_state_helper.c index 9f3b696aceeb..b670bcdbd3c6 100644 --- a/drivers/gpu/drm/display/drm_hdmi_state_helper.c +++ b/drivers/gpu/drm/display/drm_hdmi_state_helper.c @@ -887,6 +887,8 @@ int drm_atomic_helper_connector_hdmi_check(struct drm_c= onnector *connector, struct drm_crtc *crtc =3D new_conn_state->crtc; struct drm_crtc_state *crtc_state; =20 + new_conn_state->link_bpc =3D new_conn_state->hdmi.output_bpc; + crtc_state =3D drm_atomic_get_crtc_state(state, crtc); if (IS_ERR(crtc_state)) return PTR_ERR(crtc_state); diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connecto= r.c index 85557106218b..2a2e56bf062d 100644 --- a/drivers/gpu/drm/drm_connector.c +++ b/drivers/gpu/drm/drm_connector.c @@ -693,6 +693,10 @@ int drmm_connector_hdmi_init(struct drm_device *dev, drm_connector_attach_max_bpc_property(connector, 8, max_bpc); connector->max_bpc =3D max_bpc; =20 + ret =3D drm_connector_attach_link_bpc_property(connector, max_bpc); + if (ret) + return ret; + if (max_bpc > 8) drm_connector_attach_hdr_output_metadata_property(connector); =20 --=20 2.53.0