Re: [PATCH v2 1/2] dt-bindings: Add new boards to bcm4708 DT bindings

[Scott Branden]

On 15-10-15 01:52 PM, Hauke Mehrtens wrote:
> On 10/15/2015 12:14 AM, Jon Mason wrote:
> > Add the 4708, 4709, and 953012k SVKs to the the documentation for the
> > Broadcom Northstar device tree bindings.
> >
> > Signed-off-by: Jon Mason <jonmason@xxxxxxxxxxxx>
> > ---
> >   Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt | 7 +++++++
> >   1 file changed, 7 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt
> > index 6b0f49f..b9cc308 100644
> > --- a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt
> > +++ b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt
> > @@ -5,4 +5,11 @@ Boards with the BCM4708 SoC shall have the following properties:
> >
> >   Required root node property:
> >
> > +bcm94708
this should be bcm4708
> I thought we wanted to add the SoC names here, isn't bcm94708 a board
> the bcm4708, the SoC ion this board? I do not really care about this
> file, has anyone found the documentation what should be listed here?

Yes - the 9 infront indicates its some type of board.  The SoC part 
numbers should be listed here.  The SoCs are all variants of 
Northstar/Northstar plus family.  Some have different io, packaging, 
speed grades, etc.

> Hauke
>
> >   compatible = "brcm,bcm4708";
> > +
> > +bcm94709
This should be bcm4709
> > +compatible = "brcm,bcm4709";
> > +
> > +bcm953012
This should be bcm53012
> > +compatible = "brcm,bcm53012";

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/