From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: "Sebastian Reichel" <sre@ring0.de>,
"Javier Martinez Canillas" <javier@dowhile0.org>,
"Tomi Valkeinen" <tomi.valkeinen@ti.com>,
"Florian Vaussard" <florian.vaussard@epfl.ch>,
"Benoît Cousson" <bcousson@baylibre.com>,
"Tony Lindgren" <tony@atomide.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"Ash Charles" <ash@gumstix.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/2] ARM: dts: duovero-parlor: Add HDMI output
Date: Tue, 25 Feb 2014 20:56:34 +0000 [thread overview]
Message-ID: <20140225205634.GZ27282@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <2364124.7r818M0xAj@avalon>
On Tue, Feb 25, 2014 at 05:51:21PM +0100, Laurent Pinchart wrote:
> I don't think all physical connectors require a DT binding per-se, but they
> need to be represented in DT as they're part of the hardware. We could push
> connector-related information to the nodes of all chips that have interfaces
> wired directly to connectors, but that would result in more complex DT
> bindings and core. I believe modeling connectors using separate DT nodes is be
> best, and would allow easier support for more complex connectors that carry
> multiple streams/signals in parallel (video, audio, DDC, ...).
There is some sanity to representing physical connectors in DT, but it's
not for the reason you mention above. If you consider that it's possible
on PCs to find out what connectors are on the motherboard and where they
are located, this is very useful information to be stored and presented.
However, the idea that you combine streams at connectors is not a
universal truth, and is certainly false for HDMI. HDMI combines video
and audio at the encoder stage, not at the connector stage, and many
HDMI encoders will provide everything required for driving the connector.
However, my major objection here is not really that: my major objection
is using something as generic as "hdmi-connector" as a compatible string.
The reason is that we have to remember that DT is not just "a Linux thing".
It's /supposed/ to be an OS independent representation of the hardware.
If we invent something generic called a "hdmi-connector" then we had
better first do a thorough search to make sure we're not trampling on
anything which is standardized or becoming a standard - if there is,
we should work with them - and if that's not possible, then we need to
distingush ourselves from them.
What we can't do is go around inventing generic stuff without having our
eyes wide open.
So, here's a good question to probe how far this has been thought through
already: what has been done to discuss the creation of this generic
"hdmi-connector" thing with the various parties who are interested in
HDMI outputs under DRM using device tree?
If that hasn't happened, that's quite a failing - it means that we're
on the road to having two _implementation specific_ DT representations
for the same hardware - one for fbdev and one for DRM. That really
isn't on.
Yes, it then opens a pandora's box of problems about how we determine
whether DRM or fbdev should bind to the DT nodes, but that should be
an entirely separate issue (and, ideally of course, both should use
the same sub-drivers for the components.)
--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
next prev parent reply other threads:[~2014-02-25 20:56 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-24 17:07 [PATCH 0/2] ARM: dts: OMAP4: Add support for DuoVero/Parlor Florian Vaussard
2014-02-24 17:07 ` [PATCH 1/2] ARM: dts: Add support for OMAP4 Gumstix DuoVero/Parlor Florian Vaussard
2014-03-03 10:08 ` Florian Vaussard
2014-03-04 18:17 ` Tony Lindgren
2014-02-24 17:07 ` [PATCH 2/2] ARM: dts: duovero-parlor: Add HDMI output Florian Vaussard
2014-02-24 18:03 ` Russell King - ARM Linux
2014-02-24 20:22 ` Javier Martinez Canillas
2014-02-25 7:54 ` Florian Vaussard
2014-02-25 12:39 ` Russell King - ARM Linux
2014-02-25 13:05 ` Florian Vaussard
[not found] ` <20140225123921.GY27282-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2014-02-25 13:41 ` Sebastian Reichel
2014-02-25 16:51 ` Laurent Pinchart
2014-02-25 20:56 ` Russell King - ARM Linux [this message]
2014-02-26 11:14 ` Tomi Valkeinen
[not found] ` <530DCC8A.6060708-l0cyMroinI0@public.gmane.org>
2014-02-26 12:03 ` Russell King - ARM Linux
2014-02-26 12:44 ` Tomi Valkeinen
2014-02-26 13:28 ` Russell King - ARM Linux
2014-02-26 14:35 ` Tomi Valkeinen
2014-03-06 23:29 ` 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=20140225205634.GZ27282@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=ash@gumstix.com \
--cc=bcousson@baylibre.com \
--cc=devicetree@vger.kernel.org \
--cc=florian.vaussard@epfl.ch \
--cc=javier@dowhile0.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=sre@ring0.de \
--cc=tomi.valkeinen@ti.com \
--cc=tony@atomide.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).