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".
