[U-Boot] [PATCH v5 4/4] ARM: SAMSUNG: support sdhci controller

Prabhakar Lad prabhakar.csengg at gmail.com
Fri Apr 13 08:16:24 CEST 2012


Hi Jaehoon,

On Thu, Apr 12, 2012 at 4:29 PM, Jaehoon Chung <jh80.chung at samsung.com> wrote:
> To support sdhci controller, remove the CONFIG_S5P_MMC..
> Instead, use the CONFIG_S5P_SDHCI/CONFIG_SDHCI.
>
> Signed-off-by: Jaehoon Chung <jh80.chung at samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> Acked-by: Lei Wen<leiwen at marvell.com>
> ---
>  include/configs/origen.h            |    3 ++-
>  include/configs/s5p_goni.h          |    3 ++-
>  include/configs/s5pc210_universal.h |    3 ++-
>  include/configs/smdk5250.h          |    3 ++-
>  include/configs/smdkv310.h          |    3 ++-
>  include/configs/trats.h             |    3 ++-
>  6 files changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/include/configs/origen.h b/include/configs/origen.h
> index 8ede825..ea8066f 100644
> --- a/include/configs/origen.h
> +++ b/include/configs/origen.h
> @@ -71,7 +71,8 @@
>  /* SD/MMC configuration */
>  #define CONFIG_GENERIC_MMC             1
>  #define CONFIG_MMC                     1
> -#define CONFIG_S5P_MMC                 1
> +#define CONFIG_SDHCI                   1
> +#define CONFIG_S5P_SDHCI               1
>
>  /* PWM */
>  #define CONFIG_PWM                     1
> diff --git a/include/configs/s5p_goni.h b/include/configs/s5p_goni.h
> index 56b5547..f659699 100644
> --- a/include/configs/s5p_goni.h
> +++ b/include/configs/s5p_goni.h
> @@ -65,7 +65,8 @@
>  /* MMC */
>  #define CONFIG_GENERIC_MMC             1
>  #define CONFIG_MMC                     1
> -#define CONFIG_S5P_MMC                 1
> +#define CONFIG_SDHCI                   1
> +#define CONFIG_S5P_SDHCI               1
>
>  /* PWM */
>  #define CONFIG_PWM                     1
> diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
> index 1301275..241d209 100644
> --- a/include/configs/s5pc210_universal.h
> +++ b/include/configs/s5pc210_universal.h
> @@ -68,7 +68,8 @@
>  /* MMC */
>  #define CONFIG_GENERIC_MMC     1
>  #define CONFIG_MMC             1
> -#define CONFIG_S5P_MMC         1
> +#define CONFIG_SDHCI           1
> +#define CONFIG_S5P_SDHCI       1
>
>  /* PWM */
>  #define CONFIG_PWM                     1
> diff --git a/include/configs/smdk5250.h b/include/configs/smdk5250.h
> index 9659f9e..0f63040 100644
> --- a/include/configs/smdk5250.h
> +++ b/include/configs/smdk5250.h
> @@ -78,7 +78,8 @@
>  /* SD/MMC configuration */
>  #define CONFIG_GENERIC_MMC
>  #define CONFIG_MMC
> -#define CONFIG_S5P_MMC
> +#define CONFIG_SDHCI
> +#define CONFIG_S5P_SDHCI
    There is no consistency for the #define  CONFIG_S5P_SDHCI,
    few have been assigned to value 1, And looking at the patchset
    the value of it is not used if I am not wrong, depending or defined
    or not some file is built. Best way could be not to assign any value
    for such cases, just  #define CONFIG_S5P_SDHCI would make
    sense. It would be good If you also do the same for other CONFIG'S
    defined.

Thx,
--Prabhakar Lad

>
>  #define CONFIG_BOARD_EARLY_INIT_F
>
> diff --git a/include/configs/smdkv310.h b/include/configs/smdkv310.h
> index 93c25da..077f0e0 100644
> --- a/include/configs/smdkv310.h
> +++ b/include/configs/smdkv310.h
> @@ -70,7 +70,8 @@
>  /* SD/MMC configuration */
>  #define CONFIG_GENERIC_MMC             1
>  #define CONFIG_MMC                     1
> -#define CONFIG_S5P_MMC                 1
> +#define CONFIG_SDHCI                   1
> +#define CONFIG_S5P_SDHCI               1
>
>  /* PWM */
>  #define CONFIG_PWM                     1
> diff --git a/include/configs/trats.h b/include/configs/trats.h
> index 10f11d9..71b9393 100644
> --- a/include/configs/trats.h
> +++ b/include/configs/trats.h
> @@ -74,7 +74,8 @@
>  /* MMC */
>  #define CONFIG_GENERIC_MMC
>  #define CONFIG_MMC
> -#define CONFIG_S5P_MMC
> +#define CONFIG_S5P_SDHCI
> +#define CONFIG_SDHCI
>
>  /* PWM */
>  #define CONFIG_PWM
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list