[U-Boot-Users] [GIT-PULL][Blackfin] Please pull http://www.denx.de/git/u-boot-blackfin.git

Aubrey Li aubrey.adi at gmail.com
Sat Mar 10 06:15:53 CET 2007


On 3/10/07, Wolfgang Denk <wd at denx.de> wrote:
> Dear Aubrey,
>
> in message <27d85ee10703082203m74822868i5d3fe6fd0eaa269a at mail.gmail.com> you wrote:
> > The following changes since commit eb92f613556800f7483666db09d9a237ad911d4a:
> >
> > Aubrey.Li (2)
> >     [Patch 1/2] Remove obsolete blackfin port and add bf533 platform support
>
> Sorry, but I have to reject to merge the current version.
>
>
> Commit 3f0606ad0b5639f7f22848fe5b4574e754d0470f breaks building in
> other directories, i.e. changes like this one are bad and need to be
> fixed:
>
> board/bf533-ezkit/Makefile:
>
>
> -LIB    = $(obj)lib$(BOARD).a
> +LIB    = lib$(BOARD).a
> ...
> -SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> -OBJS   := $(addprefix $(obj),$(COBJS))
> -SOBJS  := $(addprefix $(obj),$(SOBJS))
> +$(LIB):        .depend $(OBJS) u-boot.lds
> +       $(AR) cr $@ $(OBJS)
>
> -$(LIB):        $(obj).depend $(OBJS)
> -       $(AR) $(ARFLAGS) $@ $(OBJS)
> +u-boot.lds: u-boot.lds.S
> +       $(CPP) $(CPPFLAGS) -P -Ubfin $^ > $@.tmp
> +       mv -f $@.tmp $@
>
> etc.
>
> The same is happens in other Makefiles.

u-boot.lds assumes the virtual memory address(VMA) of u-boot is the
same as load memory address(LMA), which is not fit for blackfin.
Blackfin uses on-chip L1 memory to load parts of u-boot code. So we
need to add memory section in the link script, like that:
======================================
MEMORY
 {
 ram :       ORIGIN = 0x3FC0000, LENGTH = (256 * 1024)
 l1_code : ORIGIN = 0xFFA00000, LENGTH = 0xC000
 l1_data :  ORIGIN = 0xFF900000, LENGTH = 0x4000
 }
=======================================
We don't like hardcoded memory address in the link script, that will
make u-boot not good to port. We are using to memory macros in the
config file, so that user just need to change one configuration item
to make all the relative place right. I think this is more easy to
port u-boot to the new boards.

> Commit 3f0606ad0b5639f7f22848fe5b4574e754d0470f breaks building in
> other directories

Sorry, I don't understand. Where should I test building u-boot?

>
>
> Also, turning back Copyright entries is a Bad Thing, too.
>
>
> Also, there is a LOT of Coding Style problems:
>
> C++ comments in ./include/asm-blackfin/arch-bf533/defBF532.h,
> ./include/asm-blackfin/arch-common/cdef_LPBlackfin.h,
> ./include/configs/bf533-ezkit.h, ./include/configs/bf533-stamp.h
>
> Trailing white space in ./cpu/bf533/bf533_serial.h,
> ./cpu/bf533/cache.S, ./cpu/bf533/flush.S, ./cpu/bf533/init_sdram.S,
> ./cpu/bf533/init_sdram_bootrom_initblock.S, ./cpu/bf533/interrupt.S,
> ./cpu/bf533/start.S, ./cpu/bf533/traps.c, ./cpu/bf533/video.c
>
> Indentation not by TABs at least in
> ./cpu/bf533/init_sdram_bootrom_initblock.S, ./cpu/bf533/start.S,
> ,./include/asm-blackfin/arch-bf533/anomaly.h
> ./include/asm-blackfin/mem_init.h, ./include/configs/bf533-ezkit.h,
> ./include/configs/bf533-stamp.h
>
> Too many empty lines in ./lib_blackfin/memcpy.S
>
> Trailing empty lines in ./board/bf533-ezkit/u-boot.lds.S,
> ./board/bf533-stamp/u-boot.lds.S, ./cpu/bf533/init_sdram.S,
> ./cpu/bf533/init_sdram_bootrom_initblock.S
>

Wired, c files and h files was formatted by the kernel script Lindent,
and when I use quilt to maintain these patches, quilt doesn't warn me
anymore. How do you check the coding style issues? and is there any
script can help to format assemble files?
Thanks a lot.

Best Regards,
-Aubrey




More information about the U-Boot mailing list