[U-Boot] [PATCH 1/2] coldfire: Fix compilation for most coldfire boards

Jin Zhengxiong-R64188 R64188 at freescale.com
Wed Sep 14 12:10:33 CEST 2011


> >> In message <BE72CBA69B1D6F44B0680DF984332A32253A40 at 039-SN1MPN1-
> >> 005.039d.mgd.msft.net> you wrote:
> >>>
> >>>> From: Stany MARCEL [mailto:stany.marcel at novasys-ingenierie.com]
> >>>> Sent: Monday, September 05, 2011 9:29 PM
> >>>> To: u-boot at lists.denx.de
> >>>> Cc: Jin Zhengxiong-R64188; Jin Zhengxiong-R64188; Stany MARCEL
> >>>> Subject: [PATCH 1/2] coldfire: Fix compilation for most coldfire
> >>>> boards
> >>>>
> >>>> Many different patches to fix compilation of coldfire boards.
> >>>> Compilation could now be done in a different directory from the
> >>>> sources one. Simple board config switched from Makefile to
> >>>> boards.cfg EP2500 broken board removed from boards.cfg. Link files
> >>>> corrected to remove zlib.o and multiple defined symbols.
> >>>>
> >>>> Signed-off-by: Stany MARCEL <stany.marcel at novasys-ingenierie.com>
> >>>> ---
> >>>>  MAKEALL                              |    6 --
> >>>>  Makefile                             |  124 +++++------------------------
> -
> >> ----
> >>>>  arch/m68k/cpu/mcf5227x/Makefile      |    2 +-
> >>>>  arch/m68k/cpu/mcf523x/Makefile       |    2 +-
> >>>>  arch/m68k/cpu/mcf532x/Makefile       |    2 +-
> >>>>  arch/m68k/cpu/mcf5445x/Makefile      |    2 +-
> >>>>  arch/m68k/cpu/mcf547x_8x/Makefile    |    2 +-
> >>>>  board/BuS/EB+MCF-EV123/u-boot.lds    |    8 --
> >>>>  board/cobra5272/u-boot.lds           |    7 --
> >>>>  board/esd/tasreg/u-boot.lds          |    7 --
> >>>>  board/freescale/m52277evb/.gitignore |    1 +
> >>>>  board/freescale/m5235evb/.gitignore  |    1 +
> >>>>  board/freescale/m5235evb/u-boot.16   |    8 --
> >>>>  board/freescale/m5235evb/u-boot.32   |   16 -----
> >>>>  board/freescale/m5249evb/u-boot.lds  |    7 --
> >>>>  board/freescale/m5253evbe/u-boot.lds |    7 --
> >>>>  board/freescale/m5272c3/u-boot.lds   |    7 --
> >>>>  board/freescale/m5275evb/u-boot.lds  |    6 --
> >>>>  board/freescale/m54451evb/.gitignore |    1 +
> >>>>  board/freescale/m54455evb/.gitignore |    1 +
> >>>>  board/idmr/u-boot.lds                |    7 --
> >>>>  boards.cfg                           |   21 ++++++-
> >>>>  include/configs/M5329EVB.h           |    8 +-
> >>>>  23 files changed, 50 insertions(+), 203 deletions(-)  create mode
> >>>> 100644 board/freescale/m52277evb/.gitignore
> >>>>  create mode 100644 board/freescale/m5235evb/.gitignore
> >>>>  create mode 100644 board/freescale/m54451evb/.gitignore
> >>>>  create mode 100644 board/freescale/m54455evb/.gitignore
> >>>>
> >>> [Jin Zhengxiong-R64188]
> >>> Please remove the unused git information from the patch. Thanks.
> >>
> >> I agree with your other comments, but what sort of "unused git
> >> information" are you referring to here?
> >>
> >> There is no "unused git information" in this patch. It is a proper
> >> patch generated by git-format-patch and submitted using git-send-email.
> >>
> >> There is nothing that should be changed in the format.
> >>
> > [Jin Zhengxiong-R64188]
> > Ok, Thanks a lot, I had thought that the .gitignore which ignore the lds
> files are empty files.
> >
> > Best Regards,
> > Jason
> >
> >
> Hello,
> 
> Sorry for my late reply, I was working on another subject.
> 
> My work is now separated in different patches and I also made some additional
> improvements.
> 
> What is the best way to submit this new work ? I don't know if it must be a
> version two of the previous patch or a new set of patches.
> 
[Jin Zhengxiong-R64188] 
My personal suggestion is to send a new set of patches, Thanks.

> Here is the new list of patches' subjects:
>  * ColdFire: Cleanup lds files for multiple defined symbols
>  * ColdFire: Add $(obj) before cpu lib to correct build
>  * ColdFire: Move boards with simple _config rules to boards.cfg
>  * ColdFire: Merge differentiated linking files into a sigle one by board
>  * ColdFire: Clean Makefile _config rules
>  * ColdFire: Fix compilation with CONFIG_SYS_DRAMSZ1 defined
> 
[Jin Zhengxiong-R64188]
 I cannot see the details of your patches, So I don't know whether
it's good idea to split the _config cleanup to two patches, and split the lds clean up 
to two patches.

BTW. Could you please try to use CONFIG_SYS_LDSCRIPT to avoid copying the lds file? Thanks.

> For Information, for ColdFire I can only test with a M5485EVB. For other
> ColdFire boards I can only verify that the build is performed correctly.

[Jin Zhengxiong-R64188] 
OK, Thanks.

BTW. Please try to compose your email with plaintext, Thanks.

Jason



More information about the U-Boot mailing list