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

Po-Yu Chuang ratbert.chuang at gmail.com
Wed Mar 2 08:18:36 CET 2011


Hi Albert,

On Wed, Mar 2, 2011 at 3:03 PM, Albert ARIBAUD <albert.aribaud at free.fr> wrote:
> Hi Po-Yu Chuang,
>
> Le 02/03/2011 07:02, Po-Yu Chuang a écrit :
>>
>> From: Po-Yu Chuang<ratbert at 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 Chuang<ratbert at 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


More information about the U-Boot mailing list