From: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
To: Xaver Hugl <xaver.hugl@kde.org>
Cc: "Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"David Airlie" <airlied@gmail.com>,
"Simona Vetter" <simona@ffwll.ch>,
"Harry Wentland" <harry.wentland@amd.com>,
"Leo Li" <sunpeng.li@amd.com>,
"Rodrigo Siqueira" <siqueira@igalia.com>,
"Alex Deucher" <alexander.deucher@amd.com>,
"Christian König" <christian.koenig@amd.com>,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Daniel Stone" <daniels@collabora.com>,
"Dmitry Baryshkov" <dmitry.baryshkov@oss.qualcomm.com>,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
amd-gfx@lists.freedesktop.org, kernel@collabora.com,
"Derek Foreman" <derek.foreman@collabora.com>,
"Marius Vlad" <marius.vlad@collabora.com>,
wayland-devel@lists.freedesktop.org
Subject: Re: [PATCH v5 0/3] Add "link bpc" DRM property
Date: Wed, 01 Apr 2026 14:14:40 +0200 [thread overview]
Message-ID: <4970515.ElGaqSPkdT@workhorse> (raw)
In-Reply-To: <CAFZQkGzxfGP4E18owJf5d6L4n25-nvYn0TG=rTLCzjpj1Hk23w@mail.gmail.com>
On Wednesday, 1 April 2026 13:57:08 Central European Summer Time Xaver Hugl wrote:
> Am Do., 19. März 2026 um 13:28 Uhr schrieb Nicolas Frattaroli
> <nicolas.frattaroli@collabora.com>:
> >
> > This series adds a new "link bpc" DRM property. It reflects the display
> > link's actual achieved output bits per component, considering any
> > degradation of the bit depth done by drivers for bandwidth or other
> > reasons. The property's value is updated during an atomic commit, which
> > is also when it fires an uevent if it changed to let userspace know.
>
> Hi,
> I think it's a really good idea to have a property for knowing the
> actual bpc of the link... however, I do have one big concern with this
> API specifically: It only gives me this information after a modeset.
>
> With this limitation, I can at most show the user which bpc was chosen
> after the apply display settings and have the end user manually test
> and figure things out, but I cannot show in the UI which bpc will be
> chosen with some configuration before they apply it, and I cannot do
> atomic tests to find a desired tradeoff automatically on the
> compositor side.
To do this I'd need to see if there's some feedback mechanism for
the output configuration chosen by the atomic check phase, so that
userspace can then run a DRM_MODE_ATOMIC_TEST_ONLY and get the value
back somehow.
The current implementation wouldn't be able to do this since it
updates the property on commit_tail. I'll need to look into whether
drivers already have everything figured out with regards to link bpc
in the check phase, and how that would best be communicated to
userspace.
> As a side note, for future patches relevant for compositors, please cc
> wayland-devel. It really shouldn't be up to chance whether or not
> compositor developers that would later use the API find out about it
> before it's merged, and keeping track of all of dri-devel is way too
> much to ask from userspace developers.
I'll do that, but consider using lei[1] to have saved searches on lore
delivered to a local Mailbox, a suitable query here may be
dfb:drm_property
which will return all e-mails with diffs matching added lines with
"drm_property" in them, from which lei can then fetch the full
threads.
>
> - Xaver
>
Kind regards,
Nicolas Frattaroli
https://people.kernel.org/monsieuricon/lore-lei-part-1-getting-started [1]
next prev parent reply other threads:[~2026-04-01 18:19 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-19 12:28 [PATCH v5 0/3] Add "link bpc" DRM property Nicolas Frattaroli
2026-03-19 12:28 ` [PATCH v5 1/3] drm/connector: Add a 'link bpc' property Nicolas Frattaroli
2026-03-19 12:28 ` [PATCH v5 2/3] drm/connector: hdmi: Add support for " Nicolas Frattaroli
2026-03-19 12:28 ` [PATCH v5 3/3] drm/amd/display: " Nicolas Frattaroli
2026-03-20 14:32 ` [PATCH v5 0/3] Add "link bpc" DRM property Michel Dänzer
2026-03-20 18:02 ` Nicolas Frattaroli
2026-03-21 2:33 ` Mario Kleiner
2026-03-23 11:44 ` Michel Dänzer
2026-03-23 10:55 ` Michel Dänzer
2026-03-23 12:05 ` Nicolas Frattaroli
2026-03-23 14:38 ` Michel Dänzer
2026-03-23 16:55 ` Nicolas Frattaroli
2026-03-23 17:27 ` Michel Dänzer
2026-03-24 15:25 ` Nicolas Frattaroli
2026-03-24 16:44 ` Michel Dänzer
2026-03-26 12:17 ` Nicolas Frattaroli
2026-03-30 16:57 ` Michel Dänzer
2026-03-30 18:52 ` Harry Wentland
2026-03-31 12:50 ` Pekka Paalanen
2026-03-31 17:47 ` Harry Wentland
2026-04-01 8:40 ` Daniel Stone
2026-04-01 11:10 ` Ville Syrjälä
2026-04-01 11:43 ` Daniel Stone
2026-04-01 12:46 ` Nicolas Frattaroli
2026-04-02 17:06 ` Harry Wentland
2026-04-01 13:57 ` Ville Syrjälä
2026-04-01 14:17 ` Daniel Stone
2026-04-02 2:55 ` Mario Kleiner
2026-04-02 17:01 ` Harry Wentland
2026-03-26 13:53 ` Pekka Paalanen
2026-03-30 19:01 ` Harry Wentland
2026-03-31 10:28 ` Pekka Paalanen
2026-03-31 17:37 ` Harry Wentland
2026-04-09 15:05 ` Nicolas Dufresne
2026-03-31 8:01 ` Michel Dänzer
2026-03-31 12:38 ` Pekka Paalanen
2026-03-31 12:56 ` Michel Dänzer
2026-03-31 14:21 ` Pekka Paalanen
2026-04-01 7:46 ` Michel Dänzer
2026-04-09 22:20 ` Dmitry Baryshkov
2026-04-01 11:57 ` Xaver Hugl
2026-04-01 12:11 ` Ville Syrjälä
2026-04-01 12:25 ` Daniel Stone
2026-04-01 12:56 ` Ville Syrjälä
2026-04-01 12:14 ` Nicolas Frattaroli [this message]
2026-04-03 10:23 ` Michel Dänzer
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=4970515.ElGaqSPkdT@workhorse \
--to=nicolas.frattaroli@collabora.com \
--cc=airlied@gmail.com \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=christian.koenig@amd.com \
--cc=daniels@collabora.com \
--cc=derek.foreman@collabora.com \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=harry.wentland@amd.com \
--cc=kernel@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=marius.vlad@collabora.com \
--cc=mripard@kernel.org \
--cc=simona@ffwll.ch \
--cc=siqueira@igalia.com \
--cc=sunpeng.li@amd.com \
--cc=tzimmermann@suse.de \
--cc=ville.syrjala@linux.intel.com \
--cc=wayland-devel@lists.freedesktop.org \
--cc=xaver.hugl@kde.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.