> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Wednesday, July 08, 2015 3:50 AM
> To: Wang Dongsheng-B40534
> Cc: robh...@kernel.org; ijc+devicet...@hellion.org.uk; ga...@codeaurora.org;
> Jain Priyanka-B32167; devicetree@vger.kernel.org
> Subject: Re: [PATCH 1/2] powerpc/85xx: Add binding for SCFG
> 
> On Tue, 2015-07-07 at 17:47 +0800, Dongsheng Wang wrote:
> > From: Wang Dongsheng <dongsheng.w...@freescale.com>
> >
> > SCFG provides SoC specific configuration and status registers for the
> > chip. Add this for powerpc platform.
> >
> > Signed-off-by: Wang Dongsheng <dongsheng.w...@freescale.com>
> >
> > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/board.txt
> > b/Documentation/devicetree/bindings/powerpc/fsl/board.txt
> > index cff38bd..55604fe 100644
> > --- a/Documentation/devicetree/bindings/powerpc/fsl/board.txt
> > +++ b/Documentation/devicetree/bindings/powerpc/fsl/board.txt
> > @@ -3,6 +3,25 @@ Freescale Reference Board Bindings  This document
> > describes device tree bindings for various devices that  exist on some
> > Freescale reference boards.
> >
> > +* Supplement configuration unit (SCFG)
> > +
> > +SCFG is the supplemental configuration unit, that provides SoC
> > +specific configuration and status registers for the chip. Such as
> > +getting PEX port status.
> > +
> > +Required properties:
> > +
> > +- compatible: should be "fsl,<board>-scfg"
> 
> fsl,<chip>-scfg

Thanks.

> > +- reg: should contain base address and length of SCFG memory-mapped
> > +registers
> > +
> > +Example:
> > +
> > +     scfg:  global-utilities@fc000{
> > +             compatible = "fsl,t1040-scfg";
> > +             reg = <0xfc000 0x1000>;
> > +     };
> > +
> 
> I don't think board.txt is the right place for this, since it's part of the 
> SoC,
> not the board.  Put it in its own file.

OK, will put it in Documentation/devicetree/bindings/powerpc/fsl/scfg.txt.

Regards,
-Dongsheng

Reply via email to