[PATCH v2 1/2] powerpc: Board support for GE Fanuc SBC610
Martyn Welch
martyn.welch at gefanuc.com
Fri Aug 22 21:11:52 EST 2008
On Fri, 22 Aug 2008 05:56:53 -0500
Kumar Gala <galak at kernel.crashing.org> wrote:
> >
> > + PowerPC,8641 at 0 {
> > + device_type = "cpu";
> > + reg = <0x00000000>;
> > + d-cache-line-size = <32>; // 32 bytes
> > + i-cache-line-size = <32>; // 32 bytes
> > + d-cache-size = <32768>; // L1, 32K
> > + i-cache-size = <32768>; // L1, 32K
> > + timebase-frequency = <0>; // From uboot
> > + bus-frequency = <0>; // From uboot
> > + clock-frequency = <0>; // From uboot
> > + l2cr = <0x80000000>; // Enable L2
>
> do you expect the 'l2cr' prop to do anything? Not sure if this is
> something apple invented or you did?
>
TBH, it was in the dts file that I based this on - I thought it was in the others as well - my mistake, I shall remove it.
>
> >
> > + i2c1: i2c at 3000 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + device_type = "i2c";
>
> drop device_type.
>
Ah - _all_ device_type!
<snip>
> >
> > + };
> > + };
> > +
> > + enet0: ethernet at 24000 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
>
> there is no need for #add/size cells here.
>
Will remove.
> >
> > + device_type = "network";
> > + model = "eTSEC";
> > + compatible = "gianfar";
> > + reg = <0x00024000 0x00001000>;
> > + mac-address = [ 00 00 00 00 00 00 ]; // set by u-boot
> > + interrupts = <0x1d 0x2 0x1e 0x2 0x22 0x2>;
> > + interrupt-parent = <&mpic>;
> > + phy-handle = <&phy0>;
> > + phy-connection-type = "gmii";
> > + };
> > +
> > + enet1: ethernet at 26000 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
>
> there is no need for #add/size cells here.
>
ditto.
> >
> > + device_type = "network";
> > + model = "eTSEC";
> > + compatible = "gianfar";
> > + reg = <0x00026000 0x00001000>;
> > + mac-address = [ 00 00 00 00 00 00 ]; // set by u-boot
> > + interrupts = <0x1f 0x2 0x20 0x2 0x21 0x2>;
> > + interrupt-parent = <&mpic>;
> > + phy-handle = <&phy2>;
> > + phy-connection-type = "gmii";
> > + };
> > +
>
> You may want to add in msi interrupts for PCIe support (see
> mpc8572ds.dtb)
>
Interrupts are routed on this board - just via a custom interrupt controller.
I have a patch to add this, but I was getting the basics right first...
> > +
> > + mpic: pic at 40000 {
> > + clock-frequency = <0>;
> > + interrupt-controller;
> > + #address-cells = <0>;
> > + #interrupt-cells = <2>;
> > + reg = <0x00040000 0x00040000>;
> > + compatible = "chrp,open-pic";
> > + device_type = "open-pic";
> > + };
> > +
> > + global-utilities at e0000 {
> > + compatible = "fsl,mpc8641-guts";
> > + reg = <0x000e0000 0x00001000>;
> > + fsl,has-rstcr;
> > + };
> > + };
> > +
> > + pci0: pcie at fef08000 {
> > + compatible = "fsl,mpc8641-pcie";
> > + device_type = "pci";
> > + #interrupt-cells = <1>;
> > + #size-cells = <2>;
> > + #address-cells = <3>;
> > + reg = <0xfef08000 0x00001000>;
> > + bus-range = <0x0 0xff>;
> > + ranges = <0x02000000 0x0 0x80000000 0x80000000 0x0 0x40000000
> > + 0x01000000 0x0 0x00000000 0xfe000000 0x0 0x00400000>;
> > + clock-frequency = <33333333>;
> > + interrupt-parent = <&mpic>;
> > + interrupts = <0x18 0x2>;
> > + interrupt-map-mask = <0xf800 0x0 0x0 0x7>;
> > + interrupt-map = <
> > + 0x0000 0x0 0x0 0x1 &mpic 0x0 0x1
> > + 0x0000 0x0 0x0 0x2 &mpic 0x1 0x1
> > + 0x0000 0x0 0x0 0x3 &mpic 0x2 0x1
> > + 0x0000 0x0 0x0 0x4 &mpic 0x3 0x1
> > + >;
> > +
> > + pcie at 0 {
> > + reg = <0 0 0 0 0>;
> > + #size-cells = <2>;
> > + #address-cells = <3>;
> > + device_type = "pci";
> > + ranges = <0x02000000 0x0 0x80000000
> > + 0x02000000 0x0 0x80000000
> > + 0x0 0x40000000
> > +
> > + 0x01000000 0x0 0x00000000
> > + 0x01000000 0x0 0x00000000
> > + 0x0 0x00400000>;
> > + };
> > + };
> > +};
>
>
> >
> > diff --git a/arch/powerpc/platforms/86xx/Makefile b/arch/powerpc/
> > platforms/86xx/Makefile
> > index 8fee37d..cb9fc8f 100644
> > --- a/arch/powerpc/platforms/86xx/Makefile
> > +++ b/arch/powerpc/platforms/86xx/Makefile
> > @@ -7,3 +7,4 @@ obj-$(CONFIG_SMP) += mpc86xx_smp.o
> > obj-$(CONFIG_MPC8641_HPCN) += mpc86xx_hpcn.o
> > obj-$(CONFIG_SBC8641D) += sbc8641d.o
> > obj-$(CONFIG_MPC8610_HPCD) += mpc8610_hpcd.o
> > +obj-$(CONFIG_GEF_SBC610) += gef_sbc610.o
> > diff --git a/arch/powerpc/platforms/86xx/gef_sbc610.c b/arch/powerpc/
> > platforms/86xx/gef_sbc610.c
> > new file mode 100644
> > index 0000000..a1382da
> > --- /dev/null
> > +++ b/arch/powerpc/platforms/86xx/gef_sbc610.c
> > @@ -0,0 +1,177 @@
> > +/*
> > + * GE Fanuc SBC610 board support
> > + *
> > + * Author: Martyn Welch <martyn.welch at gefanuc.com>
> > + *
> > + * Copyright 2008 GE Fanuc Intelligent Platforms Embedded Systems,
> > Inc.
> > + *
> > + * This program is free software; you can redistribute it and/or
> > modify it
> > + * under the terms of the GNU General Public License as
> > published by the
> > + * Free Software Foundation; either version 2 of the License, or
> > (at your
> > + * option) any later version.
> > + *
> > + * Based on: mpc86xx_hpcn.c (MPC86xx HPCN board specific routines)
> > + * Copyright 2006 Freescale Semiconductor Inc.
> > + */
> > +
> > +#include <linux/stddef.h>
> > +#include <linux/kernel.h>
> > +#include <linux/pci.h>
> > +#include <linux/kdev_t.h>
> > +#include <linux/delay.h>
> > +#include <linux/seq_file.h>
> > +#include <linux/of_platform.h>
> > +
> > +#include <asm/system.h>
> > +#include <asm/time.h>
> > +#include <asm/machdep.h>
> > +#include <asm/pci-bridge.h>
> > +#include <asm/mpc86xx.h>
> > +#include <asm/prom.h>
> > +#include <mm/mmu_decl.h>
> > +#include <asm/udbg.h>
> > +
> > +#include <asm/mpic.h>
> > +
> > +#include <sysdev/fsl_pci.h>
> > +#include <sysdev/fsl_soc.h>
> > +
> > +#include "mpc86xx.h"
> > +
> > +#undef DEBUG
> > +
> > +#ifdef DEBUG
> > +#define DBG (fmt...) do { printk(KERN_ERR "SBC610: " fmt); } while
> > (0)
> > +#else
> > +#define DBG (fmt...) do { } while (0)
> > +#endif
> > +
> > +static void __init
> > +gef_sbc610_init_irq(void)
> > +{
> > + struct mpic *mpic1;
> > + struct device_node *np;
> > + struct resource res;
> > +
> > + /* Determine PIC address. */
> > + np = of_find_node_by_type(NULL, "open-pic");
> > + if (np == NULL)
> > + return;
> > + of_address_to_resource(np, 0, &res);
> > +
> > + mpic1 = mpic_alloc(np, res.start,
> > + MPIC_PRIMARY | MPIC_WANTS_RESET | MPIC_BIG_ENDIAN,
> > + 0, 256, "mpic");
> > + of_node_put(np);
> > + BUG_ON(mpic1 == NULL);
> > +
> > + mpic_init(mpic1);
> > +}
>
> any reason not to use mpc86xx_init_irq() in pic.c?
>
Yes, the afore-mentioned chained interrupt controller that is rather specific to our boards that I will/have added to this function in a patch that I shall post soon.
Martyn
--
Martyn Welch MEng MPhil MIET (Principal Software Engineer) T:+44(0)1327322748
GE Fanuc Intelligent Platforms Ltd, |Registered in England and Wales
Tove Valley Business Park, Towcester, |(3828642) at 100 Barbirolli Square,
Northants, NN12 6PF, UK T:+44(0)1327359444 |Manchester,M2 3AB VAT:GB 729849476
More information about the Linuxppc-dev
mailing list