Re: [PATCH] arm64: dts: renesas: r8a77965: Add second port to rcar_sound placeholder

2018-06-29 Thread Simon Horman
On Thu, Jun 28, 2018 at 03:33:59PM +0200, Geert Uytterhoeven wrote:
> On Thu, Jun 28, 2018 at 2:43 PM Simon Horman  
> wrote:
> > This node is just a placeholder but fills that function better if it does
> > not trigger build warnings. This update satisfies the requirement that
> > nodes with #address-cells/#size-cells properties have more than one child
> > node.
> >
> > This is flagged by dtc as follows:
> >  # make dtbs W=1
> >  arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dtb: Warning 
> > (graph_child_address): /soc/sound@ec50/ports: graph node has single 
> > child node 'port@0', #address-cells/#size-cells are not necessary
> >  arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dtb: Warning 
> > (graph_child_address): /soc/sound@ec50/ports: graph node has single 
> > child node 'port@0', #address-cells/#size-cells are not necessary
> >
> > Signed-off-by: Simon Horman 
> 
> Reviewed-by: Geert Uytterhoeven 

Thanks, applied.


Re: [PATCH] arm64: dts: renesas: r8a77965: Add second port to rcar_sound placeholder

2018-06-28 Thread Geert Uytterhoeven
On Thu, Jun 28, 2018 at 2:43 PM Simon Horman  wrote:
> This node is just a placeholder but fills that function better if it does
> not trigger build warnings. This update satisfies the requirement that
> nodes with #address-cells/#size-cells properties have more than one child
> node.
>
> This is flagged by dtc as follows:
>  # make dtbs W=1
>  arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dtb: Warning 
> (graph_child_address): /soc/sound@ec50/ports: graph node has single child 
> node 'port@0', #address-cells/#size-cells are not necessary
>  arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dtb: Warning 
> (graph_child_address): /soc/sound@ec50/ports: graph node has single child 
> node 'port@0', #address-cells/#size-cells are not necessary
>
> Signed-off-by: Simon Horman 

Reviewed-by: Geert Uytterhoeven 

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


[PATCH] arm64: dts: renesas: r8a77965: Add second port to rcar_sound placeholder

2018-06-28 Thread Simon Horman
This node is just a placeholder but fills that function better if it does
not trigger build warnings. This update satisfies the requirement that
nodes with #address-cells/#size-cells properties have more than one child
node.

This is flagged by dtc as follows:
 # make dtbs W=1
 arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dtb: Warning 
(graph_child_address): /soc/sound@ec50/ports: graph node has single child 
node 'port@0', #address-cells/#size-cells are not necessary
 arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dtb: Warning 
(graph_child_address): /soc/sound@ec50/ports: graph node has single child 
node 'port@0', #address-cells/#size-cells are not necessary

Signed-off-by: Simon Horman 
---
Based on renesas-devel-20180628-v4.18-rc2
---
 arch/arm64/boot/dts/renesas/r8a77965.dtsi | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi 
b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
index be8631dff1b4..a37749caf40e 100644
--- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
@@ -1269,6 +1269,9 @@
port@0 {
reg = <0>;
};
+   port@1 {
+   reg = <1>;
+   };
};
};
 
-- 
2.11.0