* [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support [not found] <cover.90fc3153078e79a3f74af832de923ac675eb29ad.1497469745.git-series.kieran.bingham+renesas@ideasonboard.com> @ 2017-06-14 19:58 ` Kieran Bingham 2017-06-23 7:43 ` Laurent Pinchart 0 siblings, 1 reply; 4+ messages in thread From: Kieran Bingham @ 2017-06-14 19:58 UTC (permalink / raw) To: linux-arm-kernel From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> The Salvator boards use an ADV7482 receiver for HDMI and CVBS inputs. Provide ADV7482 node on the i2c4 bus, along with connectors for the hdmi and cvbs inputs, and link to the csi20 and csi40 nodes as outputs. Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> v4: - dt: Rebase to dts/renesas/salvator-x.dtsi - dt: Use AIN0-7 rather than AIN1-8 v5: - dt: Move to salvator-common.dtsi --- arch/arm64/boot/dts/renesas/salvator-common.dtsi | 123 ++++++++++++++++- 1 file changed, 123 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index aef35e0b685a..d74fb55c38bc 100644 --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi @@ -65,6 +65,27 @@ enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>; }; + cvbs-in { + compatible = "composite-video-connector"; + label = "CVBS IN"; + + port { + cvbs_con: endpoint { + }; + }; + }; + + hdmi-in { + compatible = "hdmi-connector"; + label = "HDMI IN"; + type = "a"; + + port { + hdmi_in_con: endpoint { + }; + }; + }; + reg_1p8v: regulator0 { compatible = "regulator-fixed"; regulator-name = "fixed-1.8V"; @@ -257,6 +278,59 @@ }; }; +&csi20 { + status = "okay"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port at 0 { + reg = <0>; + csi20_in: endpoint { + clock-lanes = <0>; + data-lanes = <1>; + remote-endpoint = <&adv7482_txb>; + }; + }; + }; +}; + +&csi40 { + status = "okay"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port at 0 { + reg = <0>; + + csi40_in: endpoint { + clock-lanes = <0>; + data-lanes = <1 2 3 4>; + remote-endpoint = <&adv7482_txa>; + }; + }; + }; +}; + +&cvbs_con { + port { + cvbs_in: endpoint { + remote-endpoint = <&adv7482_ain7>; + }; + }; +}; + +&hdmi_in_con { + port { + hdmi_in: endpoint { + remote-endpoint = <&adv7482_hdmi>; + }; + }; +}; + &du { pinctrl-0 = <&du_pins>; pinctrl-names = "default"; @@ -343,6 +417,55 @@ shunt-resistor-micro-ohms = <5000>; }; + + video-receiver at 70 { + compatible = "adi,adv7482"; + reg = <0x70>; + + #address-cells = <1>; + #size-cells = <0>; + + interrupt-parent = <&gpio6>; + interrupt-names = "intrq1", "intrq2"; + interrupts = <30 IRQ_TYPE_LEVEL_LOW>, + <31 IRQ_TYPE_LEVEL_LOW>; + + port at 7 { + reg = <7>; + + adv7482_ain7: endpoint { + remote-endpoint = <&cvbs_in>; + }; + }; + + port at 8 { + reg = <8>; + + adv7482_hdmi: endpoint { + remote-endpoint = <&hdmi_in>; + }; + }; + + port at 10 { + reg = <10>; + + adv7482_txa: endpoint { + clock-lanes = <0>; + data-lanes = <1 2 3 4>; + remote-endpoint = <&csi40_in>; + }; + }; + + port at 11 { + reg = <11>; + + adv7482_txb: endpoint { + clock-lanes = <0>; + data-lanes = <1>; + remote-endpoint = <&csi20_in>; + }; + }; + }; }; &i2c_dvfs { -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support 2017-06-14 19:58 ` [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support Kieran Bingham @ 2017-06-23 7:43 ` Laurent Pinchart 2017-06-27 9:25 ` Kieran Bingham 0 siblings, 1 reply; 4+ messages in thread From: Laurent Pinchart @ 2017-06-23 7:43 UTC (permalink / raw) To: linux-arm-kernel Hi Kieran, Thank you for the patch. On Wednesday 14 Jun 2017 20:58:13 Kieran Bingham wrote: > From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > The Salvator boards use an ADV7482 receiver for HDMI and CVBS inputs. > > Provide ADV7482 node on the i2c4 bus, along with connectors for the > hdmi and cvbs inputs, and link to the csi20 and csi40 nodes as outputs. > > Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > v4: > - dt: Rebase to dts/renesas/salvator-x.dtsi > - dt: Use AIN0-7 rather than AIN1-8 > > v5: > - dt: Move to salvator-common.dtsi > --- > arch/arm64/boot/dts/renesas/salvator-common.dtsi | 123 ++++++++++++++++- > 1 file changed, 123 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index > aef35e0b685a..d74fb55c38bc 100644 > --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > @@ -65,6 +65,27 @@ > enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>; > }; > > + cvbs-in { > + compatible = "composite-video-connector"; > + label = "CVBS IN"; > + > + port { > + cvbs_con: endpoint { > + }; > + }; > + }; > + > + hdmi-in { > + compatible = "hdmi-connector"; > + label = "HDMI IN"; > + type = "a"; > + > + port { > + hdmi_in_con: endpoint { > + }; > + }; > + }; > + > reg_1p8v: regulator0 { > compatible = "regulator-fixed"; > regulator-name = "fixed-1.8V"; > @@ -257,6 +278,59 @@ > }; > }; > > +&csi20 { > + status = "okay"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port at 0 { > + reg = <0>; Aren't #address-cells, #size-cells and reg already present in the SoC .dtsi ? And if they're not, shouldn't they be ? :-) > + csi20_in: endpoint { > + clock-lanes = <0>; > + data-lanes = <1>; > + remote-endpoint = <&adv7482_txb>; > + }; > + }; > + }; > +}; > + > +&csi40 { > + status = "okay"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port at 0 { > + reg = <0>; > + > + csi40_in: endpoint { > + clock-lanes = <0>; > + data-lanes = <1 2 3 4>; > + remote-endpoint = <&adv7482_txa>; > + }; > + }; > + }; > +}; > + > +&cvbs_con { > + port { > + cvbs_in: endpoint { > + remote-endpoint = <&adv7482_ain7>; > + }; > + }; > +}; You could merge this with the cvbs_con node above. Apart from that the patch looks good to me. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > +&hdmi_in_con { > + port { > + hdmi_in: endpoint { > + remote-endpoint = <&adv7482_hdmi>; > + }; > + }; > +}; > + > &du { > pinctrl-0 = <&du_pins>; > pinctrl-names = "default"; > @@ -343,6 +417,55 @@ > > shunt-resistor-micro-ohms = <5000>; > }; > + > + video-receiver at 70 { > + compatible = "adi,adv7482"; > + reg = <0x70>; > + > + #address-cells = <1>; > + #size-cells = <0>; > + > + interrupt-parent = <&gpio6>; > + interrupt-names = "intrq1", "intrq2"; > + interrupts = <30 IRQ_TYPE_LEVEL_LOW>, > + <31 IRQ_TYPE_LEVEL_LOW>; > + > + port at 7 { > + reg = <7>; > + > + adv7482_ain7: endpoint { > + remote-endpoint = <&cvbs_in>; > + }; > + }; > + > + port at 8 { > + reg = <8>; > + > + adv7482_hdmi: endpoint { > + remote-endpoint = <&hdmi_in>; > + }; > + }; > + > + port at 10 { > + reg = <10>; > + > + adv7482_txa: endpoint { > + clock-lanes = <0>; > + data-lanes = <1 2 3 4>; > + remote-endpoint = <&csi40_in>; > + }; > + }; > + > + port at 11 { > + reg = <11>; > + > + adv7482_txb: endpoint { > + clock-lanes = <0>; > + data-lanes = <1>; > + remote-endpoint = <&csi20_in>; > + }; > + }; > + }; > }; > > &i2c_dvfs { -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support 2017-06-23 7:43 ` Laurent Pinchart @ 2017-06-27 9:25 ` Kieran Bingham 2017-06-27 13:57 ` Laurent Pinchart 0 siblings, 1 reply; 4+ messages in thread From: Kieran Bingham @ 2017-06-27 9:25 UTC (permalink / raw) To: linux-arm-kernel Hi Laurent, Thanks for the review, On 23/06/17 08:43, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Wednesday 14 Jun 2017 20:58:13 Kieran Bingham wrote: >> From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> >> >> The Salvator boards use an ADV7482 receiver for HDMI and CVBS inputs. >> >> Provide ADV7482 node on the i2c4 bus, along with connectors for the >> hdmi and cvbs inputs, and link to the csi20 and csi40 nodes as outputs. >> >> Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> >> >> v4: >> - dt: Rebase to dts/renesas/salvator-x.dtsi >> - dt: Use AIN0-7 rather than AIN1-8 >> >> v5: >> - dt: Move to salvator-common.dtsi >> --- >> arch/arm64/boot/dts/renesas/salvator-common.dtsi | 123 ++++++++++++++++- >> 1 file changed, 123 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi >> b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index >> aef35e0b685a..d74fb55c38bc 100644 >> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi >> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi >> @@ -65,6 +65,27 @@ >> enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>; >> }; >> >> + cvbs-in { >> + compatible = "composite-video-connector"; >> + label = "CVBS IN"; >> + >> + port { >> + cvbs_con: endpoint { >> + }; >> + }; >> + }; >> + >> + hdmi-in { >> + compatible = "hdmi-connector"; >> + label = "HDMI IN"; >> + type = "a"; >> + >> + port { >> + hdmi_in_con: endpoint { >> + }; >> + }; >> + }; >> + >> reg_1p8v: regulator0 { >> compatible = "regulator-fixed"; >> regulator-name = "fixed-1.8V"; >> @@ -257,6 +278,59 @@ >> }; >> }; >> >> +&csi20 { >> + status = "okay"; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port at 0 { >> + reg = <0>; > > Aren't #address-cells, #size-cells and reg already present in the SoC .dtsi ? > And if they're not, shouldn't they be ? :-) Hrm - I think I added them because I was getting warnings from the DTC. I've removed them from the csi nodes and no warning returns :) So they're gone. > >> + csi20_in: endpoint { >> + clock-lanes = <0>; >> + data-lanes = <1>; >> + remote-endpoint = <&adv7482_txb>; >> + }; >> + }; >> + }; >> +}; >> + >> +&csi40 { >> + status = "okay"; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; # Removed >> + >> + port at 0 { >> + reg = <0>; >> + >> + csi40_in: endpoint { >> + clock-lanes = <0>; >> + data-lanes = <1 2 3 4>; >> + remote-endpoint = <&adv7482_txa>; >> + }; >> + }; >> + }; >> +}; >> + >> +&cvbs_con { >> + port { >> + cvbs_in: endpoint { >> + remote-endpoint = <&adv7482_ain7>; >> + }; >> + }; >> +}; > > You could merge this with the cvbs_con node above. What is the distinction that means cvbs_con can be merged where hdmi_in_con would not ? Similarly, I see that VGA out is 'merged' where hdmi outs are not ... is it just determined by whether the connector is on all boards that inherit salvator-common.dtsi ? > > Apart from that the patch looks good to me. > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > >> +&hdmi_in_con { >> + port { >> + hdmi_in: endpoint { >> + remote-endpoint = <&adv7482_hdmi>; >> + }; >> + }; >> +}; >> + >> &du { >> pinctrl-0 = <&du_pins>; >> pinctrl-names = "default"; >> @@ -343,6 +417,55 @@ >> >> shunt-resistor-micro-ohms = <5000>; >> }; >> + >> + video-receiver at 70 { >> + compatible = "adi,adv7482"; >> + reg = <0x70>; >> + >> + #address-cells = <1>; >> + #size-cells = <0>; But I can't remove these # without generating a warning message ... >> + >> + interrupt-parent = <&gpio6>; >> + interrupt-names = "intrq1", "intrq2"; >> + interrupts = <30 IRQ_TYPE_LEVEL_LOW>, >> + <31 IRQ_TYPE_LEVEL_LOW>; >> + >> + port at 7 { >> + reg = <7>; >> + >> + adv7482_ain7: endpoint { >> + remote-endpoint = <&cvbs_in>; >> + }; >> + }; >> + >> + port at 8 { >> + reg = <8>; >> + >> + adv7482_hdmi: endpoint { >> + remote-endpoint = <&hdmi_in>; >> + }; >> + }; >> + >> + port at 10 { >> + reg = <10>; >> + >> + adv7482_txa: endpoint { >> + clock-lanes = <0>; >> + data-lanes = <1 2 3 4>; >> + remote-endpoint = <&csi40_in>; >> + }; >> + }; >> + >> + port at 11 { >> + reg = <11>; >> + >> + adv7482_txb: endpoint { >> + clock-lanes = <0>; >> + data-lanes = <1>; >> + remote-endpoint = <&csi20_in>; >> + }; >> + }; >> + }; >> }; >> >> &i2c_dvfs { > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support 2017-06-27 9:25 ` Kieran Bingham @ 2017-06-27 13:57 ` Laurent Pinchart 0 siblings, 0 replies; 4+ messages in thread From: Laurent Pinchart @ 2017-06-27 13:57 UTC (permalink / raw) To: linux-arm-kernel Hi Kieran, On Tuesday 27 Jun 2017 10:25:39 Kieran Bingham wrote: > On 23/06/17 08:43, Laurent Pinchart wrote: > > On Wednesday 14 Jun 2017 20:58:13 Kieran Bingham wrote: > >> From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > >> > >> The Salvator boards use an ADV7482 receiver for HDMI and CVBS inputs. > >> > >> Provide ADV7482 node on the i2c4 bus, along with connectors for the > >> hdmi and cvbs inputs, and link to the csi20 and csi40 nodes as outputs. > >> > >> Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > >> > >> v4: > >> - dt: Rebase to dts/renesas/salvator-x.dtsi > >> - dt: Use AIN0-7 rather than AIN1-8 > >> > >> v5: > >> - dt: Move to salvator-common.dtsi > >> > >> --- > >> > >> arch/arm64/boot/dts/renesas/salvator-common.dtsi | 123 ++++++++++++++++- > >> 1 file changed, 123 insertions(+) > >> > >> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > >> b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index > >> aef35e0b685a..d74fb55c38bc 100644 > >> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > >> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > >> @@ -65,6 +65,27 @@ > >> enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>; > >> }; > >> > >> + cvbs-in { > >> + compatible = "composite-video-connector"; > >> + label = "CVBS IN"; > >> + > >> + port { > >> + cvbs_con: endpoint { > >> + }; > >> + }; > >> + }; > >> + > >> + hdmi-in { > >> + compatible = "hdmi-connector"; > >> + label = "HDMI IN"; > >> + type = "a"; > >> + > >> + port { > >> + hdmi_in_con: endpoint { > >> + }; > >> + }; > >> + }; > >> + > >> reg_1p8v: regulator0 { > >> compatible = "regulator-fixed"; > >> regulator-name = "fixed-1.8V"; > >> @@ -257,6 +278,59 @@ > >> }; > >> }; > >> > >> +&csi20 { > >> + status = "okay"; > >> + > >> + ports { > >> + #address-cells = <1>; > >> + #size-cells = <0>; > >> + > >> + port at 0 { > >> + reg = <0>; > > > > Aren't #address-cells, #size-cells and reg already present in the SoC > > .dtsi ? And if they're not, shouldn't they be ? :-) > > Hrm - I think I added them because I was getting warnings from the DTC. > > I've removed them from the csi nodes and no warning returns :) > So they're gone. You need the #address-cells and #size-cells properties in the ports node, but as they're already added by the SoC .dtsi, you don't need to duplicate them here. > >> + csi20_in: endpoint { > >> + clock-lanes = <0>; > >> + data-lanes = <1>; > >> + remote-endpoint = <&adv7482_txb>; > >> + }; > >> + }; > >> + }; > >> +}; > >> + > >> +&csi40 { > >> + status = "okay"; > >> + > >> + ports { > >> + #address-cells = <1>; > >> + #size-cells = <0>; > > # Removed > > >> + > >> + port at 0 { > >> + reg = <0>; > >> + > >> + csi40_in: endpoint { > >> + clock-lanes = <0>; > >> + data-lanes = <1 2 3 4>; > >> + remote-endpoint = <&adv7482_txa>; > >> + }; > >> + }; > >> + }; > >> +}; > >> + > >> +&cvbs_con { > >> + port { > >> + cvbs_in: endpoint { > >> + remote-endpoint = <&adv7482_ain7>; > >> + }; > >> + }; > >> +}; > > > > You could merge this with the cvbs_con node above. > > What is the distinction that means cvbs_con can be merged where hdmi_in_con > would not ? No reason, the same comment applies to HDMI too :-) > Similarly, I see that VGA out is 'merged' where hdmi outs are not ... is it > just determined by whether the connector is on all boards that inherit > salvator-common.dtsi ? My point was that it seems a bit pointless to define a node and extend it with additional properties in the same file, when you could merge those properties in the node definition. You only need to extend nodes with additional properties when they're defined in one file and extended in another one. > > Apart from that the patch looks good to me. > > > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > >> +&hdmi_in_con { > >> + port { > >> + hdmi_in: endpoint { > >> + remote-endpoint = <&adv7482_hdmi>; > >> + }; > >> + }; > >> +}; > >> + > >> &du { > >> pinctrl-0 = <&du_pins>; > >> pinctrl-names = "default"; > >> @@ -343,6 +417,55 @@ > >> > >> shunt-resistor-micro-ohms = <5000>; > >> }; > >> + > >> + video-receiver at 70 { > >> + compatible = "adi,adv7482"; > >> + reg = <0x70>; > >> + > >> + #address-cells = <1>; > >> + #size-cells = <0>; > > But I can't remove these # without generating a warning message ... The properties are needed, why should you remove them ? :-) > >> + > >> + interrupt-parent = <&gpio6>; > >> + interrupt-names = "intrq1", "intrq2"; > >> + interrupts = <30 IRQ_TYPE_LEVEL_LOW>, > >> + <31 IRQ_TYPE_LEVEL_LOW>; > >> + > >> + port at 7 { > >> + reg = <7>; > >> + > >> + adv7482_ain7: endpoint { > >> + remote-endpoint = <&cvbs_in>; > >> + }; > >> + }; > >> + > >> + port at 8 { > >> + reg = <8>; > >> + > >> + adv7482_hdmi: endpoint { > >> + remote-endpoint = <&hdmi_in>; > >> + }; > >> + }; > >> + > >> + port at 10 { > >> + reg = <10>; > >> + > >> + adv7482_txa: endpoint { > >> + clock-lanes = <0>; > >> + data-lanes = <1 2 3 4>; > >> + remote-endpoint = <&csi40_in>; > >> + }; > >> + }; > >> + > >> + port at 11 { > >> + reg = <11>; > >> + > >> + adv7482_txb: endpoint { > >> + clock-lanes = <0>; > >> + data-lanes = <1>; > >> + remote-endpoint = <&csi20_in>; > >> + }; > >> + }; > >> + }; > >> }; > >> > >> &i2c_dvfs { -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-06-27 13:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.90fc3153078e79a3f74af832de923ac675eb29ad.1497469745.git-series.kieran.bingham+renesas@ideasonboard.com>
2017-06-14 19:58 ` [PATCH v5 2/2] arm64: dts: renesas: salvator-x: Add ADV7482 support Kieran Bingham
2017-06-23 7:43 ` Laurent Pinchart
2017-06-27 9:25 ` Kieran Bingham
2017-06-27 13:57 ` Laurent Pinchart
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).