Patchwork [U-Boot,RESEND,v3,1/2] ARM: IGEP0033: rename config file to am335x_igep0033.h

login
register
mail settings
Submitter Javier Martinez Canillas
Date Oct. 17, 2013, 4:30 a.m.
Message ID <1381984221-20991-1-git-send-email-javier.martinez@collabora.co.uk>
Download mbox | patch
Permalink /patch/284079/
State Not Applicable
Delegated to: Tom Rini
Headers show

Comments

Javier Martinez Canillas - Oct. 17, 2013, 4:30 a.m.
From: Javier Martinez Canillas <javier@dowhile0.org>

There seems to be a naming convention for the configuration
files for boards using the same SoC family. This makes
easier to do changes that affect different boards based
on the same SoC.

Since the IGEP COM AQUILA use TI AM335x processors, is better
to rename its board config to use this naming scheme.

Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
Acked-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
---

Changes since v2:
 - Rebase patch to adapt to new boards.cfg file format.

Changes since v1:
 - Fix some issues in the commit changelog pointed out by Enric Balletbo.

 boards.cfg                                        | 2 +-
 include/configs/{igep0033.h => am335x_igep0033.h} | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename include/configs/{igep0033.h => am335x_igep0033.h} (100%)
Javier Martinez Canillas - Oct. 24, 2013, 8:22 p.m.
On Thu, Oct 17, 2013 at 6:30 AM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
> From: Javier Martinez Canillas <javier@dowhile0.org>
>
> There seems to be a naming convention for the configuration
> files for boards using the same SoC family. This makes
> easier to do changes that affect different boards based
> on the same SoC.
>
> Since the IGEP COM AQUILA use TI AM335x processors, is better
> to rename its board config to use this naming scheme.
>
> Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> Acked-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> ---
>
> Changes since v2:
>  - Rebase patch to adapt to new boards.cfg file format.
>
> Changes since v1:
>  - Fix some issues in the commit changelog pointed out by Enric Balletbo.
>
>  boards.cfg                                        | 2 +-
>  include/configs/{igep0033.h => am335x_igep0033.h} | 0
>  2 files changed, 1 insertion(+), 1 deletion(-)
>  rename include/configs/{igep0033.h => am335x_igep0033.h} (100%)
>
> diff --git a/boards.cfg b/boards.cfg
> index aa2ee64..c8b7868 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -246,7 +246,7 @@ Active  arm         arm946es       -           armltd          integrator
>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca15_tc2                    -                                                                                                                                 -
>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca5x2                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca9x4                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
> -Active  arm         armv7          am33xx      isee            igep0033            igep0033                             -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
> +Active  arm         armv7          am33xx      isee            igep0033            am335x_igep0033                      -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
>  Active  arm         armv7          am33xx      phytec          pcm051              pcm051                               pcm051                                                                                                                            Lars Poeschel <poeschel@lemonage.de>
>  Active  arm         armv7          am33xx      siemens         dxr2                dxr2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
>  Active  arm         armv7          am33xx      siemens         pxm2                pxm2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
> diff --git a/include/configs/igep0033.h b/include/configs/am335x_igep0033.h
> similarity index 100%
> rename from include/configs/igep0033.h
> rename to include/configs/am335x_igep0033.h
> --
> 1.8.4.rc3
>

Hi Tom,

Any comments on these patches?

Thanks a lot and best regards,
Javier
Tom Rini - Oct. 25, 2013, 7:41 p.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 10/24/2013 04:22 PM, Javier Martinez Canillas wrote:
> On Thu, Oct 17, 2013 at 6:30 AM, Javier Martinez Canillas
> <javier@dowhile0.org> wrote:
>> From: Javier Martinez Canillas <javier@dowhile0.org>
>>
>> There seems to be a naming convention for the configuration
>> files for boards using the same SoC family. This makes
>> easier to do changes that affect different boards based
>> on the same SoC.
>>
>> Since the IGEP COM AQUILA use TI AM335x processors, is better
>> to rename its board config to use this naming scheme.
>>
>> Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
>> Acked-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>> ---
>>
>> Changes since v2:
>>  - Rebase patch to adapt to new boards.cfg file format.
>>
>> Changes since v1:
>>  - Fix some issues in the commit changelog pointed out by Enric Balletbo.
>>
>>  boards.cfg                                        | 2 +-
>>  include/configs/{igep0033.h => am335x_igep0033.h} | 0
>>  2 files changed, 1 insertion(+), 1 deletion(-)
>>  rename include/configs/{igep0033.h => am335x_igep0033.h} (100%)
>>
>> diff --git a/boards.cfg b/boards.cfg
>> index aa2ee64..c8b7868 100644
>> --- a/boards.cfg
>> +++ b/boards.cfg
>> @@ -246,7 +246,7 @@ Active  arm         arm946es       -           armltd          integrator
>>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca15_tc2                    -                                                                                                                                 -
>>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca5x2                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
>>  Active  arm         armv7          -           armltd          vexpress            vexpress_ca9x4                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
>> -Active  arm         armv7          am33xx      isee            igep0033            igep0033                             -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
>> +Active  arm         armv7          am33xx      isee            igep0033            am335x_igep0033                      -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
>>  Active  arm         armv7          am33xx      phytec          pcm051              pcm051                               pcm051                                                                                                                            Lars Poeschel <poeschel@lemonage.de>
>>  Active  arm         armv7          am33xx      siemens         dxr2                dxr2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
>>  Active  arm         armv7          am33xx      siemens         pxm2                pxm2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
>> diff --git a/include/configs/igep0033.h b/include/configs/am335x_igep0033.h
>> similarity index 100%
>> rename from include/configs/igep0033.h
>> rename to include/configs/am335x_igep0033.h
>> --
>> 1.8.4.rc3
>>
> 
> Hi Tom,
> 
> Any comments on these patches?

Looks good, and I do need to put on my u-boot-ti hat on and collect up
patches.

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJSasl5AAoJENk4IS6UOR1WBtIP+QFpWTpYSlV4n1p7RN7If/SN
ctpTuzXUQE0i4ySVM3WE5GgwCbwy3EXvstDEs8bcc9+BPdYXCg+wQ6q2+Vtq6DLK
FVwJP6sfW/KqpPQm9M7wgoGLkH/ylBzCjY+O90LpvBN9bx+lACjtfSxix/QzIG4j
ra5jjnxGW9t1IG1imrKkot/aVWybed1XiU87IVzctLXEOsO1MJlgwmodvBb4p1HB
v0JVKzcZu9JmYd/tYiMcokUpa/CzfEso6WjDX6wpZZzMQgM1ktZHI6Rqfx8i9hXk
wiLfddSH2J8XLZ2zV8ztU/zR05MoPQTGX7poc/Pf8IPjsQKf17hGsljS4jdCR0/T
oui7U9kKDbAb3fqDSW+nEsTxVY+YJxC5lp7YE0BPs8YZ2KI4k1+9XfMd0U8oVVM5
k8QnlIMhYfBqIlCmMMoMzcxN6OvzQc2u+rrCHjZvf8YpzaXhXcS7lDXSxTzowcLG
Y0oDIiiyg8BhdwMK+OlgqtrJ47VIb+G2/GGlCqUQHz6FCvX19T1asYuyYcr4tbC2
U9OK31FdlYvRPD5uUy7VbYxE2qmzdmKCQhM5pvWb5/Md5++l2/Vq9Fq/Eg3QNGJD
bdTWxsqOwVvHi5wM9KQcCczv7/GJxCeQlSrVkeabnlaWSiqZvfe5iwZRdaz9E3Rq
FrlRthxunLf80VDy0Cxu
=I7L2
-----END PGP SIGNATURE-----

Patch

diff --git a/boards.cfg b/boards.cfg
index aa2ee64..c8b7868 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -246,7 +246,7 @@  Active  arm         arm946es       -           armltd          integrator
 Active  arm         armv7          -           armltd          vexpress            vexpress_ca15_tc2                    -                                                                                                                                 -
 Active  arm         armv7          -           armltd          vexpress            vexpress_ca5x2                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
 Active  arm         armv7          -           armltd          vexpress            vexpress_ca9x4                       -                                                                                                                                 Matt Waddel <matt.waddel@linaro.org>
-Active  arm         armv7          am33xx      isee            igep0033            igep0033                             -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
+Active  arm         armv7          am33xx      isee            igep0033            am335x_igep0033                      -                                                                                                                                 Enric Balletbo i Serra <eballetbo@iseebcn.com>
 Active  arm         armv7          am33xx      phytec          pcm051              pcm051                               pcm051                                                                                                                            Lars Poeschel <poeschel@lemonage.de>
 Active  arm         armv7          am33xx      siemens         dxr2                dxr2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
 Active  arm         armv7          am33xx      siemens         pxm2                pxm2                                 -                                                                                                                                 Roger Meier <r.meier@siemens.com>
diff --git a/include/configs/igep0033.h b/include/configs/am335x_igep0033.h
similarity index 100%
rename from include/configs/igep0033.h
rename to include/configs/am335x_igep0033.h