[U-Boot] [PATCH v6] AT91SAM9*: Change kernel address in dataflash to match u-boot's size

[Alexandre Belloni]

On Mon, Jul 02, 2012 at 02:01:00PM +0200, Andreas Bießmann wrote :
> really sorry for that, I should have seen it before. Shouldn't we also
> make the U-Boot partition greater when we move the start of kernel
> partition?
> Can you please send another version, rest is fine (checkpatch, apply
> cleanly, no build errors).
> 
> >  	{0x00252000, 0xFFFFFFFF, FLAG_PROTECT_CLEAR, 0,	"FS"},
> >  };

Ah, sure, I'll fix that.

> 
> <snip>
> 
> > diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
> > index 07b1968..ef25fa5 100644
> > --- a/include/configs/at91sam9260ek.h
> > +++ b/include/configs/at91sam9260ek.h
> > @@ -187,7 +187,7 @@
> >  #define CONFIG_ENV_OFFSET		0x4200
> >  #define CONFIG_ENV_ADDR		(CONFIG_SYS_DATAFLASH_LOGIC_ADDR_CS0 + CONFIG_ENV_OFFSET)
> >  #define CONFIG_ENV_SIZE		0x4200
> > -#define CONFIG_BOOTCOMMAND	"cp.b 0xC0042000 0x22000000 0x210000; bootm"
> > +#define CONFIG_BOOTCOMMAND	"cp.b 0xC0084000 0x22000000 0x210000; bootm"
> 
> also sorry, but size is wrong here! We reduced the partition to
> 0x1CE000, it is about 1.8 MiB now and not 2 MiB as before.
> 

Right, I didn't really care about the end of the kernel partition as the
9261ek has an 8MiB dataflash and my rootfs is in NAND. The kernel is
around 1.7MiB using the default configuration. Maybe, we should also
extend its partition ?

> Best regards
> 
> Andreas Bießmann
> 

-- 
Alexandre Belloni