Re: [PATCH] arm64: dts: renesas: salvator: add dummy 2nd port to rcar_sound

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux