Re: [PATCH/RFC] powerpc: DBox2 Board Support

2008-01-02 Thread Scott Wood
Jochen Friedrich wrote: >>> +[EMAIL PROTECTED] { >>> +label = "Flash without bootloader"; >>> +reg = <2 7e>; >>> +}; >>> +[EMAIL PROTECTED] { >>> +label = "Complete Flash"; >>> +reg = <0 8000

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2008-01-02 Thread Jochen Friedrich
Hi Scott, > These unit addresses look wrong. I know. These have (hopefully) been fixed in v3. > What does dbox2-config mean? I used it to search for the vendor info byte in the flash. It has been moved to the boot wrapper in v3. Later this should all be moved to u-boot, IMHO. >> +

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2008-01-02 Thread Scott Wood
On Sat, Dec 22, 2007 at 08:13:31PM +0100, Jochen Friedrich wrote: > + [EMAIL PROTECTED] { > + compatible = "c-cube,gtx"; > + reg = <40 3000 0 20>; > + interrupts = <2 2>; > + interrupt-parent = <&PIC

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-26 Thread Jochen Friedrich
Hi David, + // Port D is LCD exclusive. Don't export as GPIO + CPM1_PIO: [EMAIL PROTECTED] { + compatible = "fsl,cpm1-pario"; + reg = <970 180>; + num-ports = <3>;

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-24 Thread David Gibson
On Mon, Dec 24, 2007 at 12:01:35PM +0100, Jochen Friedrich wrote: > Hi David, > > >> + [EMAIL PROTECTED] { > >> + compatible = "betaresearch,dbox2-localbus"; > > > > Is this bus interface really board specific? I would have thought the > > localbus bridge would have been part of the So

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-24 Thread Kumar Gala
> > diff --git a/include/asm-powerpc/mpc8xx.h b/include/asm-powerpc/ > mpc8xx.h > index 2be014b..b6fd7d6 100644 > --- a/include/asm-powerpc/mpc8xx.h > +++ b/include/asm-powerpc/mpc8xx.h > @@ -23,6 +23,10 @@ > #include > #endif > > +#if defined(CONFIG_DBOX2) > +#include > +#endif > + > #ifdef CON

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-24 Thread Jochen Friedrich
Hi David, >> +[EMAIL PROTECTED] { >> +compatible = "betaresearch,dbox2-localbus"; > > Is this bus interface really board specific? I would have thought the > localbus bridge would have been part of the SoC in which case the > compatible string should mention the 823 rather than t

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-23 Thread David Gibson
On Sat, Dec 22, 2007 at 08:13:31PM +0100, Jochen Friedrich wrote: > This patch adds device tree source, default config and setup code for > DBox2 devices. [snip] > diff --git a/arch/powerpc/boot/dts/dbox2.dts b/arch/powerpc/boot/dts/dbox2.dts > new file mode 100644 > index 000..8d91510 > --- /

Re: [PATCH/RFC] powerpc: DBox2 Board Support

2007-12-23 Thread Stephen Rothwell
On Sat, 22 Dec 2007 20:13:31 +0100 Jochen Friedrich <[EMAIL PROTECTED]> wrote: > > +++ b/arch/powerpc/platforms/8xx/dbox2.h > + > +#include Nothing in this header files uses anything in that one, so don't include it. > +++ b/arch/powerpc/platforms/8xx/dbox2_setup.c > + > +char *dbox2_manuf_name[

[PATCH/RFC] powerpc: DBox2 Board Support

2007-12-22 Thread Jochen Friedrich
This patch adds device tree source, default config and setup code for DBox2 devices. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- arch/powerpc/boot/dts/dbox2.dts | 263 arch/powerpc/configs/dbox2_defconfig | 1042 ++ arch/powerpc/platf