On Wed, 2015-09-02 at 12:23 -0500, Andy Fleming wrote:
> On Wed, Sep 2, 2015 at 11:53 AM, Scott Wood <scottw...@freescale.com> wrote:
> >
> > On Wed, 2015-09-02 at 01:36 -0500, Andy Fleming wrote:
> > > This board uses a P5020 or P5040 chip, and boots just fine using
> > > the corenet_generic code. The device tree is very similar to the
> > > P5020DS, except that there is no Flash memory. The environment is,
> > > instead, stored on an MMC card on the motherboard.
> > >
> > > Signed-off-by: Andy Fleming <aflem...@gmail.com>
> > > ---
> > > arch/powerpc/boot/dts/cyrus.dts | 175
> > > ++++++++++++++++++++++++++
> > > arch/powerpc/platforms/85xx/corenet_generic.c | 1 +
> > > 2 files changed, 176 insertions(+)
> > > create mode 100644 arch/powerpc/boot/dts/cyrus.dts
> > >
> > > diff --git a/arch/powerpc/boot/dts/cyrus.dts
> > > b/arch/powerpc/boot/dts/cyrus.dts
> > > new file mode 100644
> > > index 0000000..07fe509
> > > --- /dev/null
> > > +++ b/arch/powerpc/boot/dts/cyrus.dts
> > > @@ -0,0 +1,175 @@
> > > +/*
> > > + * P5020DS Device Tree Source
> > > + *
> > > + * Copyright 2010 - 2014 Freescale Semiconductor Inc.
> > > + *
> > > + * Redistribution and use in source and binary forms, with or without
> > > + * modification, are permitted provided that the following conditions
> > > are
> > > met:
> > > + * * Redistributions of source code must retain the above copyright
> > > + * notice, this list of conditions and the following disclaimer.
> > > + * * Redistributions in binary form must reproduce the above
> > > copyright
> > > + * notice, this list of conditions and the following disclaimer
> > > in
> > > the
> > > + * documentation and/or other materials provided with the
> > > distribution.
> > > + * * Neither the name of Freescale Semiconductor nor the
> > > + * names of its contributors may be used to endorse or promote
> > > products
> > > + * derived from this software without specific prior written
> > > permission.
> > > + *
> > > + *
> > > + * ALTERNATIVELY, this software may be distributed under the terms of
> > > the
> > > + * GNU General Public License ("GPL") as published by the Free Software
> > > + * Foundation, either version 2 of that License or (at your option) any
> > > + * later version.
> > > + *
> > > + * THIS SOFTWARE IS PROVIDED BY Freescale Semiconductor ``AS IS'' AND
> > > ANY
> > > + * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> > > IMPLIED
> > > + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> > > ARE
> > > + * DISCLAIMED. IN NO EVENT SHALL Freescale Semiconductor BE LIABLE FOR
> > > ANY
> > > + * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
> > > DAMAGES
> > > + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
> > > SERVICES;
> > > + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
> > > CAUSED
> > > AND
> > > + * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
> > > OR
> > > TORT
> > > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE
> > > USE
> > > OF THIS
> > > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> > > + */
> > > +
> > > +/include/ "fsl/p5020si-pre.dtsi"
> >
> > What happens when the board has a p5040?
>
>
> Hmm...I hadn't thought of that. I can respin the dts to be
> cyrus_5020.dts, and provide a cyrus_5040.dts later. That version of
> the product doesn't exist yet. Does that make sense?
Sure.
> > > diff --git a/arch/powerpc/platforms/85xx/corenet_generic.c
> > > b/arch/powerpc/platforms/85xx/corenet_generic.c
> > > index b395571..76e521e 100644
> > > --- a/arch/powerpc/platforms/85xx/corenet_generic.c
> > > +++ b/arch/powerpc/platforms/85xx/corenet_generic.c
> > > @@ -161,6 +161,7 @@ static const char * const boards[] __initconst = {
> > > "fsl,T1042RDB",
> > > "fsl,T1042RDB_PI",
> > > "keymile,kmcoge4",
> > > + "varisys,CYRUS",
> > > NULL
> >
> > I don't see this compatible in the dts. Why allcaps?
>
> I wondered about the allcaps, too, but there are already systems
> floating around, possibly with dtbs with that compatible in there. I'm
> reluctant to change it.
That's fine. It seemed like the name wasn't in use yet based on the dts. :-)
-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
