[PATCH 2/2] ARM: dts: duovero-parlor: Add HDMI output
Russell King - ARM Linux
linux at arm.linux.org.uk
Tue Feb 25 07:39:21 EST 2014
On Mon, Feb 24, 2014 at 05:22:27PM -0300, Javier Martinez Canillas wrote:
> Hi Russell,
>
> On Mon, Feb 24, 2014 at 3:03 PM, Russell King - ARM Linux
> <linux at arm.linux.org.uk> wrote:
> > On Mon, Feb 24, 2014 at 06:07:49PM +0100, Florian Vaussard wrote:
> >> + hdmi0: connector at 0 {
> >> + compatible = "hdmi-connector";
> >
> > This looks way to generic a compatible string. Are you sure it's
> > correct?
> >
>
> That compatible string is correct according to the latest series
> posted by Tomi Valkeinen to add DT bindings for the OMAP Display
> SubSystem (DSS) [0].
in which case:
N N AA CCCC K K
NN N A A C C K K
N N N A A C KK
N N N AAAAAA C KK
N NN A A C C K K
N N A A CCCC K K
Yes, that's a very big nack. Two things:
1. OMAP really doesn't have the right to define a compatible string which
is as generic as "hdmi-connector".
2. Even with "omapdss," before it, the convention that DT people have adopted
is for the prefix to be "companyname," and not a subsystem.
I'm not the only one with this concern - I discussed it with Arnd last
night and his comments were about it being "obviously bogus".
> The property is added on this patch [1] and as far as I understood the
> idea is that it could be a generic DT binding that can be used by
> platform specific HDMI connectors like the omap dss HDMI connector [2].
Why do the physical connectors need a DT binding? Surely what needs the
DT binding is the HDMI encoder - and that certainly should no way be a
generic name, because there's no such thing as a generic HDMI encoder
chip.
--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
More information about the linux-arm-kernel
mailing list