[U-Boot] [PATCH v14 5/9] nds32/lib: add generic funcs in NDS32 lib

Wolfgang Denk wd at denx.de
Thu Oct 6 23:25:47 CEST 2011


Dear Macpaul Lin,

In message <1316574638-20460-5-git-send-email-macpaul at andestech.com> you wrote:
> Add Makefile, board.c, interrupts.c and bootm.c functions
> to nds32 architecture.
> 
> Signed-off-by: Macpaul Lin <macpaul at andestech.com>
> ---
> Changes for v1-v4:
>   - code clean up and formatting style.
> Changes for v5-v6:
>   - board.c
>    - Do some clean up and add code
>    - Remove display banner which hasn't support.
>    - Add ftpmu010 related power management unit code.
>    - Remove useless LED related code.
>    - Move SDRAM init to board sepecific files. (ex. adp-ag101.c)
>    - Remove CONFIG_SOFT_I2C which hasn't been support.
>    - Remove CONFIG_FSL_ESDHC which hasn't been support.
>    - clean up.
> Changes for v7:
>   - clean up.
>   - move single file patch arch/nds32/config.mk to this commit.
>   - interrupts.c refine origin interrupt enable and disable.
> Changes for v8:
>   - interrups.c: fix up for new ptraces.h.
> Changes for v9:
>   - support CONFIG_STANDALONE_LOAD_ADDR in config.mk
> Changes for v10:
>   - config.mk:
>    - add -fpie flag.
>    - replace -ffixed-8 to -ffixed-10.
>    - add -mrelax and --gc-sections to LDFLAG
>   - board.c:
>    - fix lib/board.c for relocation.
>    - fix dram init for relocation.
> Changes for v11:
>   - arch/nds32/lib/Makefile
>    - replace $(AR) $(call cmd_link_o_target,...)
> Changes for v12:
>   - config.mk
>    - remove $(SRCTREE)/$(CPUDIR)/u-boot.lds
>   - board.c:
>    - remove obsolelte version_string.
>    - remove declaration "extern __bss_end" which is not need.
>    - replace sizeof(gd_t) and sizeof(bd_t) to
>      GENERATED_GBL_DATA_SIZE and GENERATED_BD_INFO_SIZE
>    - add memset to board info (bd)
>    - remove compiler optimization barrier which is not need.
> Changes for v13:
>   - board.c: remove unused CONFIG_IDENT_STRING.
>   - arch/nds32/lib/Makefile: remove unused clean and distclean.
> Changes for v14:
>   - No change.
> 
>  arch/nds32/config.mk        |   35 ++++
>  arch/nds32/lib/Makefile     |   46 +++++
>  arch/nds32/lib/board.c      |  443 +++++++++++++++++++++++++++++++++++++++++++
>  arch/nds32/lib/bootm.c      |  241 +++++++++++++++++++++++
>  arch/nds32/lib/interrupts.c |  131 +++++++++++++
>  5 files changed, 896 insertions(+), 0 deletions(-)
>  create mode 100644 arch/nds32/config.mk
>  create mode 100644 arch/nds32/lib/Makefile
>  create mode 100644 arch/nds32/lib/board.c
>  create mode 100644 arch/nds32/lib/bootm.c
>  create mode 100644 arch/nds32/lib/interrupts.c

Checkpatch says:

total: 0 errors, 8 warnings, 896 lines checked

Please clean up and resubmit.  Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Accident: A condition in which presence of mind is good, but  absence
of body is better.


More information about the U-Boot mailing list