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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Thu, 23 May 2024 04:51:13 +0200
From: Rino André Johnsen <rinoandrejohnsen@...il.com>
To: Simon Ser <contact@...rsion.fr>
Cc: Mario Limonciello <mario.limonciello@....com>, 
	Christian König <christian.koenig@....com>, 
	alexander.deucher@....com, Harry Wentland <harry.wentland@....com>, 
	Leo Li <sunpeng.li@....com>, Rodrigo Siqueira <Rodrigo.Siqueira@....com>, 
	"Pan, Xinhui" <Xinhui.Pan@....com>, David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>, 
	Aurabindo Pillai <aurabindo.pillai@....com>, Hersen Wu <hersenxs.wu@....com>, 
	Hamza Mahfooz <hamza.mahfooz@....com>, Wayne Lin <wayne.lin@....com>, 
	Srinivasan Shanmugam <srinivasan.shanmugam@....com>, Fangzhi Zuo <jerry.zuo@....com>, 
	Tom Chung <chiahsuan.chung@....com>, Nicholas Kazlauskas <nicholas.kazlauskas@....com>, 
	amd-gfx@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] drm/amd/display: Add pixel encoding info to debugfs

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@...rsion.fr> wrote:
>
> On Wednesday, May 22nd, 2024 at 15:36, Mario Limonciello <mario.limonciello@....com> 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@yngvason.is/
>
> The patch also allows user-space to set the "pixel encoding".