[U-Boot] worth submitting a patch to fix u-boot for lite 5200b?

[Tom Rini]

On Sun, Nov 20, 2011 at 12:34 PM, Robert P. J. Day
<rpjday at crashcourse.ca> wrote:
>
>  from back in april, karim yaghmour pointed out a problem related to
> older lite5200 boards, and suggested a fix:
>
> http://old.nabble.com/-U-Boot---PATCH--Env.-vars-fix-for-lite5200-td31013626.html
>
> i checked my git clone of u-boot and it seems that the file
> include/configs/IceCube.h still contains the original code:
>
> #if defined(CONFIG_SYS_LOWBOOT16)
> #define CONFIG_ENV_ADDR         (CONFIG_SYS_FLASH_BASE + 0x00040000)
> #endif
> #endif  /* CONFIG_SYS_LOWBOOT */
> #endif /* CONFIG_LITE5200B */
>
>  would it be proper to submit a patch to change 0x00040000 to
> 0x00050000?  i admit i don't know enough about what else might be
> affected or what else might also need to change, i know only that that
> change is necessary for u-boot to run properly on that type of board.
>
>  thoughts?  is that an appropriate standalone patch?

Well, a patch that addresses Wolfgang's comments and is git am'able
would be a step in the right direction.

-- 
Tom