Re: [PATCH 1/2] net: dsa: b53: Add compatible strings for the Cygnus-family BCM11360.

[Scott Branden]

On 17-04-24 04:54 PM, Eric Anholt wrote:
> Scott Branden <scott.branden@xxxxxxxxxxxx> writes:
>
> > Minor comment in line
> >
> > On 17-04-24 02:50 PM, Eric Anholt wrote:
> > > Cygnus is a small family of SoCs, of which we currently have
> > > devicetree for BCM11360 and BCM58300.  The 11360's B53 is mostly the
> > > same as 58xx, just requiring a tiny bit of setup that was previously
> > > missing.
> > >
> > > Signed-off-by: Eric Anholt <eric@xxxxxxxxxx>
> > > ---
> > >  Documentation/devicetree/bindings/net/dsa/b53.txt | 3 +++
> > >  drivers/net/dsa/b53/b53_srab.c                    | 2 ++
> > >  2 files changed, 5 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/net/dsa/b53.txt b/Documentation/devicetree/bindings/net/dsa/b53.txt
> > > index d6c6e41648d4..49c93d3c0839 100644
> > > --- a/Documentation/devicetree/bindings/net/dsa/b53.txt
> > > +++ b/Documentation/devicetree/bindings/net/dsa/b53.txt
> > > @@ -29,6 +29,9 @@ Required properties:
> > >        "brcm,bcm58625-srab"
> > >        "brcm,bcm88312-srab" and the mandatory "brcm,nsp-srab string
> > >
> > > +  For the BCM11360 SoC, must be:
> > > +      "brcm,bcm11360-srab" and the mandatory "brcm,cygnus-srab string
> > > +
> > place in alphabetical order in the doc?
>
> Moved it above BCM5310x now.  I hope that's what you meant.
Yes, sorry for not being more clear.