Re: [PATCH v6 7/7] arm64: dts: NS2: add AMAC ethernet support

[Sergei Shtylyov]

On 11/4/2016 7:30 PM, Jon Mason wrote:

> > > Add support for the AMAC ethernet to the Broadcom Northstar2 SoC device
> > > tree
> > >
> > > Signed-off-by: Jon Mason <jon.mason@xxxxxxxxxxxx>
> > > ---
> > > arch/arm64/boot/dts/broadcom/ns2-svk.dts |  5 +++++
> > > arch/arm64/boot/dts/broadcom/ns2.dtsi    | 12 ++++++++++++
> > > 2 files changed, 17 insertions(+)
> > >
> > > diff --git a/arch/arm64/boot/dts/broadcom/ns2-svk.dts b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> > > index b09f3bc..c4d5442 100644
> > > --- a/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> > > +++ b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> > > @@ -56,6 +56,10 @@
> > > 	};
> > > };
> > >
> > > +&enet {
> > > +	status = "ok";
> >
> >    The spec dictates it should be "okay" (although "ok" is also recognized).
>
> The rest of the file uses "ok".  So, the addition above is consistent
> with the other entries.
>
> Perhaps a patch outside this series to convert the entire file from
> "ok" to "okay" would be acceptable to you.

   OK, it would...

> Thanks,
> Jon

MBR, Sergei