Re: [patch 3/4] 4xx: Convert Walnut flash mappings to new binding

[Josh Boyer <jwboyer@linux.vnet.ibm.com>]

On Sat, 2007-09-15 at 05:23 +0200, Stefan Roese wrote:
> On Friday 14 September 2007, Josh Boyer wrote:
> > A new binding for flash devices was recently introduced.  This updates the
> > Walnut DTS to use the new binding.
> >
> > Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
> > ---
> >  arch/powerpc/boot/dts/walnut.dts |   17 ++++++++++++-----
> >  1 file changed, 12 insertions(+), 5 deletions(-)
> >
> > --- linux-2.6.orig/arch/powerpc/boot/dts/walnut.dts
> > +++ linux-2.6/arch/powerpc/boot/dts/walnut.dts
> > @@ -137,6 +137,10 @@
> >  			dcr-reg = <012 2>;
> >  			#address-cells = <2>;
> >  			#size-cells = <1>;
> > +			/* The ranges property is supplied by the bootwrapper
> > +			 * and is based on the firmware's configuration of the
> > +			 * EBC bridge
> > +			 */
> >  			clock-frequency = <0>; /* Filled in by zImage */
> >
> >  			sram@0,0 {
> > @@ -144,13 +148,16 @@
> >  			};
> >
> >  			flash@0,80000 {
> > -				device_type = "rom";
> > -				compatible = "direct-mapped";
> > -				probe-type = "JEDEC";
> > +				compatible = "jedec-flash";
> >  				bank-width = <1>;
> > -				partitions = <0 80000>;
> > -				partition-names = "OpenBIOS";
> >  				reg = <0 80000 80000>;
> > +				#address-cells = <1>;
> > +				#size-cells = <1>;
> > +				partition@0 {
> > +					label = "OpenBIOS";
> > +					reg = <0 80000>;
> > +					read-only;
> > +				};
> 
> There are not only Bamboo board running PIBS, but running U-Boot too. How 
> should we handle this different FLASH partitioning? Same goes for Ebony too 
> btw.

That's a good question.  I'm working on making the NOR flash show up for
Bamboo right now, and I had intended to just leave the partition
subnodes missing.

josh