Hi Biju,
On Thu, Oct 10, 2019 at 3:04 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote:
> > Subject: Re: [PATCH v3 1/2] arm64: dts: renesas: r8a774a1: Remove audio
> > port node
> > On Tue, Oct 8, 2019 at 3:43 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote:
> > > > Subject: Re: [PATCH v3 1/2] arm64: dts: renesas: r8a774a1: Remove
> > > > audio port node On Fri, Oct 4, 2019 at 4:53 PM Biju Das
> > > > <biju.das@xxxxxxxxxxxxxx> wrote:
> > > > > This patch removes audio port node from SoC device tree.
> > > >
> > > > Standard review comment: "Why?"
> >
> > Bcause it fixes a dtb warning?
> >
> > Warning (unit_address_vs_reg): /soc/sound@ec500000/ports/port@0:
> > node has
> > a unit name, but no reg property
>
> I also noticed this warning.
>
> We will get this warning, if we apply the patch "[v3,2/2] arm64: dts: renesas: r8a774b1: Add Sound and Audio DMAC
> device nodes" first and then this one.
>
> If it is other way around, then there won't be any warning. That is the reason I didn't mention it in the commit message.
I even see the warning with current renesas-devel ("make dtbs W=1"),
which does not have "[v3,2/2] arm64: dts: renesas: r8a774b1: Add Sound
and Audio DMAC device nodes" yet.
So I'll apply this patch with the warning message in the commit log, unless
I'm missing something?
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
