[U-Boot] [PATCH 1/5] ARM: vexpress: move files in preparation for adding a new platform

Albert ARIBAUD albert.u.boot at aribaud.net
Thu Apr 4 12:06:34 CEST 2013


Hi Andre,

On Wed,  3 Apr 2013 15:44:32 +0200, Andre Przywara
<andre.przywara at linaro.org> wrote:

> From: Ryan Harkin <ryan.harkin at linaro.org>
> 
> The current ca9x4_ct_vxp platform contains support for a Versatile Express
> motherboard with a quad core A9 core tile.
> 
> This patch is the first stage of making separating the Versatile Express
> motherboard code and the A9 specific code, before adding support for the
> dual core A5 core tile.
> 
> Signed-off-by: Ryan Harkin <ryan.harkin at linaro.org>
> Signed-off-by: Andre Przywara <andre.przywara at linaro.org>
> ---
>  MAINTAINERS                             |   2 +-
>  board/armltd/vexpress/Makefile          |   2 +-
>  board/armltd/vexpress/ca9x4_ct_vxp.c    | 257 --------------------------------
>  board/armltd/vexpress/vexpress_common.c | 257 ++++++++++++++++++++++++++++++++
>  boards.cfg                              |   2 +-
>  include/configs/ca9x4_ct_vxp.h          | 198 ------------------------
>  include/configs/vexpress_common.h       | 198 ++++++++++++++++++++++++
>  7 files changed, 458 insertions(+), 458 deletions(-)
>  delete mode 100644 board/armltd/vexpress/ca9x4_ct_vxp.c
>  create mode 100644 board/armltd/vexpress/vexpress_common.c
>  delete mode 100644 include/configs/ca9x4_ct_vxp.h
>  create mode 100644 include/configs/vexpress_common.h

This patch seems to not take moves into account, and rather, lists
these as deletes/creates. Did you use git format-patch, and if so, did
you use tne -M option?

Also, the commit summary could be less generic, something like "ARM:
vexpress: rename board ca9x4_ct_vxp as vexpress_common".

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list