From: Christopher Harvey <charvey@matrox.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/5] Removed unused define, CONFIG_ARMV7.
Date: Mon, 4 Jul 2011 14:46:59 -0400 [thread overview]
Message-ID: <20110704184659.GG3016@harvey-pc.matrox.com> (raw)
In-Reply-To: <20110704180012.GB30477@titan.lakedaemon.net>
On Mon, Jul 04, 2011 at 02:00:12PM -0400, Jason wrote:
> On Mon, Jul 04, 2011 at 01:44:26PM -0400, Christopher Harvey wrote:
> > Signed-off-by: Christopher Harvey <charvey@matrox.com>
> > ---
> > include/configs/am3517_crane.h | 2 +-
> > include/configs/am3517_evm.h | 2 +-
> > include/configs/ca9x4_ct_vxp.h | 2 +-
> > include/configs/cm_t35.h | 2 +-
> > include/configs/devkit8000.h | 2 +-
> > include/configs/dig297.h | 2 +-
> > include/configs/igep0020.h | 2 +-
> > include/configs/igep0030.h | 2 +-
> > include/configs/omap3_beagle.h | 2 +-
> > include/configs/omap3_evm.h | 2 +-
> > include/configs/omap3_overo.h | 2 +-
> > include/configs/omap3_pandora.h | 2 +-
> > include/configs/omap3_sdp3430.h | 2 +-
> > include/configs/omap3_zoom1.h | 2 +-
> > include/configs/omap3_zoom2.h | 2 +-
> > include/configs/omap4_panda.h | 2 +-
> > include/configs/omap4_sdp4430.h | 2 +-
> > include/configs/s5p_goni.h | 2 +-
> > include/configs/s5pc210_universal.h | 2 +-
> > include/configs/smdkc100.h | 2 +-
> > include/configs/smdkv310.h | 2 +-
> > 21 files changed, 21 insertions(+), 21 deletions(-)
> >
> > diff --git a/include/configs/am3517_crane.h b/include/configs/am3517_crane.h
> > index 09cb951..b809053 100644
> > --- a/include/configs/am3517_crane.h
> > +++ b/include/configs/am3517_crane.h
> > @@ -28,7 +28,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
>
> Need the empty line that wasn't there before? Same Q for every file in
> this patch.
I generated this patch with sed actually. I could easily re-run it and
tell it to remove the newlines. Lets see if it gets
considered/accepted first.
>
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3_AM3517CRANE 1 /* working with CRANEBOARD */
> > diff --git a/include/configs/am3517_evm.h b/include/configs/am3517_evm.h
> > index 80ad342..db026c4 100644
> > --- a/include/configs/am3517_evm.h
> > +++ b/include/configs/am3517_evm.h
> > @@ -28,7 +28,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3_AM3517EVM 1 /* working with AM3517EVM */
> > diff --git a/include/configs/ca9x4_ct_vxp.h b/include/configs/ca9x4_ct_vxp.h
> > index 7f83249..fd92137 100644
> > --- a/include/configs/ca9x4_ct_vxp.h
> > +++ b/include/configs/ca9x4_ct_vxp.h
> > @@ -33,7 +33,7 @@
> > #define CONFIG_SYS_TEXT_BASE 0x60800000
> >
> > /* High Level Configuration Options */
> > -#define CONFIG_ARMV7 1
> > +
> >
>
> If that was the only remaining high level config option, should the
> comment be removed?
I figured maintainers would get to it eventually.
>
> > #define CONFIG_SYS_MEMTEST_START 0x60000000
> > #define CONFIG_SYS_MEMTEST_END 0x20000000
> > diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
> > index 93a1b26..b4cec35 100644
> > --- a/include/configs/cm_t35.h
> > +++ b/include/configs/cm_t35.h
> > @@ -36,7 +36,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
> > index 125c690..f97a4ed 100644
> > --- a/include/configs/devkit8000.h
> > +++ b/include/configs/devkit8000.h
> > @@ -32,7 +32,7 @@
> > #define __CONFIG_H
> >
> > /* High Level Configuration Options */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/dig297.h b/include/configs/dig297.h
> > index 7aeb24e..ee0c6be 100644
> > --- a/include/configs/dig297.h
> > +++ b/include/configs/dig297.h
> > @@ -35,7 +35,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX /* which is a 34XX */
> > #define CONFIG_OMAP3430 /* which is in a 3430 */
> > diff --git a/include/configs/igep0020.h b/include/configs/igep0020.h
> > index 5af9bec..1c36bc2 100644
> > --- a/include/configs/igep0020.h
> > +++ b/include/configs/igep0020.h
> > @@ -25,7 +25,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/igep0030.h b/include/configs/igep0030.h
> > index 92144af..8594b87 100644
> > --- a/include/configs/igep0030.h
> > +++ b/include/configs/igep0030.h
> > @@ -25,7 +25,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h
> > index 9fd80ed..aa602e4 100644
> > --- a/include/configs/omap3_beagle.h
> > +++ b/include/configs/omap3_beagle.h
> > @@ -31,7 +31,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
> > index 13a4fbf..d07c63e 100644
> > --- a/include/configs/omap3_evm.h
> > +++ b/include/configs/omap3_evm.h
> > @@ -36,7 +36,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_overo.h b/include/configs/omap3_overo.h
> > index 242b317..127ff47 100644
> > --- a/include/configs/omap3_overo.h
> > +++ b/include/configs/omap3_overo.h
> > @@ -23,7 +23,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
> > index 39c87a8..e8caa8c 100644
> > --- a/include/configs/omap3_pandora.h
> > +++ b/include/configs/omap3_pandora.h
> > @@ -26,7 +26,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_sdp3430.h b/include/configs/omap3_sdp3430.h
> > index 55bbcd4..2a1a13d 100644
> > --- a/include/configs/omap3_sdp3430.h
> > +++ b/include/configs/omap3_sdp3430.h
> > @@ -36,7 +36,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_zoom1.h b/include/configs/omap3_zoom1.h
> > index 9183849..beac7b0 100644
> > --- a/include/configs/omap3_zoom1.h
> > +++ b/include/configs/omap3_zoom1.h
> > @@ -32,7 +32,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap3_zoom2.h b/include/configs/omap3_zoom2.h
> > index 3573edf..214c13b 100644
> > --- a/include/configs/omap3_zoom2.h
> > +++ b/include/configs/omap3_zoom2.h
> > @@ -33,7 +33,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP34XX 1 /* which is a 34XX */
> > #define CONFIG_OMAP3430 1 /* which is in a 3430 */
> > diff --git a/include/configs/omap4_panda.h b/include/configs/omap4_panda.h
> > index b4e7f41..d26f903 100644
> > --- a/include/configs/omap4_panda.h
> > +++ b/include/configs/omap4_panda.h
> > @@ -30,7 +30,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP44XX 1 /* which is a 44XX */
> > #define CONFIG_OMAP4430 1 /* which is in a 4430 */
> > diff --git a/include/configs/omap4_sdp4430.h b/include/configs/omap4_sdp4430.h
> > index 584a52b..dee32bc 100644
> > --- a/include/configs/omap4_sdp4430.h
> > +++ b/include/configs/omap4_sdp4430.h
> > @@ -31,7 +31,7 @@
> > /*
> > * High Level Configuration Options
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_OMAP 1 /* in a TI OMAP core */
> > #define CONFIG_OMAP44XX 1 /* which is a 44XX */
> > #define CONFIG_OMAP4430 1 /* which is in a 4430 */
> > diff --git a/include/configs/s5p_goni.h b/include/configs/s5p_goni.h
> > index d648ce8..5309d57 100644
> > --- a/include/configs/s5p_goni.h
> > +++ b/include/configs/s5p_goni.h
> > @@ -28,7 +28,7 @@
> > #define __CONFIG_H
> >
> > /* High Level Configuration Options */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_SAMSUNG 1 /* in a SAMSUNG core */
> > #define CONFIG_S5P 1 /* which is in a S5P Family */
> > #define CONFIG_S5PC110 1 /* which is in a S5PC110 */
> > diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
> > index 5915984..b410f97 100644
> > --- a/include/configs/s5pc210_universal.h
> > +++ b/include/configs/s5pc210_universal.h
> > @@ -30,7 +30,7 @@
> > * High Level Configuration Options
> > * (easy to change)
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_SAMSUNG 1 /* in a SAMSUNG core */
> > #define CONFIG_S5P 1 /* which is in a S5P Family */
> > #define CONFIG_S5PC210 1 /* which is in a S5PC210 */
> > diff --git a/include/configs/smdkc100.h b/include/configs/smdkc100.h
> > index 70e23b5..b2f8a00 100644
> > --- a/include/configs/smdkc100.h
> > +++ b/include/configs/smdkc100.h
> > @@ -32,7 +32,7 @@
> > * High Level Configuration Options
> > * (easy to change)
> > */
> > -#define CONFIG_ARMV7 1 /* This is an ARM V7 CPU core */
> > +
> > #define CONFIG_SAMSUNG 1 /* in a SAMSUNG core */
> > #define CONFIG_S5P 1 /* which is in a S5P Family */
> > #define CONFIG_S5PC100 1 /* which is in a S5PC100 */
> > diff --git a/include/configs/smdkv310.h b/include/configs/smdkv310.h
> > index a7f5850..5d0806f 100644
> > --- a/include/configs/smdkv310.h
> > +++ b/include/configs/smdkv310.h
> > @@ -26,7 +26,7 @@
> > #define __CONFIG_H
> >
> > /* High Level Configuration Options */
> > -#define CONFIG_ARMV7 1 /*This is an ARM V7 CPU core */
> > +
> > #define CONFIG_SAMSUNG 1 /* in a SAMSUNG core */
> > #define CONFIG_S5P 1 /* S5P Family */
> > #define CONFIG_S5PC210 1 /* which is in a S5PC210 SoC */
> > --
> > 1.7.3.4
>
> thx,
>
> Jason.
thanks for the input,
-Chris
next prev parent reply other threads:[~2011-07-04 18:46 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1309799825.git.charvey@matrox.com>
2011-07-04 17:43 ` [U-Boot] [PATCH 1/5] Added documentation for CONFIG_SYS_TEXT_BASE for ARM Christopher Harvey
2011-07-04 19:39 ` Wolfgang Denk
2011-07-04 17:43 ` [U-Boot] [PATCH 2/5] Added extra documentation about how the relocation address to RAM is picked " Christopher Harvey
2011-07-04 19:43 ` Wolfgang Denk
2011-07-06 20:58 ` Christopher Harvey
2011-07-06 21:29 ` Wolfgang Denk
2011-07-07 16:10 ` Albert ARIBAUD
2011-07-04 17:44 ` [U-Boot] [PATCH 3/5] Removed unused define, CONFIG_ARMV7 Christopher Harvey
2011-07-04 18:00 ` Jason
2011-07-04 18:46 ` Christopher Harvey [this message]
2011-07-04 19:47 ` Wolfgang Denk
2011-07-04 17:45 ` [U-Boot] [PATCH 4/5] Don't compile in large memory test function by default Christopher Harvey
2011-07-07 16:13 ` Albert ARIBAUD
2011-07-04 17:45 ` [U-Boot] [PATCH 5/5] Warn when the machine ID isn't passed to an ARM kernel and u-boot is compiled in debug mode. The kernel cannot boot without it Christopher Harvey
2011-07-04 18:08 ` Jason
2011-07-04 18:55 ` Christopher Harvey
2011-07-04 19:56 ` Wolfgang Denk
2011-07-04 20:13 ` Jason
2011-07-04 20:32 ` Christopher Harvey
2011-07-04 21:24 ` Jason
2011-07-05 7:21 ` Igor Grinberg
2011-07-05 7:31 ` Igor Grinberg
2011-07-04 19:53 ` Wolfgang Denk
2011-07-05 7:38 ` Igor Grinberg
2011-07-05 10:04 ` Wolfgang Denk
2011-07-05 10:46 ` Igor Grinberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110704184659.GG3016@harvey-pc.matrox.com \
--to=charvey@matrox.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox