linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Misc function enhance to Banana Pi M2 Zero board
@ 2018-07-11 13:22 Icenowy Zheng
  2018-07-11 13:22 ` [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero Icenowy Zheng
  0 siblings, 1 reply; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-11 13:22 UTC (permalink / raw)
  To: Maxime Ripard, Chen-Yu Tsai
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-sunxi, Icenowy Zheng

This patchset adds two device-tree based function enhancements to the
Banana Pi M2 Zero board.

PATCH 1 adds the HDMI output. (Note: although the connector type is set
to "c" in the device tree, currently sun8i-dw-hdmi doesn't check it and
still set the type to HDMI-A).

PATCH 2 adds VDD-CPUX regulator, and link it to DVFS.

Icenowy Zheng (2):
  ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  ARM: sun8i: h3: add SY8113B regulator on Banana Pi M2 Zero board

 .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 46 +++++++++++++++++++
 1 file changed, 46 insertions(+)

-- 
2.17.1


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 13:22 [PATCH 0/2] Misc function enhance to Banana Pi M2 Zero board Icenowy Zheng
@ 2018-07-11 13:22 ` Icenowy Zheng
  2018-07-11 15:05   ` Maxime Ripard
  2018-07-18 22:13   ` [linux-sunxi] " Julian Calaby
  0 siblings, 2 replies; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-11 13:22 UTC (permalink / raw)
  To: Maxime Ripard, Chen-Yu Tsai
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-sunxi, Icenowy Zheng

Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
controller of Allwinner H3 SoC.

Enable the HDMI output in Banana Pi M2 Zero device tree.

Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
---
 .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25 +++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
index 7d01f9322658..eb61dcf32797 100644
--- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
+++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
@@ -26,6 +26,17 @@
 		stdout-path = "serial0:115200n8";
 	};
 
+	hdmi-connector {
+		compatible = "hdmi-connector";
+		type = "c";
+
+		port {
+			hdmi_con_in: endpoint {
+				remote-endpoint = <&hdmi_out_con>;
+			};
+		};
+	};
+
 	leds {
 		compatible = "gpio-leds";
 		pinctrl-names = "default";
@@ -55,10 +66,24 @@
 	};
 };
 
+&de {
+	status = "okay";
+};
+
 &ehci0 {
 	status = "okay";
 };
 
+&hdmi {
+	status = "okay";
+};
+
+&hdmi_out {
+	hdmi_out_con: endpoint {
+		remote-endpoint = <&hdmi_con_in>;
+	};
+};
+
 &mmc0 {
 	vmmc-supply = <&reg_vcc3v3>;
 	bus-width = <4>;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 13:22 ` [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero Icenowy Zheng
@ 2018-07-11 15:05   ` Maxime Ripard
  2018-07-11 15:14     ` [linux-sunxi] " Jernej Škrabec
  2018-07-11 15:15     ` Icenowy Zheng
  2018-07-18 22:13   ` [linux-sunxi] " Julian Calaby
  1 sibling, 2 replies; 12+ messages in thread
From: Maxime Ripard @ 2018-07-11 15:05 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi

[-- Attachment #1: Type: text/plain, Size: 1081 bytes --]

Hi,

On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
> controller of Allwinner H3 SoC.
> 
> Enable the HDMI output in Banana Pi M2 Zero device tree.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> ---
>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25 +++++++++++++++++++
>  1 file changed, 25 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> index 7d01f9322658..eb61dcf32797 100644
> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> @@ -26,6 +26,17 @@
>  		stdout-path = "serial0:115200n8";
>  	};
>  
> +	hdmi-connector {
> +		compatible = "hdmi-connector";
> +		type = "c";

This is not one of the connector type declared in DRM, how is it
exposed to the userspace?

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 15:05   ` Maxime Ripard
@ 2018-07-11 15:14     ` Jernej Škrabec
  2018-07-11 15:19       ` Icenowy Zheng
  2018-07-11 15:15     ` Icenowy Zheng
  1 sibling, 1 reply; 12+ messages in thread
From: Jernej Škrabec @ 2018-07-11 15:14 UTC (permalink / raw)
  To: linux-sunxi, maxime.ripard
  Cc: Icenowy Zheng, Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel

Dne sreda, 11. julij 2018 ob 17:05:32 CEST je Maxime Ripard napisal(a):
> Hi,
> 
> On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
> > Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
> > controller of Allwinner H3 SoC.
> > 
> > Enable the HDMI output in Banana Pi M2 Zero device tree.
> > 
> > Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> > ---
> > 
> >  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25 +++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> > b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts index
> > 7d01f9322658..eb61dcf32797 100644
> > --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> > +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> > @@ -26,6 +26,17 @@
> > 
> >  		stdout-path = "serial0:115200n8";
> >  	
> >  	};
> > 
> > +	hdmi-connector {
> > +		compatible = "hdmi-connector";
> > +		type = "c";
> 
> This is not one of the connector type declared in DRM, how is it
> exposed to the userspace?

DT binding allows it:

HDMI Connector
==============

Required properties:
- compatible: "hdmi-connector"
- type: the HDMI connector type: "a", "b", "c", "d" or "e"

but DW HDMI bridge driver hardcodes it to type A. Actually, kernel has defines 
only for type A and B.

Actually, many HDMI drivers have type A hardcoded...

Best regards,
Jernej




^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 15:05   ` Maxime Ripard
  2018-07-11 15:14     ` [linux-sunxi] " Jernej Škrabec
@ 2018-07-11 15:15     ` Icenowy Zheng
  2018-07-12  6:46       ` Maxime Ripard
  1 sibling, 1 reply; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-11 15:15 UTC (permalink / raw)
  To: maxime.ripard, Maxime Ripard
  Cc: Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi



于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
>Hi,
>
>On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
>> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
>> controller of Allwinner H3 SoC.
>> 
>> Enable the HDMI output in Banana Pi M2 Zero device tree.
>> 
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>> ---
>>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
>+++++++++++++++++++
>>  1 file changed, 25 insertions(+)
>> 
>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> index 7d01f9322658..eb61dcf32797 100644
>> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> @@ -26,6 +26,17 @@
>>  		stdout-path = "serial0:115200n8";
>>  	};
>>  
>> +	hdmi-connector {
>> +		compatible = "hdmi-connector";
>> +		type = "c";
>
>This is not one of the connector type declared in DRM, how is it
>exposed to the userspace?

So just use "A" here (because of single link)?

>
>Maxime

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 15:14     ` [linux-sunxi] " Jernej Škrabec
@ 2018-07-11 15:19       ` Icenowy Zheng
  0 siblings, 0 replies; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-11 15:19 UTC (permalink / raw)
  To: jernej.skrabec, Jernej Škrabec, linux-sunxi, maxime.ripard
  Cc: Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel



于 2018年7月11日 GMT+08:00 下午11:14:19, "Jernej Škrabec" <jernej.skrabec@gmail.com> 写到:
>Dne sreda, 11. julij 2018 ob 17:05:32 CEST je Maxime Ripard napisal(a):
>> Hi,
>> 
>> On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
>> > Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
>> > controller of Allwinner H3 SoC.
>> > 
>> > Enable the HDMI output in Banana Pi M2 Zero device tree.
>> > 
>> > Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>> > ---
>> > 
>> >  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
>+++++++++++++++++++
>> >  1 file changed, 25 insertions(+)
>> > 
>> > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> > b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts index
>> > 7d01f9322658..eb61dcf32797 100644
>> > --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> > +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> > @@ -26,6 +26,17 @@
>> > 
>> >  		stdout-path = "serial0:115200n8";
>> >  	
>> >  	};
>> > 
>> > +	hdmi-connector {
>> > +		compatible = "hdmi-connector";
>> > +		type = "c";
>> 
>> This is not one of the connector type declared in DRM, how is it
>> exposed to the userspace?
>
>DT binding allows it:
>
>HDMI Connector
>==============
>
>Required properties:
>- compatible: "hdmi-connector"
>- type: the HDMI connector type: "a", "b", "c", "d" or "e"
>
>but DW HDMI bridge driver hardcodes it to type A. Actually, kernel has
>defines 
>only for type A and B.

I assume it's for single link/dual link.

>
>Actually, many HDMI drivers have type A hardcoded...
>
>Best regards,
>Jernej

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 15:15     ` Icenowy Zheng
@ 2018-07-12  6:46       ` Maxime Ripard
  2018-07-12  7:09         ` Icenowy Zheng
  0 siblings, 1 reply; 12+ messages in thread
From: Maxime Ripard @ 2018-07-12  6:46 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi

[-- Attachment #1: Type: text/plain, Size: 1484 bytes --]

On Wed, Jul 11, 2018 at 11:15:50PM +0800, Icenowy Zheng wrote:
> 
> 
> 于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
> >Hi,
> >
> >On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
> >> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
> >> controller of Allwinner H3 SoC.
> >> 
> >> Enable the HDMI output in Banana Pi M2 Zero device tree.
> >> 
> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> >> ---
> >>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
> >+++++++++++++++++++
> >>  1 file changed, 25 insertions(+)
> >> 
> >> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> index 7d01f9322658..eb61dcf32797 100644
> >> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> @@ -26,6 +26,17 @@
> >>  		stdout-path = "serial0:115200n8";
> >>  	};
> >>  
> >> +	hdmi-connector {
> >> +		compatible = "hdmi-connector";
> >> +		type = "c";
> >
> >This is not one of the connector type declared in DRM, how is it
> >exposed to the userspace?
> 
> So just use "A" here (because of single link)?

If the connector type is C, it should be C, and DRM / driver adjusted
to handle it.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-12  6:46       ` Maxime Ripard
@ 2018-07-12  7:09         ` Icenowy Zheng
  2018-07-12 14:53           ` Maxime Ripard
  0 siblings, 1 reply; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-12  7:09 UTC (permalink / raw)
  To: linux-arm-kernel, Maxime Ripard
  Cc: devicetree, Chen-Yu Tsai, linux-sunxi, linux-kernel



于 2018年7月12日 GMT+08:00 下午2:46:01, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
>On Wed, Jul 11, 2018 at 11:15:50PM +0800, Icenowy Zheng wrote:
>> 
>> 
>> 于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard
><maxime.ripard@bootlin.com> 写到:
>> >Hi,
>> >
>> >On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
>> >> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
>> >> controller of Allwinner H3 SoC.
>> >> 
>> >> Enable the HDMI output in Banana Pi M2 Zero device tree.
>> >> 
>> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>> >> ---
>> >>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
>> >+++++++++++++++++++
>> >>  1 file changed, 25 insertions(+)
>> >> 
>> >> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> index 7d01f9322658..eb61dcf32797 100644
>> >> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> @@ -26,6 +26,17 @@
>> >>  		stdout-path = "serial0:115200n8";
>> >>  	};
>> >>  
>> >> +	hdmi-connector {
>> >> +		compatible = "hdmi-connector";
>> >> +		type = "c";
>> >
>> >This is not one of the connector type declared in DRM, how is it
>> >exposed to the userspace?
>> 
>> So just use "A" here (because of single link)?
>
>If the connector type is C, it should be C, and DRM / driver adjusted
>to handle it.

Okay I won't change this property, if there's any further revisions.

For DRM driver, I think it can be an independent patchset.

>
>Maxime

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-12  7:09         ` Icenowy Zheng
@ 2018-07-12 14:53           ` Maxime Ripard
  2018-07-13  2:28             ` Icenowy Zheng
  0 siblings, 1 reply; 12+ messages in thread
From: Maxime Ripard @ 2018-07-12 14:53 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: linux-arm-kernel, devicetree, Chen-Yu Tsai, linux-sunxi, linux-kernel

On Thu, Jul 12, 2018 at 03:09:55PM +0800, Icenowy Zheng wrote:
> 于 2018年7月12日 GMT+08:00 下午2:46:01, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
> >On Wed, Jul 11, 2018 at 11:15:50PM +0800, Icenowy Zheng wrote:
> >> 
> >> 
> >> 于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard
> ><maxime.ripard@bootlin.com> 写到:
> >> >Hi,
> >> >
> >> >On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
> >> >> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
> >> >> controller of Allwinner H3 SoC.
> >> >> 
> >> >> Enable the HDMI output in Banana Pi M2 Zero device tree.
> >> >> 
> >> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> >> >> ---
> >> >>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
> >> >+++++++++++++++++++
> >> >>  1 file changed, 25 insertions(+)
> >> >> 
> >> >> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> index 7d01f9322658..eb61dcf32797 100644
> >> >> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> @@ -26,6 +26,17 @@
> >> >>  		stdout-path = "serial0:115200n8";
> >> >>  	};
> >> >>  
> >> >> +	hdmi-connector {
> >> >> +		compatible = "hdmi-connector";
> >> >> +		type = "c";
> >> >
> >> >This is not one of the connector type declared in DRM, how is it
> >> >exposed to the userspace?
> >> 
> >> So just use "A" here (because of single link)?
> >
> >If the connector type is C, it should be C, and DRM / driver adjusted
> >to handle it.
> 
> Okay I won't change this property, if there's any further revisions.
> 
> For DRM driver, I think it can be an independent patchset.

Not really, since it's going to break userspace as soon as you merge
that other patchset.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-12 14:53           ` Maxime Ripard
@ 2018-07-13  2:28             ` Icenowy Zheng
  2018-07-13  7:45               ` Maxime Ripard
  0 siblings, 1 reply; 12+ messages in thread
From: Icenowy Zheng @ 2018-07-13  2:28 UTC (permalink / raw)
  To: linux-arm-kernel, Maxime Ripard
  Cc: devicetree, Chen-Yu Tsai, linux-sunxi, linux-kernel



于 2018年7月12日 GMT+08:00 下午10:53:06, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
>On Thu, Jul 12, 2018 at 03:09:55PM +0800, Icenowy Zheng wrote:
>> 于 2018年7月12日 GMT+08:00 下午2:46:01, Maxime Ripard
><maxime.ripard@bootlin.com> 写到:
>> >On Wed, Jul 11, 2018 at 11:15:50PM +0800, Icenowy Zheng wrote:
>> >> 
>> >> 
>> >> 于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard
>> ><maxime.ripard@bootlin.com> 写到:
>> >> >Hi,
>> >> >
>> >> >On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
>> >> >> Banana Pi M2 Zero board has a miniHDMI port connected to the
>HDMI
>> >> >> controller of Allwinner H3 SoC.
>> >> >> 
>> >> >> Enable the HDMI output in Banana Pi M2 Zero device tree.
>> >> >> 
>> >> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>> >> >> ---
>> >> >>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
>> >> >+++++++++++++++++++
>> >> >>  1 file changed, 25 insertions(+)
>> >> >> 
>> >> >> diff --git
>a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> >b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> >> index 7d01f9322658..eb61dcf32797 100644
>> >> >> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> >> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
>> >> >> @@ -26,6 +26,17 @@
>> >> >>  		stdout-path = "serial0:115200n8";
>> >> >>  	};
>> >> >>  
>> >> >> +	hdmi-connector {
>> >> >> +		compatible = "hdmi-connector";
>> >> >> +		type = "c";
>> >> >
>> >> >This is not one of the connector type declared in DRM, how is it
>> >> >exposed to the userspace?
>> >> 
>> >> So just use "A" here (because of single link)?
>> >
>> >If the connector type is C, it should be C, and DRM / driver
>adjusted
>> >to handle it.
>> 
>> Okay I won't change this property, if there's any further revisions.
>> 
>> For DRM driver, I think it can be an independent patchset.
>
>Not really, since it's going to break userspace as soon as you merge
>that other patchset.

Should we bother the DRM maintainer to ask for any advice?

>
>Maxime
>
>-- 
>Maxime Ripard, Bootlin (formerly Free Electrons)
>Embedded Linux and Kernel engineering
>https://bootlin.com
>
>_______________________________________________
>linux-arm-kernel mailing list
>linux-arm-kernel@lists.infradead.org
>http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-13  2:28             ` Icenowy Zheng
@ 2018-07-13  7:45               ` Maxime Ripard
  0 siblings, 0 replies; 12+ messages in thread
From: Maxime Ripard @ 2018-07-13  7:45 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: linux-arm-kernel, devicetree, Chen-Yu Tsai, linux-sunxi, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 2423 bytes --]

On Fri, Jul 13, 2018 at 10:28:08AM +0800, Icenowy Zheng wrote:
> 
> 
> 于 2018年7月12日 GMT+08:00 下午10:53:06, Maxime Ripard <maxime.ripard@bootlin.com> 写到:
> >On Thu, Jul 12, 2018 at 03:09:55PM +0800, Icenowy Zheng wrote:
> >> 于 2018年7月12日 GMT+08:00 下午2:46:01, Maxime Ripard
> ><maxime.ripard@bootlin.com> 写到:
> >> >On Wed, Jul 11, 2018 at 11:15:50PM +0800, Icenowy Zheng wrote:
> >> >> 
> >> >> 
> >> >> 于 2018年7月11日 GMT+08:00 下午11:05:32, Maxime Ripard
> >> ><maxime.ripard@bootlin.com> 写到:
> >> >> >Hi,
> >> >> >
> >> >> >On Wed, Jul 11, 2018 at 09:22:32PM +0800, Icenowy Zheng wrote:
> >> >> >> Banana Pi M2 Zero board has a miniHDMI port connected to the
> >HDMI
> >> >> >> controller of Allwinner H3 SoC.
> >> >> >> 
> >> >> >> Enable the HDMI output in Banana Pi M2 Zero device tree.
> >> >> >> 
> >> >> >> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> >> >> >> ---
> >> >> >>  .../dts/sun8i-h2-plus-bananapi-m2-zero.dts    | 25
> >> >> >+++++++++++++++++++
> >> >> >>  1 file changed, 25 insertions(+)
> >> >> >> 
> >> >> >> diff --git
> >a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> >b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> >> index 7d01f9322658..eb61dcf32797 100644
> >> >> >> --- a/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> >> +++ b/arch/arm/boot/dts/sun8i-h2-plus-bananapi-m2-zero.dts
> >> >> >> @@ -26,6 +26,17 @@
> >> >> >>  		stdout-path = "serial0:115200n8";
> >> >> >>  	};
> >> >> >>  
> >> >> >> +	hdmi-connector {
> >> >> >> +		compatible = "hdmi-connector";
> >> >> >> +		type = "c";
> >> >> >
> >> >> >This is not one of the connector type declared in DRM, how is it
> >> >> >exposed to the userspace?
> >> >> 
> >> >> So just use "A" here (because of single link)?
> >> >
> >> >If the connector type is C, it should be C, and DRM / driver
> >adjusted
> >> >to handle it.
> >> 
> >> Okay I won't change this property, if there's any further revisions.
> >> 
> >> For DRM driver, I think it can be an independent patchset.
> >
> >Not really, since it's going to break userspace as soon as you merge
> >that other patchset.
> 
> Should we bother the DRM maintainer to ask for any advice?

You can if you want.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [linux-sunxi] [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero
  2018-07-11 13:22 ` [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero Icenowy Zheng
  2018-07-11 15:05   ` Maxime Ripard
@ 2018-07-18 22:13   ` Julian Calaby
  1 sibling, 0 replies; 12+ messages in thread
From: Julian Calaby @ 2018-07-18 22:13 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Maxime Ripard, Chen-Yu Tsai, devicetree, Mailing List, Arm,
	linux-kernel, linux-sunxi

Hi Icenowy,

On Wed, Jul 11, 2018 at 11:25 PM Icenowy Zheng <icenowy@aosc.io> wrote:
>
> Banana Pi M2 Zero board has a miniHDMI port connected to the HDMI
> controller of Allwinner H3 SoC.
>
> Enable the HDMI output in Banana Pi M2 Zero device tree.
>
> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>

This is identical to the patch I'd cooked up to do this, so this is:

Tested-by: Julian Calaby <julian.calaby@gmail.com>

Thanks,

-- 
Julian Calaby

Email: julian.calaby@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2018-07-18 22:13 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-11 13:22 [PATCH 0/2] Misc function enhance to Banana Pi M2 Zero board Icenowy Zheng
2018-07-11 13:22 ` [PATCH 1/2] ARM: sun8i: h3: enable HDMI output on Banana Pi M2 Zero Icenowy Zheng
2018-07-11 15:05   ` Maxime Ripard
2018-07-11 15:14     ` [linux-sunxi] " Jernej Škrabec
2018-07-11 15:19       ` Icenowy Zheng
2018-07-11 15:15     ` Icenowy Zheng
2018-07-12  6:46       ` Maxime Ripard
2018-07-12  7:09         ` Icenowy Zheng
2018-07-12 14:53           ` Maxime Ripard
2018-07-13  2:28             ` Icenowy Zheng
2018-07-13  7:45               ` Maxime Ripard
2018-07-18 22:13   ` [linux-sunxi] " Julian Calaby

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).