On Thu, Jun 8, 2017 at 6:47 AM, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote: > On Wed, Jun 7, 2017 at 6:36 PM, Rob Herring <robh@xxxxxxxxxx> wrote: >> On Fri, Jun 02, 2017 at 09:42:19PM +0200, Maxime Ripard wrote: >>> On Fri, Jun 02, 2017 at 06:10:24PM +0800, Chen-Yu Tsai wrote: >>> > The MSI Primo81 tablet has a micro HDMI connector at the bottom. >>> > This is connected to the SoCs HDMI output. >>> > >>> > Enable the display pipeline and the HDMI output. >>> > >>> > Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxx> >>> > --- >>> > arch/arm/boot/dts/sun6i-a31s-primo81.dts | 25 +++++++++++++++++++++++++ >>> > 1 file changed, 25 insertions(+) >>> > >>> > diff --git a/arch/arm/boot/dts/sun6i-a31s-primo81.dts b/arch/arm/boot/dts/sun6i-a31s-primo81.dts >>> > index f3712753fa42..26154b2f87a3 100644 >>> > --- a/arch/arm/boot/dts/sun6i-a31s-primo81.dts >>> > +++ b/arch/arm/boot/dts/sun6i-a31s-primo81.dts >>> > @@ -52,17 +52,42 @@ >>> > / { >>> > model = "MSI Primo81 tablet"; >>> > compatible = "msi,primo81", "allwinner,sun6i-a31s"; >>> > + >>> > + hdmi-connector { >>> > + compatible = "hdmi-connector"; >>> > + type = "c"; >>> >>> Should we add a connector type for this one? >> >> c as in USB Type C? I'd prefer to see a new compatible string. Is type >> even documented? Indeed it is: http://elixir.free-electrons.com/linux/latest/source/Documentation/devicetree/bindings/display/connector/hdmi-connector.txt#L6 > Probably as in HDMI Type C (there are Type A, C, and D connectors, aka > "regular", "mini", and "micro"). Correct. ChenYu -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html