On Mon, Jan 11, 2016 at 02:51:39AM +0000, Dongsheng Wang wrote: > Hi Tom, > > Sorry for my late reply, and thanks for your reply. > > How about the following comments, following your suggestion I remove some > redundant comments? > > If my understanding is wrong, please correct me, thanks: > #if defined(CONFIG_ARMV7_SECURE_BASE) && defined(CONFIG_ARMV7_NONSEC) > /* > * If CONFIG_ARMV7_SECURE_BASE is true, secure code will not > * bundle with u-boot, and codes offset are fixed. Secure zone
code offsets > * only needs to be copied from loading address to from the loading address > * CONFIG_ARMV7_SECURE_BASE, which is the linking and running > * address for secure code. > * > * If undefine CONFIG_ARMV7_SECURE_BASE secure zone will be If CONFIG_ARMV7_SECURE_BASE is undefined, the secure zone will be > * included in u-boot space, and some absolute address were used u-boot address space > * in secure code. Accompanied by u-boot relocation secure code > * also need to relocate the absolute address. in secure code. The absolute addresses of the secure code also needs to be relocated along with the accompanying u-boot code. > * > * So DISCARD is only for CONFIG_ARMV7_SECURE_BASE. > */ > /DISCARD/ : { *(.rel._secure*) } > #endif Otherwise looks good, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot