[PATCH] drm/edid: Quirk Sensics, 3 Glasses HMD as non-desktop.

[Philipp Zabel]

On Sat, 2018-10-20 at 20:42 +0100, Ryan Pavlik wrote:
> Two of the EDID vendor/product pairs are used across a variety of
> Sensics products, as well as the OSVR HDK and HDK 2.
> 
> The third is for the "3 Glasses" brand "D3" HMD.
> 
> Signed-off-by: Ryan Pavlik <ryan.pavlik at collabora.com>
> ---
>  drivers/gpu/drm/drm_edid.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 3c9fc9964..68b7bc007 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -187,6 +187,15 @@ static const struct edid_quirk {
>  
>  	/* Sony PlayStation VR Headset */
>  	{ "SNY", 0x0704, EDID_QUIRK_NON_DESKTOP },
> +
> +	/* Sensics VR Headsets */
> +	{ "SEN", 0x1019, EDID_QUIRK_NON_DESKTOP },
> +
> +	/* Sensics, OSVR HDK and HDK2 VR Headsets */
> +	{ "SVR", 0x1019, EDID_QUIRK_NON_DESKTOP },

This looks good to me. These are indeed HMDs, and I can confirm
"SVR"/0x1019 is correct for the HDK2.

> +	/* 3Glasses D3 VR Headset */
> +	{ "TSB", 0x8888, EDID_QUIRK_NON_DESKTOP },

This looks a lot like the vendor has not bothered to set a custom
vendor/product id in the Toshiba HDMI-to-DSI bridge chip. I would
argue that it is safer to mark such displays as non-desktop, though.

>  };
>  
>  /*

Reviewed-by: Philipp Zabel <p.zabel at pengutronix.de>

Can we take this into drm-misc?

regards
Philipp