[1/6] Exynos: Add CONFIG_EXYNOS4 Macro to EXYNOS4 based boards

Message ID 1324551168-1202-2-git-send-email-chander.kashyap@linaro.org
State Superseded
Headers show

Commit Message

Chander Kashyap Dec. 22, 2011, 10:52 a.m.
Exynos architecture has varients like exynos4 and exynos5. In order to
identify the exynos4 architecture, add CONFIG_EXYNOS4 macro to exynos4
based boards i.e. Origen, smdkv310 and s5pc210_universal.

Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
---
 include/configs/origen.h            |    1 +
 include/configs/s5pc210_universal.h |    1 +
 include/configs/smdkv310.h          |    1 +
 3 files changed, 3 insertions(+), 0 deletions(-)

Comments

Minkyu Kang Dec. 23, 2011, 4:22 a.m. | #1
Dear Chander Kashyap,

On 22 December 2011 19:52, Chander Kashyap <chander.kashyap@linaro.org> wrote:
> Exynos architecture has varients like exynos4 and exynos5. In order to
> identify the exynos4 architecture, add CONFIG_EXYNOS4 macro to exynos4
> based boards i.e. Origen, smdkv310 and s5pc210_universal.
>
> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> ---
>  include/configs/origen.h            |    1 +
>  include/configs/s5pc210_universal.h |    1 +
>  include/configs/smdkv310.h          |    1 +
>  3 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/include/configs/origen.h b/include/configs/origen.h
> index cd502d1..53bd5fc 100644
> --- a/include/configs/origen.h
> +++ b/include/configs/origen.h
> @@ -28,6 +28,7 @@
>  /* High Level Configuration Options */
>  #define CONFIG_SAMSUNG                 1       /* SAMSUNG core */
>  #define CONFIG_S5P                     1       /* S5P Family */
> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */

Why need this define?

>  #define CONFIG_EXYNOS4210              1       /* which is a EXYNOS4210 SoC */
>  #define CONFIG_ORIGEN                  1       /* working with ORIGEN*/
>
> diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
> index be000cb..33c4600 100644
> --- a/include/configs/s5pc210_universal.h
> +++ b/include/configs/s5pc210_universal.h
> @@ -32,6 +32,7 @@
>  */
>  #define CONFIG_SAMSUNG         1       /* in a SAMSUNG core */
>  #define CONFIG_S5P             1       /* which is in a S5P Family */
> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */
>  #define CONFIG_EXYNOS4210      1       /* which is in a EXYNOS4210 */
>  #define CONFIG_UNIVERSAL       1       /* working with Universal */
>
> diff --git a/include/configs/smdkv310.h b/include/configs/smdkv310.h
> index 93c25da..24fbac0 100644
> --- a/include/configs/smdkv310.h
> +++ b/include/configs/smdkv310.h
> @@ -28,6 +28,7 @@
>  /* High Level Configuration Options */
>  #define CONFIG_SAMSUNG                 1       /* in a SAMSUNG core */
>  #define CONFIG_S5P                     1       /* S5P Family */
> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */
>  #define CONFIG_EXYNOS4210              1       /* which is a EXYNOS4210 SoC */
>  #define CONFIG_SMDKV310                        1       /* working with SMDKV310*/
>

Thanks.
Minkyu Kang.
Chander Kashyap Dec. 23, 2011, 6:28 a.m. | #2
Dear Minkyu Kang,

On 23 December 2011 09:52, Minkyu Kang <promsoft@gmail.com> wrote:
> Dear Chander Kashyap,
>
> On 22 December 2011 19:52, Chander Kashyap <chander.kashyap@linaro.org> wrote:
>> Exynos architecture has varients like exynos4 and exynos5. In order to
>> identify the exynos4 architecture, add CONFIG_EXYNOS4 macro to exynos4
>> based boards i.e. Origen, smdkv310 and s5pc210_universal.
>>
>> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
>> ---
>>  include/configs/origen.h            |    1 +
>>  include/configs/s5pc210_universal.h |    1 +
>>  include/configs/smdkv310.h          |    1 +
>>  3 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/configs/origen.h b/include/configs/origen.h
>> index cd502d1..53bd5fc 100644
>> --- a/include/configs/origen.h
>> +++ b/include/configs/origen.h
>> @@ -28,6 +28,7 @@
>>  /* High Level Configuration Options */
>>  #define CONFIG_SAMSUNG                 1       /* SAMSUNG core */
>>  #define CONFIG_S5P                     1       /* S5P Family */
>> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */
>
> Why need this define?
This is required for conditional inclusion of SoC specific header file.
>
>>  #define CONFIG_EXYNOS4210              1       /* which is a EXYNOS4210 SoC */
>>  #define CONFIG_ORIGEN                  1       /* working with ORIGEN*/
>>
>> diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
>> index be000cb..33c4600 100644
>> --- a/include/configs/s5pc210_universal.h
>> +++ b/include/configs/s5pc210_universal.h
>> @@ -32,6 +32,7 @@
>>  */
>>  #define CONFIG_SAMSUNG         1       /* in a SAMSUNG core */
>>  #define CONFIG_S5P             1       /* which is in a S5P Family */
>> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */
>>  #define CONFIG_EXYNOS4210      1       /* which is in a EXYNOS4210 */
>>  #define CONFIG_UNIVERSAL       1       /* working with Universal */
>>
>> diff --git a/include/configs/smdkv310.h b/include/configs/smdkv310.h
>> index 93c25da..24fbac0 100644
>> --- a/include/configs/smdkv310.h
>> +++ b/include/configs/smdkv310.h
>> @@ -28,6 +28,7 @@
>>  /* High Level Configuration Options */
>>  #define CONFIG_SAMSUNG                 1       /* in a SAMSUNG core */
>>  #define CONFIG_S5P                     1       /* S5P Family */
>> +#define CONFIG_EXYNOS4                 1       /* EXYNOS4 Family */
>>  #define CONFIG_EXYNOS4210              1       /* which is a EXYNOS4210 SoC */
>>  #define CONFIG_SMDKV310                        1       /* working with SMDKV310*/
>>
>
> Thanks.
> Minkyu Kang.
> --
> from. prom.
> www.promsoft.net

Patch

diff --git a/include/configs/origen.h b/include/configs/origen.h
index cd502d1..53bd5fc 100644
--- a/include/configs/origen.h
+++ b/include/configs/origen.h
@@ -28,6 +28,7 @@ 
 /* High Level Configuration Options */
 #define CONFIG_SAMSUNG			1	/* SAMSUNG core */
 #define CONFIG_S5P			1	/* S5P Family */
+#define CONFIG_EXYNOS4			1	/* EXYNOS4 Family */
 #define CONFIG_EXYNOS4210		1	/* which is a EXYNOS4210 SoC */
 #define CONFIG_ORIGEN			1	/* working with ORIGEN*/
 
diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
index be000cb..33c4600 100644
--- a/include/configs/s5pc210_universal.h
+++ b/include/configs/s5pc210_universal.h
@@ -32,6 +32,7 @@ 
  */
 #define CONFIG_SAMSUNG		1	/* in a SAMSUNG core */
 #define CONFIG_S5P		1	/* which is in a S5P Family */
+#define CONFIG_EXYNOS4			1	/* EXYNOS4 Family */
 #define CONFIG_EXYNOS4210	1	/* which is in a EXYNOS4210 */
 #define CONFIG_UNIVERSAL	1	/* working with Universal */
 
diff --git a/include/configs/smdkv310.h b/include/configs/smdkv310.h
index 93c25da..24fbac0 100644
--- a/include/configs/smdkv310.h
+++ b/include/configs/smdkv310.h
@@ -28,6 +28,7 @@ 
 /* High Level Configuration Options */
 #define CONFIG_SAMSUNG			1	/* in a SAMSUNG core */
 #define CONFIG_S5P			1	/* S5P Family */
+#define CONFIG_EXYNOS4			1	/* EXYNOS4 Family */
 #define CONFIG_EXYNOS4210		1	/* which is a EXYNOS4210 SoC */
 #define CONFIG_SMDKV310			1	/* working with SMDKV310*/