[PATCH v3 1/2] powerpc: Board support for GE Fanuc SBC610

[Martyn Welch]

On Fri, 29 Aug 2008 07:04:18 -0500
Kumar Gala <galak at kernel.crashing.org> wrote:

> 
> On Aug 26, 2008, at 8:13 AM, Martyn Welch 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
> 
> you can remove these since u-boot will set them.
> 

Ok.

> >
> > +		};
> > +		PowerPC,8641 at 1 {
> > +			device_type = "cpu";
> > +			reg = <0x00000001>;
> > +			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
> 
> you can remove these since u-boot will set them.
> 

Will do.

> >
> > +		};
> > +	};
> > +
> > +	memory {
> > +		device_type = "memory";
> > +		reg = <0x00000000 0x40000000>;	// set by u-boot
> 
> if the comment is zero just set it to <0 0>;
> 

So "reg = <0x0 0x40000000>;" ?

> >
> > +	};
> > +
> 
> what u-boot version are you using/shipping with these boards?

U-boot 1.2.0

> Wondering if other props (like cache info, is set by u-boot for you).
> 
> - k

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