[U-Boot-Users] [PATCH] Updated MPC8349ITX*_config to place config.tmp in correct location.

Hello All,
Please excuse any formatting issues in this email, as this is my first patch. I'm happy to resubmit, if necessary.
MPC834ITX*_config does not store config.tmp at the correct locatation, causing MPC8349ITXGP to have the wrong TEXT_BASE.
Regards, Sam
--- Makefile | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile index 0371d46..d429126 100644 --- a/Makefile +++ b/Makefile @@ -1797,10 +1797,10 @@ MPC8349ITXGP_config: unconfig @mkdir -p $(obj)board/mpc8349itx @echo "#define CONFIG_$(subst _LOWBOOT,,$(@:_config=))" >> $(obj)include/config.h @if [ "$(findstring GP,$@)" ] ; then \ - echo "TEXT_BASE = 0xFE000000" >$(obj)board/mpc8349itx/config.tmp ; \ + echo "TEXT_BASE = 0xFE000000"
$(obj)board/freescale/mpc8349itx/config.tmp ; \
fi @if [ "$(findstring LOWBOOT,$@)" ] ; then \ - echo "TEXT_BASE = 0xFE000000" >$(obj)board/mpc8349itx/config.tmp ; \ + echo "TEXT_BASE = 0xFE000000"
$(obj)board/freescale/mpc8349itx/config.tmp ; \
fi @$(MKCONFIG) -a -n $(@:_config=) MPC8349ITX ppc mpc83xx mpc8349itx freescale
-- 1.5.0.7

In message BAF8B1E0BB28024A90895E746A3B610D1B51FD@twx-exch01.twacs.local you wrote:
Please excuse any formatting issues in this email, as this is my first patch. I'm happy to resubmit, if necessary.
Signed-off-by: line missing.
@mkdir -p $(obj)board/mpc8349itx @echo "#define CONFIG_$(subst _LOWBOOT,,$(@:_config=))" >> $(obj)include/config.h
^^^^^^^^^^^^^^^^^^^^^^^^^
@if [ "$(findstring GP,$@)" ] ; then \
echo "TEXT_BASE = 0xFE000000" >$(obj)board/mpc8349itx/config.tmp ;
\
^^^^^^^^^^^
echo "TEXT_BASE = 0xFE000000"
$(obj)board/freescale/mpc8349itx/config.tmp ; \
^^^^^^^^^^^
...and corrupted because your mailer wrapped long lines,
Please fix your mailer settings, add the required information, and resubmit.
Best regards,
Wolfgang Denk
participants (2)
-
Sparks, Sam
-
Wolfgang Denk