It'd be also great if you make the UBL more straightforward to port for boards with less than 256 MB memory.

For current implementation I had to change:
    .ddrram2 starting address in ubl_davinci.lds,
    MAX_IMAGE_SIZE in ubl.h and
    RAM_END_ADDR in ubl.h

For my memory map (128 mb), I set them respectively to 0x84000000, 0x01000000 and 0x87FFFFFF

This could all be done in a single step with a -DRAM_END_ADDR=<desired size> compiler option if 1) we didn't put gNandTx[] and gNandRx[] as static arrays in ddrram2, but rather allocated them with ubl_alloc_mem() 2) made ubl_alloc_mem allocate from RAM_END_ADDR downwards, and not from RAM_START_ADDR upwards

That would:
   1) make configuration user friendly
2) obsolete the .ddrram2 section and hence make the ubl_davinci.lds one for all configurations 3) make S-record decoding more unlikely to fail due to large image sizes: since we download everything from the end of DDR downwards and application entry points are usually closer to the start of DDR than to the end of it.

Anyway, it would be also great if you check whether downloaded s-record image and decoded binary image areas overlap. Instead of calling SRecDecode() and waiting it to fail, you can UARTSendData() something like "Unable to decode srecord. Application load address falls between downloaded image boundaries"

What do you think?

Ivan

Allred, Daniel wrote:

You are correct. That got left over when I modified the file for supporting big block devices. The customer tested it on a big block device so it worked fine.

Thanks.  It'll be fixed soon.

Regards,

Daniel

Daniel J. Allred

Software Applications

Catalog DSP / Emerging End Equipment

------------------------------------------------------------------------


begin:vcard
fn:Ivan Tonchev
n:Tonchev;Ivan
org:Multimedia Solutions (MMS);IT
adr:;;Tintiava 15;Sofia;Sofia;1113;Bulgaria
email;internet:[EMAIL PROTECTED]
title:Network Administrator
tel;work:+359 2 868 9186
tel;cell:+359 88 608 9664
url:http://www.mm-sol.com
version:2.1
end:vcard

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to