[U-Boot] [PATCH 2/2] lsxl: rework boot scripts

[Michael Walle]

Hi Tom,

Am 2014-03-12 22:32, schrieb Tom Rini:
> On Wed, Mar 12, 2014 at 07:42:54PM +0100, Michael Walle wrote:
> 
>> Move addresses for kernel, ramdisk and fdt blob to own variables. Add 
>> dtb
>> blob loading to all existing boot scripts, dtb filenames were taken 
>> from
>> vanilla kernel. Introduce new boot script bootcmd_legacy, which only 
>> loads
>> a kernel and a ramdisk. Make this the default boot script. This should 
>> also
>> restore the behaviour of the original bootloader.
> 
> How much memory do these platforms have?

LS-CHLv2 has 64MB and LS-XHL has 256MB. The kernel_addr and ramdisk_addr 
are the value which was also used in the original bootloader.

> Having just spent a bunch of
> time issues about where to load what on TI platforms, I'm a little
> worried about some of the locations:
> 
>> +	"kernel_addr=0x00100000\0"					\
> 
> Below 32MB which isn't optimal.

why is that? iirc the kernel is unpacked to 0x8000, isn't it? ok there 
might be some problems unpacking the kernel.

> 
>> +	"ramdisk_addr=0x00800000\0"					\
>> +	"fdt_addr=0x007f0000\0"						\
> 
> This doesn't leave a whole lot of space for the kernel before
> overwriting either of these.

I must admit i've never worried about where to put these. Any 
suggestions? initrd at the end of the ram? although i'd like to keep 
both platforms the same, eg. i'd take 64MB as the end of ram.

> 
>> +	"bootcmd_legacy=ide reset "					\
>> +		"&& ext2load ide ${hdpart} ${kernel_addr} "		\
> 
> Add CONFIG_CMD_FS_GENERIC and just 'load' here and elsewhere (incl
> fatload).

i'll have a look at it.

thanks for the suggestions,

-michael