Re: [U-Boot] [PATCH 3/3] arm: fix incorrect monitor protection region in FLASH

2 Mar 2011


      Hi Albert,
On Wed, Mar 2, 2011 at 3:03 PM, Albert ARIBAUD albert.aribaud@free.fr wrote:
...
Hi Po-Yu Chuang,
Le 02/03/2011 07:02, Po-Yu Chuang a écrit :
...
From: Po-Yu Chuangratbert@faraday-tech.com
Monitor protection region in FLASH did not cover .rel.dyn section
and .dynsym sections, because it uses __bss_start to compute
monitor_flash_len. Use _end instead.
Add _end_ofs to all the start.S.
Signed-off-by: Po-Yu Chuangratbert@faraday-tech.com
Open question: why not group patches 2 and 3 as a single functional patch?
Patch 2 seems purposeless to me as a standalone patch.
I thought that patches 2 and 3 are separate steps, but I am fine with both
ways. :-)
If no other comment, I will resend them as a single patch.
regards,
Po-Yu Chuang