Message ID | 1476264373-2966-2-git-send-email-Zhiqiang.Hou@nxp.com |
---|---|
State | Deferred |
Delegated to: | York Sun |
Headers | show |
> -----Original Message----- > From: Zhiqiang Hou [mailto:Zhiqiang.Hou@nxp.com] > Sent: Wednesday, October 12, 2016 2:56 PM > To: u-boot@lists.denx.de; albert.u.boot@aribaud.net; york sun > <york.sun@nxp.com>; sjg@chromium.org; bmeng.cn@gmail.com; hs@denx.de; > yamada.masahiro@socionext.com; Ruchika Gupta <ruchika.gupta@nxp.com>; > eddy.petrisor@gmail.com; s.temerkhanov@gmail.com; Prabhakar Kushwaha > <prabhakar.kushwaha@nxp.com>; sr@denx.de; van.freenix@gmail.com; > fgretief@spaceteq.co.za; rpjday@crashcourse.ca; trini@konsulko.com; Mingkai > Hu <mingkai.hu@nxp.com> > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > Subject: [PATCH 2/5] configs/ls104xa: enable the config NOR_BOOT > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > > Enable the config NOR_BOOT obviously for NOR boot defconfigs of > ls1043a and ls1046a. > Earlier, no CONFIG for NOR boot was required. Why new variable CONFIG_NOR_BOOT is even required? --prabhakar
Hi Prabhakar, Thanks for your comments! > -----Original Message----- > From: Prabhakar Kushwaha > Sent: 2016年10月17日 11:40 > To: Z.Q. Hou <zhiqiang.hou@nxp.com>; u-boot@lists.denx.de; > albert.u.boot@aribaud.net; york sun <york.sun@nxp.com>; > sjg@chromium.org; bmeng.cn@gmail.com; hs@denx.de; > yamada.masahiro@socionext.com; Ruchika Gupta <ruchika.gupta@nxp.com>; > eddy.petrisor@gmail.com; s.temerkhanov@gmail.com; sr@denx.de; > van.freenix@gmail.com; fgretief@spaceteq.co.za; rpjday@crashcourse.ca; > trini@konsulko.com; Mingkai Hu <mingkai.hu@nxp.com> > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > Subject: RE: [PATCH 2/5] configs/ls104xa: enable the config NOR_BOOT > > > > -----Original Message----- > > From: Zhiqiang Hou [mailto:Zhiqiang.Hou@nxp.com] > > Sent: Wednesday, October 12, 2016 2:56 PM > > To: u-boot@lists.denx.de; albert.u.boot@aribaud.net; york sun > > <york.sun@nxp.com>; sjg@chromium.org; bmeng.cn@gmail.com; > hs@denx.de; > > yamada.masahiro@socionext.com; Ruchika Gupta > <ruchika.gupta@nxp.com>; > > eddy.petrisor@gmail.com; s.temerkhanov@gmail.com; Prabhakar Kushwaha > > <prabhakar.kushwaha@nxp.com>; sr@denx.de; van.freenix@gmail.com; > > fgretief@spaceteq.co.za; rpjday@crashcourse.ca; trini@konsulko.com; > > Mingkai Hu <mingkai.hu@nxp.com> > > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > > Subject: [PATCH 2/5] configs/ls104xa: enable the config NOR_BOOT > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > > > > Enable the config NOR_BOOT obviously for NOR boot defconfigs of > > ls1043a and ls1046a. > > > > Earlier, no CONFIG for NOR boot was required. > Why new variable CONFIG_NOR_BOOT is even required? Now that the Boot Media configs has been introduced, it's reasonable to uniform all boot methods under this framework. And therefore the other config entry is able to be configured according to different Boot Media. Thanks, Zhiqiang
On Mon, Oct 17, 2016 at 09:12:30AM +0000, Z.Q. Hou wrote: > Hi Prabhakar, > > Thanks for your comments! > > > -----Original Message----- > > From: Prabhakar Kushwaha > > Sent: 2016年10月17日 11:40 > > To: Z.Q. Hou <zhiqiang.hou@nxp.com>; u-boot@lists.denx.de; > > albert.u.boot@aribaud.net; york sun <york.sun@nxp.com>; > > sjg@chromium.org; bmeng.cn@gmail.com; hs@denx.de; > > yamada.masahiro@socionext.com; Ruchika Gupta <ruchika.gupta@nxp.com>; > > eddy.petrisor@gmail.com; s.temerkhanov@gmail.com; sr@denx.de; > > van.freenix@gmail.com; fgretief@spaceteq.co.za; rpjday@crashcourse.ca; > > trini@konsulko.com; Mingkai Hu <mingkai.hu@nxp.com> > > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > > Subject: RE: [PATCH 2/5] configs/ls104xa: enable the config NOR_BOOT > > > > > > > -----Original Message----- > > > From: Zhiqiang Hou [mailto:Zhiqiang.Hou@nxp.com] > > > Sent: Wednesday, October 12, 2016 2:56 PM > > > To: u-boot@lists.denx.de; albert.u.boot@aribaud.net; york sun > > > <york.sun@nxp.com>; sjg@chromium.org; bmeng.cn@gmail.com; > > hs@denx.de; > > > yamada.masahiro@socionext.com; Ruchika Gupta > > <ruchika.gupta@nxp.com>; > > > eddy.petrisor@gmail.com; s.temerkhanov@gmail.com; Prabhakar Kushwaha > > > <prabhakar.kushwaha@nxp.com>; sr@denx.de; van.freenix@gmail.com; > > > fgretief@spaceteq.co.za; rpjday@crashcourse.ca; trini@konsulko.com; > > > Mingkai Hu <mingkai.hu@nxp.com> > > > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > > > Subject: [PATCH 2/5] configs/ls104xa: enable the config NOR_BOOT > > > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > > > > > > Enable the config NOR_BOOT obviously for NOR boot defconfigs of > > > ls1043a and ls1046a. > > > > > > > Earlier, no CONFIG for NOR boot was required. > > Why new variable CONFIG_NOR_BOOT is even required? > > Now that the Boot Media configs has been introduced, it's reasonable to uniform all boot methods under this framework. > And therefore the other config entry is able to be configured according to different Boot Media. But we also don't want to make this needlessly complicated. The general intention here is that we have some boot methods that are incompatible with others. For example, if a board lacks NOR and we try and run a binary that expects NOR, we panic today. So for beaglebone (where there is a NOR cape that you can boot from) we need a special build that does know about NOR. At some level, once we have enough things in DM some of this special casing can go away I hope.
diff --git a/configs/ls1043aqds_defconfig b/configs/ls1043aqds_defconfig index 9352d82..3c8ae2e 100644 --- a/configs/ls1043aqds_defconfig +++ b/configs/ls1043aqds_defconfig @@ -5,6 +5,7 @@ CONFIG_FIT=y CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y CONFIG_SYS_FSL_DDR4=y +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y diff --git a/configs/ls1043aqds_lpuart_defconfig b/configs/ls1043aqds_lpuart_defconfig index d384985..8acf732 100644 --- a/configs/ls1043aqds_lpuart_defconfig +++ b/configs/ls1043aqds_lpuart_defconfig @@ -6,6 +6,7 @@ CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y CONFIG_SYS_EXTRA_OPTIONS="LPUART" CONFIG_SYS_FSL_DDR4=y +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y diff --git a/configs/ls1043aqds_nor_ddr3_defconfig b/configs/ls1043aqds_nor_ddr3_defconfig index 1103346..9be3b2c 100644 --- a/configs/ls1043aqds_nor_ddr3_defconfig +++ b/configs/ls1043aqds_nor_ddr3_defconfig @@ -4,6 +4,7 @@ CONFIG_DEFAULT_DEVICE_TREE="fsl-ls1043a-qds-duart" CONFIG_FIT=y CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y diff --git a/configs/ls1043ardb_SECURE_BOOT_defconfig b/configs/ls1043ardb_SECURE_BOOT_defconfig index 262769d..0848d66 100644 --- a/configs/ls1043ardb_SECURE_BOOT_defconfig +++ b/configs/ls1043ardb_SECURE_BOOT_defconfig @@ -6,6 +6,7 @@ CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y CONFIG_SYS_EXTRA_OPTIONS="SECURE_BOOT" CONFIG_SYS_FSL_DDR4=y +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_MMC=y diff --git a/configs/ls1043ardb_defconfig b/configs/ls1043ardb_defconfig index e1b75ac..3b6de1d 100644 --- a/configs/ls1043ardb_defconfig +++ b/configs/ls1043ardb_defconfig @@ -5,6 +5,7 @@ CONFIG_FIT=y CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y CONFIG_SYS_FSL_DDR4=y +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_MMC=y diff --git a/configs/ls1046aqds_defconfig b/configs/ls1046aqds_defconfig index 71c3b95..d97ddd9 100644 --- a/configs/ls1046aqds_defconfig +++ b/configs/ls1046aqds_defconfig @@ -6,6 +6,7 @@ CONFIG_FIT=y CONFIG_FIT_VERBOSE=y CONFIG_OF_BOARD_SETUP=y CONFIG_SYS_EXTRA_OPTIONS="SYS_FSL_DDR4" +CONFIG_NOR_BOOT=y CONFIG_BOOTDELAY=10 CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y