diff mbox

[U-Boot,3/4] imx: mx7dsabresd: drop code for CONFIG_CMD_BMODE

Message ID 1442221736-16377-3-git-send-email-Peng.Fan@freescale.com
State Superseded
Delegated to: Stefano Babic
Headers show

Commit Message

Peng Fan Sept. 14, 2015, 9:08 a.m. UTC
Since i.MX7D does not support warm reset, so boot_mode_apply
actually will not work. Also we use outer pmic reset and drop
internal reset signal, so drop boot mode code for 7dsabresd board.

Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Adrian Alonso <aalonso@freescale.com>
---
 board/freescale/mx7dsabresd/mx7dsabresd.c | 14 --------------
 include/configs/mx7dsabresd.h             |  2 --
 2 files changed, 16 deletions(-)

Comments

Fabio Estevam Sept. 14, 2015, 6:09 p.m. UTC | #1
Hi Peng,

On Mon, Sep 14, 2015 at 6:08 AM, Peng Fan <Peng.Fan@freescale.com> wrote:
> Since i.MX7D does not support warm reset, so boot_mode_apply
> actually will not work. Also we use outer pmic reset and drop
> internal reset signal, so drop boot mode code for 7dsabresd board.

Booting via bmode is working fine here.

Regards,

Fabio Estevam
Peng Fan Sept. 15, 2015, 12:31 a.m. UTC | #2
Hi Fabio,
On Mon, Sep 14, 2015 at 03:09:24PM -0300, Fabio Estevam wrote:
>Hi Peng,
>
>On Mon, Sep 14, 2015 at 6:08 AM, Peng Fan <Peng.Fan@freescale.com> wrote:
>> Since i.MX7D does not support warm reset, so boot_mode_apply
>> actually will not work. Also we use outer pmic reset and drop
>> internal reset signal, so drop boot mode code for 7dsabresd board.
>
>Booting via bmode is working fine here.

If boot from sd1, bmode sd1 work well. If boot from qspi1, bmode qspi work well.
This is expected. If boot from qspi, but we use 'bmode sd1', this will not work.
The reason that introduce bmode command is to boot from different media, but
now we use wdog signal to trigger pmic reset, which means the SoC will be
powered off and powered on. There is no reason to keep bmode. This patch
needs my previous wdog patch set.

Regards,
Peng.
>
>Regards,
>
>Fabio Estevam
Fabio Estevam Sept. 15, 2015, 1:48 a.m. UTC | #3
Hi Peng,

On Mon, Sep 14, 2015 at 9:31 PM, Peng Fan <b51431@freescale.com> wrote:
> Hi Fabio,
> On Mon, Sep 14, 2015 at 03:09:24PM -0300, Fabio Estevam wrote:
>>Hi Peng,
>>
>>On Mon, Sep 14, 2015 at 6:08 AM, Peng Fan <Peng.Fan@freescale.com> wrote:
>>> Since i.MX7D does not support warm reset, so boot_mode_apply
>>> actually will not work. Also we use outer pmic reset and drop
>>> internal reset signal, so drop boot mode code for 7dsabresd board.
>>
>>Booting via bmode is working fine here.
>
> If boot from sd1, bmode sd1 work well. If boot from qspi1, bmode qspi work well.
> This is expected. If boot from qspi, but we use 'bmode sd1', this will not work.

Understood. I only booted from sd1 and did 'bmode sd1'.

Regards,

Fabio Estevam
diff mbox

Patch

diff --git a/board/freescale/mx7dsabresd/mx7dsabresd.c b/board/freescale/mx7dsabresd/mx7dsabresd.c
index 6d88573..2b14fb6 100644
--- a/board/freescale/mx7dsabresd/mx7dsabresd.c
+++ b/board/freescale/mx7dsabresd/mx7dsabresd.c
@@ -10,7 +10,6 @@ 
 #include <asm/arch/sys_proto.h>
 #include <asm/gpio.h>
 #include <asm/imx-common/iomux-v3.h>
-#include <asm/imx-common/boot_mode.h>
 #include <asm/io.h>
 #include <linux/sizes.h>
 #include <common.h>
@@ -458,15 +457,6 @@  int board_init(void)
 	return 0;
 }
 
-#ifdef CONFIG_CMD_BMODE
-static const struct boot_mode board_boot_modes[] = {
-	/* 4 bit bus width */
-	{"sd1", MAKE_CFGVAL(0x10, 0x10, 0x00, 0x00)},
-	{"emmc", MAKE_CFGVAL(0x10, 0x2a, 0x00, 0x00)},
-	{NULL,   0},
-};
-#endif
-
 #ifdef CONFIG_POWER
 #define I2C_PMIC	0
 int power_init_board(void)
@@ -501,10 +491,6 @@  int board_late_init(void)
 {
 	struct wdog_regs *wdog = (struct wdog_regs *)WDOG1_BASE_ADDR;
 
-#ifdef CONFIG_CMD_BMODE
-	add_board_boot_modes(board_boot_modes);
-#endif
-
 #ifdef CONFIG_ENV_IS_IN_MMC
 	mmc_late_init();
 #endif
diff --git a/include/configs/mx7dsabresd.h b/include/configs/mx7dsabresd.h
index ec6e0a6..8231c04 100644
--- a/include/configs/mx7dsabresd.h
+++ b/include/configs/mx7dsabresd.h
@@ -175,8 +175,6 @@ 
 #define CONFIG_SYS_MMC_ENV_PART		0	/* user area */
 #define CONFIG_MMCROOT			"/dev/mmcblk0p2"  /* USDHC1 */
 
-#define CONFIG_CMD_BMODE
-
 /* USB Configs */
 #define CONFIG_CMD_USB
 #define CONFIG_USB_EHCI