Woops, this time using the Real Rob. On Mon, Feb 25, 2019 at 10:49 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > CC DT > > On Mon, Feb 25, 2019 at 10:37 AM Simon Horman > <horms+renesas@xxxxxxxxxxxx> wrote: > > Add dummy 2nd port node to rcar_sound ports node to satisfy dtc > > recommendation that #address-cells / #size-cells is only needed. > > > > This addresses the following warning: > > > > arch/arm64/boot/dts/renesas/salvator-common.dtsi:703.8-719.4: Warning (graph_child_address): /soc/sound@ec500000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary > > arch/arm64/boot/dts/renesas/salvator-common.dtsi:703.8-719.4: Warning (graph_child_address): /soc/sound@ec500000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary > > > > This change should not have any run-time effect. > > > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > > > > --- > > Build tested against renesas-devel-20190214-v5.0-rc6 > > --- > > arch/arm64/boot/dts/renesas/salvator-common.dtsi | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > > index a225c2457274..20b35991bc17 100644 > > --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > > +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > > @@ -716,6 +716,15 @@ > > capture = <&ssi1 &src1 &dvc1>; > > }; > > }; > > + > > + /* Dummy node to be overriden by board DT > > + * Satisfies dtc recommendation that > > + * #address-cells / #size-cells is only needed > > + * for graph nodes with multiple child nodes. > > + */ > > + rsnd_port1: port@1 { > > + reg = <1>; > > + }; > > }; > > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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