[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