Re: [PATCH v2] drm/amd/display: Add pixel encoding info to debugfs

From: Rino André Johnsen
Date: Wed May 22 2024 - 22:51:34 EST


Looked through the patch series from Andri Yngvason and that does
exactly what I wanted in the first place. I think that the patch
series should be encouraged to be merged in as fast as possible.

For the patch I have submitted, it stands on its own, since the patch
series from Andri Yngvason does not include anything in the debugfs.
This means whenever or not the patch series gets merged, at least
those with an AMD gpu can figure out which pixel encoding that is
used.

Rino

On Wed, May 22, 2024 at 3:40 PM Simon Ser <contact@xxxxxxxxxxx> wrote:
>
> On Wednesday, May 22nd, 2024 at 15:36, Mario Limonciello <mario.limonciello@xxxxxxx> wrote:
>
> > > To be perfectly honest with you, I haven't given that much though. I
> > > used the 'bpc' and 'colorspace' property in debugfs, since I could not
> > > find that information anywhere else. And since I also needed to verify
> > > the pixel encoding being used, I added it where those other values
> > > were. That made for a simple and easy addition for this property.
> > >
> > > If you want me to do this differently, let me know. And please point
> > > me to the standardized DRM property where I should expose the values.
>
> FWIW, there is a patch from Andri to add a similar (?) property:
> https://lore.kernel.org/dri-devel/20240115160554.720247-1-andri@xxxxxxxxxxx/
>
> The patch also allows user-space to set the "pixel encoding".