> -----Original Message-----
> From: Clint Adams [mailto:cl...@debian.org]
> Sent: Wednesday, April 13, 2011 12:14 AM
> To: u-boot@lists.denx.de
> Cc: Prafulla Wadaskar; julian.pidan...@citrix.com; Clint Adams
> Subject: [PATCH 3/3] Rename openrd_base files to openrd.
> 
> ---
>  board/Marvell/openrd/Makefile           |   56 ++++++++++
>  board/Marvell/openrd/kwbimage.cfg       |  168
> +++++++++++++++++++++++++++++
>  board/Marvell/openrd/openrd.c           |  175
> +++++++++++++++++++++++++++++++
>  board/Marvell/openrd/openrd.h           |   46 ++++++++
>  board/Marvell/openrd_base/Makefile      |   56 ----------
>  board/Marvell/openrd_base/kwbimage.cfg  |  168 ------------------------
> -----
>  board/Marvell/openrd_base/openrd_base.c |  175 ------------------------
> -------
>  board/Marvell/openrd_base/openrd_base.h |   46 --------
>  boards.cfg                              |    6 +-
>  include/configs/openrd.h                |  142
> +++++++++++++++++++++++++
>  include/configs/openrd_base.h           |  142 ------------------------
> -
>  11 files changed, 590 insertions(+), 590 deletions(-)
>  create mode 100644 board/Marvell/openrd/Makefile
>  create mode 100644 board/Marvell/openrd/kwbimage.cfg
>  create mode 100644 board/Marvell/openrd/openrd.c
>  create mode 100644 board/Marvell/openrd/openrd.h
>  delete mode 100644 board/Marvell/openrd_base/Makefile
>  delete mode 100644 board/Marvell/openrd_base/kwbimage.cfg
>  delete mode 100644 board/Marvell/openrd_base/openrd_base.c
>  delete mode 100644 board/Marvell/openrd_base/openrd_base.h
>  create mode 100644 include/configs/openrd.h
>  delete mode 100644 include/configs/openrd_base.h

I think it would be good if you make this your first patch in the patch series, 
then rest of the code changes looks more relevant.

Also take care of additional suggestions form Albert about incremental 
versioning, change logs, and git-rename for delete/create logs

Also keep practice of adding some more information for patch description in the 
beginning of each patch.

Regards..
Prafulla . .
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to