* [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 = <®_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: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: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: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).