[U-Boot] [PATCH v4] powerpc/c29xpcie: add support for C29XPCIE board

Wolfgang Denk wd at denx.de
Wed Jul 3 13:14:19 CEST 2013


Dear Po Liu,

In message <1372840694-17354-1-git-send-email-Po.Liu at freescale.com> you wrote:
> From: Mingkai Hu <Mingkai.Hu at freescale.com>
> 
> C29XPCIE board is a series of Freescale PCIe add-in cards to perform
> as public key crypto accelerator or secure key management module. It
> includes C293PCIE board, C293PCIE board and C291PCIE board.

Why don't you keep thread information so we can easily look up
previous review comments?  This is a mandatory requirement when
resubmitting patches - see bullet # 3 at [1]

>  board/freescale/c29xpcie/Makefile   |  34 +++
>  board/freescale/c29xpcie/c29xpcie.c | 151 ++++++++++++
>  board/freescale/c29xpcie/cpld.c     | 134 ++++++++++
>  board/freescale/c29xpcie/cpld.h     |  43 ++++
>  board/freescale/c29xpcie/ddr.c      |  89 +++++++
>  board/freescale/c29xpcie/law.c      |  22 ++
>  board/freescale/c29xpcie/tlb.c      |  83 +++++++
>  boards.cfg                          |   2 +
>  include/configs/C29XPCIE.h          | 475 ++++++++++++++++++++++++++++++++++++
>  9 files changed, 1033 insertions(+)
>  create mode 100644 board/freescale/c29xpcie/Makefile
>  create mode 100644 board/freescale/c29xpcie/c29xpcie.c
>  create mode 100644 board/freescale/c29xpcie/cpld.c
>  create mode 100644 board/freescale/c29xpcie/cpld.h
>  create mode 100644 board/freescale/c29xpcie/ddr.c
>  create mode 100644 board/freescale/c29xpcie/law.c
>  create mode 100644 board/freescale/c29xpcie/tlb.c
>  create mode 100644 include/configs/C29XPCIE.h

Why do you ignore review comments (see [2]) ?

You still fail to add a MAINTAINERS entry!

[1] http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions
[2] http://article.gmane.org/gmane.comp.boot-loaders.u-boot/164730


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
In the beginning, there was nothing, which exploded.
                                - Terry Pratchett, _Lords and Ladies_


More information about the U-Boot mailing list