All of lore.kernel.org
 help / color / mirror / Atom feed
From: laurent.pinchart@ideasonboard.com (Laurent Pinchart)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] Re: [PATCH v7 4/8] drm/sunxi: Add DT bindings documentation of Allwinner HDMI
Date: Thu, 01 Dec 2016 13:44:31 +0200	[thread overview]
Message-ID: <3596263.47eZbXLy2S@avalon> (raw)
In-Reply-To: <20161201123009.d2b4d9d389070200c54138ee@free.fr>

On Thursday 01 Dec 2016 12:30:09 Jean-Francois Moine wrote:
> On Thu, 01 Dec 2016 12:41:20 +0200 Laurent Pinchart wrote:
> >>> If a DVI connector instead of a HDMI connector is soldered, how
> >>> should such a device tree be written?
> >> 
> >> Use a dvi-connector instead :)
> > 
> > The HDMI encoder DT node doesn't (and certainly shouldn't) report what
> > type of connector is mounted on the board. Having a connector node in DT
> > makes the connector type available to the system, allowing the DRM driver
> > to expose the right connector type to userspace (it would be confusing
> > for the user to report DRM_MODE_CONNECTOR_HDMIA for a DVI connector).
> 
> The connector type, HDMI or DVI, is known by the EDID.
> The user is not interested by the software indication of the connector
> type: (s)he knows it because (s)he connected him/herself the display
> device.

That's not correct. The connector type reported by DRM to userspace can be 
used as a hint to information the user about connectors. Displaying a "Please 
connect a monitor to the HDMI connector" is confusing when the system has a 
DVI connector.

> And the DRM driver does not do anything from the knowledge of the
> connector type in the DT. Only the EDID may tell if the display device
> may do audio streaming (direct HDMI with audio capability) or not
> (direct HDMI without audio, HDMI to DVI cable, DVI physical connector).

-- 
Regards,

Laurent Pinchart

WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
To: Jean-Francois Moine <moinejf-GANU6spQydw@public.gmane.org>
Cc: Maxime Ripard
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"moinejf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
	<moinejf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	"linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org"
	<linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: Re: [PATCH v7 4/8] drm/sunxi: Add DT bindings documentation of Allwinner HDMI
Date: Thu, 01 Dec 2016 13:44:31 +0200	[thread overview]
Message-ID: <3596263.47eZbXLy2S@avalon> (raw)
In-Reply-To: <20161201123009.d2b4d9d389070200c54138ee-GANU6spQydw@public.gmane.org>

On Thursday 01 Dec 2016 12:30:09 Jean-Francois Moine wrote:
> On Thu, 01 Dec 2016 12:41:20 +0200 Laurent Pinchart wrote:
> >>> If a DVI connector instead of a HDMI connector is soldered, how
> >>> should such a device tree be written?
> >> 
> >> Use a dvi-connector instead :)
> > 
> > The HDMI encoder DT node doesn't (and certainly shouldn't) report what
> > type of connector is mounted on the board. Having a connector node in DT
> > makes the connector type available to the system, allowing the DRM driver
> > to expose the right connector type to userspace (it would be confusing
> > for the user to report DRM_MODE_CONNECTOR_HDMIA for a DVI connector).
> 
> The connector type, HDMI or DVI, is known by the EDID.
> The user is not interested by the software indication of the connector
> type: (s)he knows it because (s)he connected him/herself the display
> device.

That's not correct. The connector type reported by DRM to userspace can be 
used as a hint to information the user about connectors. Displaying a "Please 
connect a monitor to the HDMI connector" is confusing when the system has a 
DVI connector.

> And the DRM driver does not do anything from the knowledge of the
> connector type in the DT. Only the EDID may tell if the display device
> may do audio streaming (direct HDMI with audio capability) or not
> (direct HDMI without audio, HDMI to DVI cable, DVI physical connector).

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2016-12-01 11:44 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-29 10:18 [PATCH v7 0/8] drm: sun8i: Add DE2 HDMI video support Jean-Francois Moine
2016-11-29 10:18 ` Jean-Francois Moine
2016-11-28 14:23 ` [PATCH v7 1/8] drm: sun8i: Add a basic DRM driver for Allwinner DE2 Jean-Francois Moine
2016-11-28 14:23   ` Jean-Francois Moine
2016-11-29 14:30   ` Daniel Vetter
2016-11-29 14:30     ` Daniel Vetter
2016-11-29 14:33     ` [linux-sunxi] " Icenowy Zheng
2016-11-29 14:33       ` Icenowy Zheng
2016-11-28 18:02 ` [PATCH v7 2/8] drm/sun8i: Add DT bindings documentation of " Jean-Francois Moine
2016-11-28 18:02   ` Jean-Francois Moine
2016-11-29 18:41   ` Laurent Pinchart
2016-11-29 18:41     ` Laurent Pinchart
2016-11-29 18:45     ` Laurent Pinchart
2016-11-29 18:45       ` Laurent Pinchart
2016-11-29  8:39 ` [PATCH v7 3/8] drm: sun8i: add HDMI video support to A83T and H3 Jean-Francois Moine
2016-11-29  8:39   ` Jean-Francois Moine
2016-11-29  9:08 ` [PATCH v7 4/8] drm/sunxi: Add DT bindings documentation of Allwinner HDMI Jean-Francois Moine
2016-11-29  9:08   ` Jean-Francois Moine
2016-11-29 18:46   ` Laurent Pinchart
2016-11-29 18:46     ` Laurent Pinchart
2016-11-29 19:27     ` Jean-Francois Moine
2016-11-29 19:27       ` Jean-Francois Moine
2016-11-29 19:33       ` Laurent Pinchart
2016-11-29 19:33         ` Laurent Pinchart
2016-11-29 20:04         ` Jean-Francois Moine
2016-11-29 20:04           ` Jean-Francois Moine
2016-11-29 20:10           ` Laurent Pinchart
2016-11-29 20:10             ` Laurent Pinchart
2016-11-30  8:12             ` Jean-Francois Moine
2016-11-30  8:12               ` Jean-Francois Moine
2016-11-30  8:20               ` Laurent Pinchart
2016-11-30  8:20                 ` Laurent Pinchart
2016-11-30  9:27                 ` Jean-Francois Moine
2016-11-30  9:27                   ` Jean-Francois Moine
2016-11-30  9:52                   ` Laurent Pinchart
2016-11-30  9:52                     ` Laurent Pinchart
2016-11-30 10:44                     ` Jean-Francois Moine
2016-11-30 10:44                       ` Jean-Francois Moine
2016-11-30 17:33                       ` [linux-sunxi] " Icenowy Zheng
2016-11-30 17:33                         ` Icenowy Zheng
2016-12-01  8:55                         ` [linux-sunxi] " Maxime Ripard
2016-12-01  8:55                           ` Maxime Ripard
2016-12-01 10:41                           ` [linux-sunxi] " Laurent Pinchart
2016-12-01 10:41                             ` Laurent Pinchart
2016-12-01 11:30                             ` [linux-sunxi] " Jean-Francois Moine
2016-12-01 11:30                               ` Jean-Francois Moine
2016-12-01 11:44                               ` Laurent Pinchart [this message]
2016-12-01 11:44                                 ` Laurent Pinchart
2016-11-30 17:24                   ` [linux-sunxi] " Icenowy Zheng
2016-11-30 17:24                     ` Icenowy Zheng
2016-11-29 10:10 ` [PATCH v7 5/8] clk: sunxi-ng: define the PLL DE clock Jean-Francois Moine
2016-11-29 10:10   ` Jean-Francois Moine
2016-11-29 10:12 ` [PATCH v7 6/8] ARM: dts: sun8i-h3: add HDMI video nodes Jean-Francois Moine
2016-11-29 10:12   ` Jean-Francois Moine
2016-11-29 10:14 ` [PATCH v7 7/8] ARM: dts: sun8i-h3: Add HDMI video to the Banana Pi M2+ Jean-Francois Moine
2016-11-29 10:14   ` Jean-Francois Moine
2016-11-29 10:16 ` [PATCH v7 8/8] ARM: dts: sun8i-h3: Add HDMI video to the Orange PI 2 Jean-Francois Moine
2016-11-29 10:16   ` Jean-Francois Moine
2016-11-29 21:36 ` [PATCH v7 0/8] drm: sun8i: Add DE2 HDMI video support Maxime Ripard
2016-11-29 21:36   ` Maxime Ripard
2016-11-29 22:47   ` Jernej Skrabec
2016-11-29 22:56     ` Laurent Pinchart
2016-11-29 22:56       ` Laurent Pinchart
2016-11-29 23:24       ` Jernej Skrabec
2016-11-30  8:08         ` Laurent Pinchart
2016-11-30  8:08           ` Laurent Pinchart
2016-11-30 18:04           ` Jernej Skrabec
2016-11-30  9:05   ` Jean-Francois Moine
2016-11-30  9:05     ` Jean-Francois Moine
2016-11-30 10:12     ` Laurent Pinchart
2016-11-30 10:12       ` Laurent Pinchart
2016-12-01  9:13       ` Maxime Ripard
2016-12-01  9:13         ` Maxime Ripard
2016-12-01  9:19         ` Laurent Pinchart
2016-12-01  9:19           ` Laurent Pinchart
2016-12-01  9:42           ` Maxime Ripard
2016-12-01  9:42             ` Maxime Ripard
2016-12-01  9:28         ` How should we group related devices in DT ? (was Re: [PATCH v7 0/8] drm: sun8i: Add DE2 HDMI video support) Laurent Pinchart
2016-12-01  9:28           ` Laurent Pinchart

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=3596263.47eZbXLy2S@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-arm-kernel@lists.infradead.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.