| Message ID | 1381984221-20991-1-git-send-email-javier.martinez@collabora.co.uk |
|---|---|
| State | Not Applicable |
| Delegated to: | Tom Rini |
| Headers | show |
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
-----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-----
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