[PATCH] powerpc/mpc52xx: add Phytec phyCORE-MPC5200B-IO board (pcm032)

[Grant Likely]

On Sun, Mar 1, 2009 at 8:54 AM, Jon Smirl <jonsmirl at gmail.com> wrote:
> On Sun, Mar 1, 2009 at 9:48 AM, Grant Likely <grant.likely at secretlab.ca> wrote:
>> On Sun, Mar 1, 2009 at 3:18 AM, Wolfram Sang <w.sang at pengutronix.de> wrote:
>>> Hi Grant,
>>>
>>> thanks for the review!
>>>
>>> On Thu, Feb 26, 2009 at 10:31:24PM -0700, Grant Likely wrote:
>>>> Thanks for the patch Wolfram.  Comments below.
>>>>
>>>> On Wed, Feb 25, 2009 at 8:32 AM, Wolfram Sang <w.sang at pengutronix.de> wrote:
>>>> > Signed-off-by: Wolfram Sang <w.sang at pengutronix.de>
>>>> > ---
>>>> >  arch/powerpc/boot/dts/pcm032.dts             |  391 +++++++
>>>> >  arch/powerpc/configs/52xx/pcm032_defconfig   | 1394 ++++++++++++++++++++++++++
>>>>
>>>> Do you really need a separate defconfig for this board?  Can it be
>>>> merged with an existing defconfig?
>>>
>>> Will have a look if I can make a pcm03x_defconfig out of pcm030_defconfig.
>>
>> Cool, thanks.
>>
>>>> motionpro.dts is a good example of what it should look like.
>>>
>>> Ah, thanks for the pointer. Will have a look there. Guess I'll need to update
>>> pcm030.dts later, too (from which I copied this one). Will rewrite the
>>> localbus-node addressing the comments you gave.
>>
>> pcm030.dts in mainline doesn't have a localbus node.
>
> I've been using this on my pcm030 locally.

Hey Jon, a few comments below for the record.  :-)

> pcm030 doesn't have a PCI socket but it can be added via the expansion bus.
> With just the basic pcm030 hardware, if I uncomment the pci node I
> sometimes hang on boot.

Hmmm.  Weird.  Have you been able to identify what code it hangs in?

>
>        lpb at ff000000 {

This node doesn't really have an address since it is simply a bus (the
control registers are elsewhere).  Should make this simply: locabus {

>                compatible = "fsl,lpb", "simple-bus";

Ideally this should be: compatible =
"fsl,mpc5200b-lpb",fsl,mpc5200-lpb","simple-bus".  However, I forgot
to add "simple-bus" to arch/powerpc/platforms/52xx/mpc52xx-common.c
for the bus probing code and so this won't work in current mainline.
The fix is queued up in -next, and I'm going to try and get it into
.29

Cheers,
g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.