All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Sefa Eyeoglu <contact@scrumplex.net>, dri-devel@lists.freedesktop.org
Cc: Sefa Eyeoglu <contact@scrumplex.net>
Subject: Re: [PATCH] drm/edid: add non-desktop quirk to Bigscreen Beyond HMD
Date: Fri, 17 May 2024 15:54:18 +0300	[thread overview]
Message-ID: <874jawbnut.fsf@intel.com> (raw)
In-Reply-To: <20240517105555.654262-1-contact@scrumplex.net>

On Fri, 17 May 2024, Sefa Eyeoglu <contact@scrumplex.net> wrote:
> The Bigscreen Beyond VR headset is a non-desktop output and should be
> marked as such using an EDID quirk.

I'd appreciate a bug being filed at [1], attaching the EDID of the panel
there, maybe dmesg with drm.debug=14 enabled too, and referencing the
bug in the commit message. It gets terribly hard to figure anything out
about the quirks afterwards when some time has passed.

Thanks,
Jani.



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

>
> Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
> ---
>  drivers/gpu/drm/drm_edid.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 4f54c91b31b2..d407efc0fb55 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -200,6 +200,9 @@ static const struct edid_quirk {
>  	/* Rotel RSX-1058 forwards sink's EDID but only does HDMI 1.1*/
>  	EDID_QUIRK('E', 'T', 'R', 13896, EDID_QUIRK_FORCE_8BPC),
>  
> +	/* Bigscreen Beyond Headset */
> +	EDID_QUIRK('B', 'I', 'G', 0x1234, EDID_QUIRK_NON_DESKTOP),
> +
>  	/* Valve Index Headset */
>  	EDID_QUIRK('V', 'L', 'V', 0x91a8, EDID_QUIRK_NON_DESKTOP),
>  	EDID_QUIRK('V', 'L', 'V', 0x91b0, EDID_QUIRK_NON_DESKTOP),

-- 
Jani Nikula, Intel

  reply	other threads:[~2024-05-17 12:54 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-17 10:55 [PATCH] drm/edid: add non-desktop quirk to Bigscreen Beyond HMD Sefa Eyeoglu
2024-05-17 12:54 ` Jani Nikula [this message]
2024-05-17 14:14   ` Sefa Eyeoglu
2024-05-17 14:09 ` Sefa Eyeoglu
2024-05-17 14:52   ` Philipp Zabel
2024-05-17 15:36     ` Sefa Eyeoglu
2024-05-20  8:40       ` Jani Nikula

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=874jawbnut.fsf@intel.com \
    --to=jani.nikula@linux.intel.com \
    --cc=contact@scrumplex.net \
    --cc=dri-devel@lists.freedesktop.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.