[U-Boot] [PATCH 1/2] mpc83xx: fix the corruption of u-boot when saveenv

york sun york.sun at nxp.com
Wed Jul 20 00:32:42 CEST 2016


On 07/08/2016 12:19 AM, Kevin Hao wrote:
> Robert P. J. Day has pointed that the value of SYS_MONITOR_LEN in
> MPC8315ERDB.h is smaller than the u-boot.bin. This will cause the
> overlap between the code of u-boot and the environment variable.
> So when executing saveenv, it will corrupt the code of u-boot and
> causes the board not boot. Fix this for all the mpc83xx boards by
> reserving a 512K area.
>
> Reported-by: Robert P. J. Day <rpjday at crashcourse.ca>
> Signed-off-by: Kevin Hao <kexin.hao at windriver.com>
> ---
>  include/configs/MPC8308RDB.h  | 2 +-
>  include/configs/MPC8313ERDB.h | 2 +-
>  include/configs/MPC8315ERDB.h | 2 +-
>  include/configs/MPC8323ERDB.h | 2 +-
>  include/configs/MPC832XEMDS.h | 2 +-
>  include/configs/MPC8349EMDS.h | 2 +-
>  include/configs/MPC8349ITX.h  | 2 +-
>  include/configs/MPC837XEMDS.h | 2 +-
>  include/configs/MPC837XERDB.h | 2 +-
>  9 files changed, 9 insertions(+), 9 deletions(-)
>

I don't have any mpc83xx board to test the change. But it looks OK.

Sinan, do you have any comment for this and next patch in this set?

York


More information about the U-Boot mailing list