[U-Boot] [PATCH v2] powerpc/mpc85xx: The end address of the bss in the SPL should be 4byte alignment
Scott Wood
scottwood at freescale.com
Wed May 15 18:50:36 CEST 2013
On 05/14/2013 05:30:05 AM, Wolfgang Denk wrote:
> Dear ying.zhang at freescale.com,
>
> In message
> <1368520918-22496-1-git-send-email-ying.zhang at freescale.com> you
> wrote:
> > From: Ying Zhang <b40530 at freescale.com>
>
> You mark this patch as V2 but there is no history of changes. Please
> always include information what exactly was changed.
>
> > There will clear the BSS in the function clear_bss(), the reset
> address of
> > the BSS started from the __bss_start, and increased by four-byte
> increments,
> > finally stoped depending on the adress is equal to the _bss_end. If
> the end
> > address __bss_end is not alignment to 4byte, it will be an infinite
> loop.
> >
> > The end address of the bss should be 4byte aligned.
>
> NAK. This is the wrong way to fix this.
>
> Instead, the test in the loop should be fixed to test for "<=".
I agree that we should use <= in the loop, but 4-byte alignment is also
needed unless we change it from an stw loop to an stb loop.
-Scott
More information about the U-Boot
mailing list