[PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[devbrones]

This fixes a bug where some Playstation VR Headsets would not be assigned
the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
certain software under Wayland.

Signed-off-by: devbrones <jonas.cronholm@protonmail.com>
---
 drivers/gpu/drm/drm_edid.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 13bc4c290b17..51b4d7a02c02 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -239,6 +239,7 @@ static const struct edid_quirk {
 
 	/* Sony PlayStation VR Headset */
 	EDID_QUIRK('S', 'N', 'Y', 0x0704, EDID_QUIRK_NON_DESKTOP),
+	EDID_QUIRK('S', 'N', 'Y', 0xB403, EDID_QUIRK_NON_DESKTOP),
 
 	/* Sensics VR Headsets */
 	EDID_QUIRK('S', 'E', 'N', 0x1019, EDID_QUIRK_NON_DESKTOP),
-- 
2.45.2

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Jani Nikula]

On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
> This fixes a bug where some Playstation VR Headsets would not be assigned
> the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
> certain software under Wayland.

Please file a bug over at [1], and attach the EDID on that bug, so we
have some clue what's going on.

Thanks,
Jani.


[1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new


>
> Signed-off-by: devbrones <jonas.cronholm@protonmail.com>
> ---
>  drivers/gpu/drm/drm_edid.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 13bc4c290b17..51b4d7a02c02 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -239,6 +239,7 @@ static const struct edid_quirk {
>  
>  	/* Sony PlayStation VR Headset */
>  	EDID_QUIRK('S', 'N', 'Y', 0x0704, EDID_QUIRK_NON_DESKTOP),
> +	EDID_QUIRK('S', 'N', 'Y', 0xB403, EDID_QUIRK_NON_DESKTOP),
>  
>  	/* Sensics VR Headsets */
>  	EDID_QUIRK('S', 'E', 'N', 0x1019, EDID_QUIRK_NON_DESKTOP),

-- 
Jani Nikula, Intel

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 673 bytes --]

On Tue, Mar 25, 2025 at 11:16:47AM +0200, Jani Nikula wrote:
> On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
> > This fixes a bug where some Playstation VR Headsets would not be assigned
> > the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
> > certain software under Wayland.
> 
> Please file a bug over at [1], and attach the EDID on that bug, so we
> have some clue what's going on.
>
> [1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new

I'd rather have them in the commit log. Nobody uses gitlab issues for
drm-misc, and those kind of issues are just lingering around and
becoming stale.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Jani Nikula]

On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
> On Tue, Mar 25, 2025 at 11:16:47AM +0200, Jani Nikula wrote:
>> On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
>> > This fixes a bug where some Playstation VR Headsets would not be assigned
>> > the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
>> > certain software under Wayland.
>> 
>> Please file a bug over at [1], and attach the EDID on that bug, so we
>> have some clue what's going on.
>>
>> [1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new
>
> I'd rather have them in the commit log. Nobody uses gitlab issues for
> drm-misc, and those kind of issues are just lingering around and
> becoming stale.

For this one, it's fine as long as we preserve the raw EDID for
posterity. Unless the EDID does indicate VR and we need to dig deeper,
that is.

BR,
Jani.

-- 
Jani Nikula, Intel

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 1135 bytes --]

On Tue, Mar 25, 2025 at 12:47:49PM +0200, Jani Nikula wrote:
> On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
> > On Tue, Mar 25, 2025 at 11:16:47AM +0200, Jani Nikula wrote:
> >> On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
> >> > This fixes a bug where some Playstation VR Headsets would not be assigned
> >> > the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
> >> > certain software under Wayland.
> >> 
> >> Please file a bug over at [1], and attach the EDID on that bug, so we
> >> have some clue what's going on.
> >>
> >> [1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new
> >
> > I'd rather have them in the commit log. Nobody uses gitlab issues for
> > drm-misc, and those kind of issues are just lingering around and
> > becoming stale.
> 
> For this one, it's fine as long as we preserve the raw EDID for
> posterity. Unless the EDID does indicate VR and we need to dig deeper,
> that is.

What I was trying to say is if "posterity" means "a forever open issue
in drm-misc", then no, sorry, that doesn't work for me.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Jani Nikula]

On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
> On Tue, Mar 25, 2025 at 12:47:49PM +0200, Jani Nikula wrote:
>> On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
>> > On Tue, Mar 25, 2025 at 11:16:47AM +0200, Jani Nikula wrote:
>> >> On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
>> >> > This fixes a bug where some Playstation VR Headsets would not be assigned
>> >> > the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
>> >> > certain software under Wayland.
>> >> 
>> >> Please file a bug over at [1], and attach the EDID on that bug, so we
>> >> have some clue what's going on.
>> >>
>> >> [1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new
>> >
>> > I'd rather have them in the commit log. Nobody uses gitlab issues for
>> > drm-misc, and those kind of issues are just lingering around and
>> > becoming stale.
>> 
>> For this one, it's fine as long as we preserve the raw EDID for
>> posterity. Unless the EDID does indicate VR and we need to dig deeper,
>> that is.
>
> What I was trying to say is if "posterity" means "a forever open issue
> in drm-misc", then no, sorry, that doesn't work for me.

I want to check the EDID before we merge the quirk.

If the EDID does not indicate VR, we can merge. I want the EDID
preserved so we can track them down later if we need to drop or modify
the quirks.

If the EDID does indicate VR, either the quirk is unnecessary or there's
a bug somewhere. This requires further debugging, and we must not merge
the quirk.


BR,
Jani.


-- 
Jani Nikula, Intel

Re: [PATCH] drm/edid: Add non-desktop quirk for Playstation VR Headsets with Product ID 0xB403

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 1917 bytes --]

On Tue, Mar 25, 2025 at 05:03:46PM +0200, Jani Nikula wrote:
> On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
> > On Tue, Mar 25, 2025 at 12:47:49PM +0200, Jani Nikula wrote:
> >> On Tue, 25 Mar 2025, Maxime Ripard <mripard@kernel.org> wrote:
> >> > On Tue, Mar 25, 2025 at 11:16:47AM +0200, Jani Nikula wrote:
> >> >> On Sat, 22 Mar 2025, devbrones <jonas.cronholm@protonmail.com> wrote:
> >> >> > This fixes a bug where some Playstation VR Headsets would not be assigned
> >> >> > the EDID_QUIRK_NON_DESKTOP quirk, causing them to be inaccessible by
> >> >> > certain software under Wayland.
> >> >> 
> >> >> Please file a bug over at [1], and attach the EDID on that bug, so we
> >> >> have some clue what's going on.
> >> >>
> >> >> [1] https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/new
> >> >
> >> > I'd rather have them in the commit log. Nobody uses gitlab issues for
> >> > drm-misc, and those kind of issues are just lingering around and
> >> > becoming stale.
> >> 
> >> For this one, it's fine as long as we preserve the raw EDID for
> >> posterity. Unless the EDID does indicate VR and we need to dig deeper,
> >> that is.
> >
> > What I was trying to say is if "posterity" means "a forever open issue
> > in drm-misc", then no, sorry, that doesn't work for me.
> 
> I want to check the EDID before we merge the quirk.
> 
> If the EDID does not indicate VR, we can merge. I want the EDID
> preserved so we can track them down later if we need to drop or modify
> the quirks.
> 
> If the EDID does indicate VR, either the quirk is unnecessary or there's
> a bug somewhere. This requires further debugging, and we must not merge
> the quirk.

I understand that, but I don't see why putting it in the commit log, or
as a mail reply to the patch is not a good solution for that.

Or why using drm-misc issues for this is a good one.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]