diff mbox

[U-Boot,v3,2/2] config_fallbacks: default CONFIG_SYS_MMC_ENV_DEV to 0

Message ID 1390480760-20564-2-git-send-email-hector.palacios@digi.com
State Changes Requested
Delegated to: Pantelis Antoniou
Headers show

Commit Message

Hector Palacios Jan. 23, 2014, 12:39 p.m. UTC 
Since function mmc_get_env_devno is __weak and can be overridden by
board code, boards do not need to mandatory define
CONFIG_SYS_MMC_ENV_DEV.
If the constant is not defined, define it to 0 by default.

Signed-off-by: Hector Palacios <hector.palacios@digi.com>
Reviewed-by: Stephen Warren <swarren@nvidia.com>
---

Notes:
    Changes since v2:
    - Move CONFIG_SYS_MMC_ENV_DEV to config_fallbacks
    Changes since v1:
    - Use default define if not set

 include/config_fallbacks.h | 3 +++
 1 file changed, 3 insertions(+)

Comments

Otavio Salvador Jan. 23, 2014, 12:43 p.m. UTC | #1 
On Thu, Jan 23, 2014 at 10:39 AM, Hector Palacios
<hector.palacios@digi.com> wrote:
> Since function mmc_get_env_devno is __weak and can be overridden by
> board code, boards do not need to mandatory define
> CONFIG_SYS_MMC_ENV_DEV.
> If the constant is not defined, define it to 0 by default.
>
> Signed-off-by: Hector Palacios <hector.palacios@digi.com>
> Reviewed-by: Stephen Warren <swarren@nvidia.com>


Acked-by: Otavio Salvador <otavio@ossystems.com.br>
diff mbox

Patch

diff --git a/include/config_fallbacks.h b/include/config_fallbacks.h
index 7a6f55feb12e..b17fb28d90b9 100644
--- a/include/config_fallbacks.h
+++ b/include/config_fallbacks.h
@@ -54,6 +54,9 @@ 
 #endif
 
 #if defined(CONFIG_MMC)
+#if !defined(CONFIG_SYS_MMC_ENV_DEV)
+#define CONFIG_SYS_MMC_ENV_DEV 0
+#endif
 #if !defined(CONFIG_SYS_MMC_ENV_PART)
 #define CONFIG_SYS_MMC_ENV_PART 0
 #endif