[U-Boot] [PATCH 1/7] mx6ul_14x14_evk: Remove CONFIG_CMD_FUSE option

From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_CMD_FUSE and CONFIG_MXC_OCOTP are selected by mx6_common.h, so there is no need to define them locally.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 5 ----- 1 file changed, 5 deletions(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index e785925..08738c3 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -33,11 +33,6 @@ #define CONFIG_MXC_UART #define CONFIG_MXC_UART_BASE UART1_BASE
-#define CONFIG_CMD_FUSE -#ifdef CONFIG_CMD_FUSE -#define CONFIG_MXC_OCOTP -#endif - /* MMC Configs */ #ifdef CONFIG_FSL_USDHC #define CONFIG_SYS_FSL_ESDHC_ADDR USDHC2_BASE_ADDR

Hi Fabio,
I have one patch to fix this and 2/7: https://patchwork.ozlabs.org/patch/512680/
Regards, Peng. On Wed, Sep 09, 2015 at 11:16:38AM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_CMD_FUSE and CONFIG_MXC_OCOTP are selected by mx6_common.h, so there is no need to define them locally.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com
include/configs/mx6ul_14x14_evk.h | 5 ----- 1 file changed, 5 deletions(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index e785925..08738c3 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -33,11 +33,6 @@ #define CONFIG_MXC_UART #define CONFIG_MXC_UART_BASE UART1_BASE
-#define CONFIG_CMD_FUSE -#ifdef CONFIG_CMD_FUSE -#define CONFIG_MXC_OCOTP -#endif
/* MMC Configs */ #ifdef CONFIG_FSL_USDHC
#define CONFIG_SYS_FSL_ESDHC_ADDR USDHC2_BASE_ADDR
1.9.1
--

From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_CMD_FUSE and CONFIG_MXC_OCOTP are selected by mx6_common.h, so there is no need to define them locally.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/cgtqmx6eval.h | 5 ----- 1 file changed, 5 deletions(-)
diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h index 92930c8..0979728 100644 --- a/include/configs/cgtqmx6eval.h +++ b/include/configs/cgtqmx6eval.h @@ -35,11 +35,6 @@ /* Thermal support */ #define CONFIG_IMX6_THERMAL
-#define CONFIG_CMD_FUSE -#if defined(CONFIG_CMD_FUSE) || defined(CONFIG_IMX6_THERMAL) -#define CONFIG_MXC_OCOTP -#endif - /* I2C Configs */ #define CONFIG_CMD_I2C #define CONFIG_SYS_I2C

From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_SYS_GENERIC_BOARD is selected by mx6_common.h, so there is no need to define it locally.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 08738c3..9232414 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -20,7 +20,6 @@ #include "imx6_spl.h"
#define CONFIG_ROM_UNIFIED_SECTIONS -#define CONFIG_SYS_GENERIC_BOARD #define CONFIG_DISPLAY_CPUINFO #define CONFIG_DISPLAY_BOARDINFO

On Wed, Sep 09, 2015 at 11:16:40AM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_SYS_GENERIC_BOARD is selected by mx6_common.h, so there is no need to define it locally.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com
include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 08738c3..9232414 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -20,7 +20,6 @@ #include "imx6_spl.h"
#define CONFIG_ROM_UNIFIED_SECTIONS -#define CONFIG_SYS_GENERIC_BOARD #define CONFIG_DISPLAY_CPUINFO #define CONFIG_DISPLAY_BOARDINFO
Reviewed-by: Peng Fan Peng.Fan@freescale.com
Regards, Peng.
-- 1.9.1
--

From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_ROM_UNIFIED_SECTIONS is not used anywhere, so let's remove it.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 9232414..b27541d 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -19,7 +19,6 @@ #define CONFIG_SPL_MMC_SUPPORT #include "imx6_spl.h"
-#define CONFIG_ROM_UNIFIED_SECTIONS #define CONFIG_DISPLAY_CPUINFO #define CONFIG_DISPLAY_BOARDINFO

On Wed, Sep 09, 2015 at 11:16:41AM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_ROM_UNIFIED_SECTIONS is not used anywhere, so let's remove it.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com
include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 9232414..b27541d 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -19,7 +19,6 @@ #define CONFIG_SPL_MMC_SUPPORT #include "imx6_spl.h"
-#define CONFIG_ROM_UNIFIED_SECTIONS #define CONFIG_DISPLAY_CPUINFO #define CONFIG_DISPLAY_BOARDINFO
Reviewed-by: Peng Fan Peng.Fan@freescale.com
Regards, Peng.
-- 1.9.1
--

From: Fabio Estevam fabio.estevam@freescale.com
There is no need to undef the config options.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index b27541d..84f26ec 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -45,13 +45,6 @@ #define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */ #endif
-#undef CONFIG_BOOTM_NETBSD -#undef CONFIG_BOOTM_PLAN9 -#undef CONFIG_BOOTM_RTEMS - -#undef CONFIG_CMD_EXPORTENV -#undef CONFIG_CMD_IMPORTENV - /* I2C configs */ #define CONFIG_CMD_I2C #ifdef CONFIG_CMD_I2C @@ -62,8 +55,6 @@
#define PHYS_SDRAM_SIZE SZ_512M
-#undef CONFIG_CMD_IMLS - #define CONFIG_SYS_MMC_IMG_LOAD_PART 1
#define CONFIG_EXTRA_ENV_SETTINGS \

On Wed, Sep 09, 2015 at 11:16:42AM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.estevam@freescale.com
There is no need to undef the config options.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com
include/configs/mx6ul_14x14_evk.h | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index b27541d..84f26ec 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -45,13 +45,6 @@ #define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */ #endif
-#undef CONFIG_BOOTM_NETBSD -#undef CONFIG_BOOTM_PLAN9 -#undef CONFIG_BOOTM_RTEMS
-#undef CONFIG_CMD_EXPORTENV -#undef CONFIG_CMD_IMPORTENV
/* I2C configs */ #define CONFIG_CMD_I2C #ifdef CONFIG_CMD_I2C @@ -62,8 +55,6 @@
#define PHYS_SDRAM_SIZE SZ_512M
-#undef CONFIG_CMD_IMLS
#define CONFIG_SYS_MMC_IMG_LOAD_PART 1
#define CONFIG_EXTRA_ENV_SETTINGS \
Reviewed-by: Peng Fan Peng.Fan@freescale.com
Regards, Peng.
-- 1.9.1
--

From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_FEC_DMA_MINALIGN is not used anywhere, so let's remove it.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 84f26ec..ff4f231 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -221,7 +221,6 @@
#define CONFIG_PHYLIB #define CONFIG_PHY_MICREL -#define CONFIG_FEC_DMA_MINALIGN 64 #endif
#define CONFIG_IMX6_THERMAL

On Wed, Sep 09, 2015 at 11:16:43AM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.estevam@freescale.com
CONFIG_FEC_DMA_MINALIGN is not used anywhere, so let's remove it.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com
include/configs/mx6ul_14x14_evk.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index 84f26ec..ff4f231 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -221,7 +221,6 @@
#define CONFIG_PHYLIB #define CONFIG_PHY_MICREL -#define CONFIG_FEC_DMA_MINALIGN 64 #endif
Reviewed-by: Peng Fan Peng.Fan@freescale.com
Regards, Peng.
#define CONFIG_IMX6_THERMAL
1.9.1
--

From: Fabio Estevam fabio.estevam@freescale.com
Entering the maximum number of characters defined by CONFIG_SYS_CBSIZE into the console and hitting enter afterwards, causes a hang in the system because CONFIG_SYS_PBSIZE is not capable of storing the extra characters of the error message: "Unknown command '' - try 'help'".
Use the default CONFIG_SYS_PBSIZE definition from config_fallbacks.h to solve this problem.
Signed-off-by: Fabio Estevam fabio.estevam@freescale.com --- include/configs/mx6ul_14x14_evk.h | 3 --- 1 file changed, 3 deletions(-)
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h index ff4f231..4285116 100644 --- a/include/configs/mx6ul_14x14_evk.h +++ b/include/configs/mx6ul_14x14_evk.h @@ -133,9 +133,6 @@ "else run netboot; fi"
/* Miscellaneous configurable options */ -/* Print Buffer Size */ -#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + sizeof(CONFIG_SYS_PROMPT) + 16) - #define CONFIG_CMD_MEMTEST #define CONFIG_SYS_MEMTEST_START 0x80000000 #define CONFIG_SYS_MEMTEST_END (CONFIG_SYS_MEMTEST_START + 0x10000000)
participants (2)
-
Fabio Estevam
-
Peng Fan