diff mbox series

[1/8] pfla02: Rework excluding NAND from SPL

Message ID 20200526190619.29583-1-trini@konsulko.com
State Accepted
Commit 049756c2fee45bc4bc9054dbe024c3c80584b446
Headers show
Series [1/8] pfla02: Rework excluding NAND from SPL | expand

Commit Message

Tom Rini May 26, 2020, 7:06 p.m. UTC
Rather than only enable CONFIG_CMD_NAND for non-SPL builds, move the CMD
options to defconfig and rework the guards to not try and call the
function in SPL builds.

Cc: Stefano Babic <sbabic at denx.de>
Signed-off-by: Tom Rini <trini at konsulko.com>
---
A grep around for setup_gpmi_nand() shows that other platforms will just
make an empty function in similar situations.
---
 board/phytec/pfla02/pfla02.c | 8 ++++----
 configs/pfla02_defconfig     | 2 +-
 include/configs/pfla02.h     | 4 ----
 3 files changed, 5 insertions(+), 9 deletions(-)

Comments

Stefano Babic May 27, 2020, 10:50 a.m. UTC | #1
On 26.05.20 21:06, Tom Rini wrote:
> Rather than only enable CONFIG_CMD_NAND for non-SPL builds, move the CMD
> options to defconfig and rework the guards to not try and call the
> function in SPL builds.
> 
> Cc: Stefano Babic <sbabic at denx.de>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> A grep around for setup_gpmi_nand() shows that other platforms will just
> make an empty function in similar situations.
> ---
>  board/phytec/pfla02/pfla02.c | 8 ++++----
>  configs/pfla02_defconfig     | 2 +-
>  include/configs/pfla02.h     | 4 ----
>  3 files changed, 5 insertions(+), 9 deletions(-)
> 
> diff --git a/board/phytec/pfla02/pfla02.c b/board/phytec/pfla02/pfla02.c
> index fed8f52e3e1d..0a961cc8a5a4 100644
> --- a/board/phytec/pfla02/pfla02.c
> +++ b/board/phytec/pfla02/pfla02.c
> @@ -117,7 +117,7 @@ static iomux_v3_cfg_t const gpios_pads[] = {
>  	IOMUX_PADS(PAD_SD4_DAT3__GPIO2_IO11 | MUX_PAD_CTRL(NO_PAD_CTRL)),
>  };
>  
> -#ifdef CONFIG_CMD_NAND
> +#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
>  /* NAND */
>  static iomux_v3_cfg_t const nfc_pads[] = {
>  	IOMUX_PADS(PAD_NANDF_CLE__NAND_CLE	| MUX_PAD_CTRL(NAND_PAD_CTRL)),
> @@ -274,7 +274,7 @@ static void setup_gpios(void)
>  	SETUP_IOMUX_PADS(gpios_pads);
>  }
>  
> -#ifdef CONFIG_CMD_NAND
> +#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
>  static void setup_gpmi_nand(void)
>  {
>  	struct mxc_ccm_reg *mxc_ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
> @@ -361,7 +361,7 @@ int board_init(void)
>  
>  	setup_gpios();
>  
> -#ifdef CONFIG_CMD_NAND
> +#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
>  	setup_gpmi_nand();
>  #endif
>  	return 0;
> @@ -657,7 +657,7 @@ void board_init_f(ulong dummy)
>  		.refr = 7,	/* 8 refresh commands per refresh cycle */
>  	};
>  
> -#ifdef CONFIG_CMD_NAND
> +#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
>  	/* Enable NAND */
>  	setup_gpmi_nand();
>  #endif
> diff --git a/configs/pfla02_defconfig b/configs/pfla02_defconfig
> index 5e5bbb05bd5e..52cebd9e22ed 100644
> --- a/configs/pfla02_defconfig
> +++ b/configs/pfla02_defconfig
> @@ -37,7 +37,7 @@ CONFIG_SPL_YMODEM_SUPPORT=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_CMD_MMC=y
> -# CONFIG_CMD_NAND is not set
> +CONFIG_CMD_NAND_TRIMFFS=y
>  CONFIG_CMD_SF=y
>  CONFIG_CMD_CACHE=y
>  CONFIG_CMD_EXT4_WRITE=y
> diff --git a/include/configs/pfla02.h b/include/configs/pfla02.h
> index d4a7de7df6a2..4162a71ca661 100644
> --- a/include/configs/pfla02.h
> +++ b/include/configs/pfla02.h
> @@ -41,15 +41,11 @@
>  #define CONFIG_SYS_I2C_MXC_I2C1		/* enable I2C bus 0 */
>  #define CONFIG_SYS_I2C_SPEED		  100000
>  
> -#ifndef CONFIG_SPL_BUILD
> -#define CONFIG_CMD_NAND
>  /* Enable NAND support */
> -#define CONFIG_CMD_NAND_TRIMFFS
>  #define CONFIG_SYS_MAX_NAND_DEVICE	1
>  #define CONFIG_SYS_NAND_BASE		0x40000000
>  #define CONFIG_SYS_NAND_5_ADDR_CYCLE
>  #define CONFIG_SYS_NAND_ONFI_DETECTION
> -#endif
>  
>  /* DMA stuff, needed for GPMI/MXS NAND support */
>  
> 

Acked-by: Stefano Babic <sbabic at denx.de>

BR,
Stefano
diff mbox series

Patch

diff --git a/board/phytec/pfla02/pfla02.c b/board/phytec/pfla02/pfla02.c
index fed8f52e3e1d..0a961cc8a5a4 100644
--- a/board/phytec/pfla02/pfla02.c
+++ b/board/phytec/pfla02/pfla02.c
@@ -117,7 +117,7 @@  static iomux_v3_cfg_t const gpios_pads[] = {
 	IOMUX_PADS(PAD_SD4_DAT3__GPIO2_IO11 | MUX_PAD_CTRL(NO_PAD_CTRL)),
 };
 
-#ifdef CONFIG_CMD_NAND
+#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
 /* NAND */
 static iomux_v3_cfg_t const nfc_pads[] = {
 	IOMUX_PADS(PAD_NANDF_CLE__NAND_CLE	| MUX_PAD_CTRL(NAND_PAD_CTRL)),
@@ -274,7 +274,7 @@  static void setup_gpios(void)
 	SETUP_IOMUX_PADS(gpios_pads);
 }
 
-#ifdef CONFIG_CMD_NAND
+#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
 static void setup_gpmi_nand(void)
 {
 	struct mxc_ccm_reg *mxc_ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
@@ -361,7 +361,7 @@  int board_init(void)
 
 	setup_gpios();
 
-#ifdef CONFIG_CMD_NAND
+#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
 	setup_gpmi_nand();
 #endif
 	return 0;
@@ -657,7 +657,7 @@  void board_init_f(ulong dummy)
 		.refr = 7,	/* 8 refresh commands per refresh cycle */
 	};
 
-#ifdef CONFIG_CMD_NAND
+#if defined(CONFIG_CMD_NAND) && !defined(CONFIG_SPL_BUILD)
 	/* Enable NAND */
 	setup_gpmi_nand();
 #endif
diff --git a/configs/pfla02_defconfig b/configs/pfla02_defconfig
index 5e5bbb05bd5e..52cebd9e22ed 100644
--- a/configs/pfla02_defconfig
+++ b/configs/pfla02_defconfig
@@ -37,7 +37,7 @@  CONFIG_SPL_YMODEM_SUPPORT=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MMC=y
-# CONFIG_CMD_NAND is not set
+CONFIG_CMD_NAND_TRIMFFS=y
 CONFIG_CMD_SF=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT4_WRITE=y
diff --git a/include/configs/pfla02.h b/include/configs/pfla02.h
index d4a7de7df6a2..4162a71ca661 100644
--- a/include/configs/pfla02.h
+++ b/include/configs/pfla02.h
@@ -41,15 +41,11 @@ 
 #define CONFIG_SYS_I2C_MXC_I2C1		/* enable I2C bus 0 */
 #define CONFIG_SYS_I2C_SPEED		  100000
 
-#ifndef CONFIG_SPL_BUILD
-#define CONFIG_CMD_NAND
 /* Enable NAND support */
-#define CONFIG_CMD_NAND_TRIMFFS
 #define CONFIG_SYS_MAX_NAND_DEVICE	1
 #define CONFIG_SYS_NAND_BASE		0x40000000
 #define CONFIG_SYS_NAND_5_ADDR_CYCLE
 #define CONFIG_SYS_NAND_ONFI_DETECTION
-#endif
 
 /* DMA stuff, needed for GPMI/MXS NAND support */