[U-Boot] [PATCH v4 0/3] WaRP7 unify secure and non-secure defconfigs
This tree here is pullable
http://git.linaro.org/landing-teams/working/mbl/u-boot.git/log/?h=linaro-mbl...
V4: - Adds Tested-by/Reviewed-by from Beno and Fabio as indicated for my patches
- Resends Pierre-Jean's patch Since my parallel patchset relies on this patch I'm sending it again with my own series to make sure it doesn't get lost. To Pierre-Jean's patch I've added - Reviewed-by as Fabio indicated - Signed-off-by from me too
V3: - Remove warp7_secure_defconfig from board/warp7/MAINTAINERS - Add RB-tag from Fabio as indicated
V2: Maintaining a secure and non-secure defconfig is a PITA. Having discussed an observed bug with the non-secure boot-path with Fabio and undertaken to ensure both upstream and NXP BSP kernels would work - this patchset does two things.
1. Switches on CONFIG_ARMV7_BOOT_SEC_DEFAULT for warp7_defconfig 2. Removes warp7_secure_defconfig in its entirety
I have an upcoming set of patches that uses the secure defconfig which I've already verified can simply be merged into mainline WaRP7 as-is so, in this context there appears to be no justification for keeping separate configuration files.
Fix the bug and zap the fat !
Bryan O'Donoghue (2): warp7: defconfig: Fix CAAM on boot with tip-of-tree warp7: secure_defconfig: Remove secure_defconfig
Pierre-Jean TEXIER (1): warp7: configs: enable CONFIG_CMD_FS_GENERIC
board/warp7/MAINTAINERS | 1 - configs/warp7_defconfig | 2 ++ 2 files changed, 2 insertions(+), 1 deletion(-)
Booting the following image with tip-of-tree we get a CAAM DECO error (and subsequent crash due to a kernel bug in 4.1).
http://freescale.github.io/#download -> BoardsWaRPboard community - WaRP - Wearable Reference PlatformFSL Community BSP 2.3fsl-image-multimediawayland
Image: fsl-image-multimedia-imx7s-warp-20180323-90.rootfs.sdcard
Error: caam 30900000.caam: Entropy delay = 3200 caam 30900000.caam: failed to acquire DECO 0 <snip> caam 30900000.caam: failed to acquire DECO 0 caam 30900000.caam: Entropy delay = 12400 caam 30900000.caam: failed to acquire DECO 0 caam 30900000.caam: failed to instantiate RNG ------------[ cut here ]------------ WARNING: CPU: 0 PID: 1 at /home/jenkins/workspace/fsl-community-bsp-pyro_xwayland_2/build/tmp/work-shared/imx7s-warp/kernel-source/mm/vmalloc.c:1465 caam_remove+0x6) Trying to vfree() nonexistent vm area (88047000) Modules linked in: CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.1.36-4.1-1.0.x-imx-warp7+ga543d1b #1 Hardware name: Freescale i.MX7 Dual (Device Tree) [<80015d54>] (unwind_backtrace) from [<80012688>] (show_stack+0x10/0x14) [<80012688>] (show_stack) from [<8076e810>] (dump_stack+0x78/0x8c) [<8076e810>] (dump_stack) from [<800346a0>] (warn_slowpath_common+0x80/0xb0) [<800346a0>] (warn_slowpath_common) from [<80034700>] (warn_slowpath_fmt+0x30/0x40) [<80034700>] (warn_slowpath_fmt) from [<8054c278>] (caam_remove+0x6c/0x3f4) [<8054c278>] (caam_remove) from [<8054ce74>] (caam_probe+0x874/0xfa8) [<8054ce74>] (caam_probe) from [<80382a7c>] (platform_drv_probe+0x48/0xa4) [<80382a7c>] (platform_drv_probe) from [<80381328>] (driver_probe_device+0x174/0x2a8) [<80381328>] (driver_probe_device) from [<8038152c>] (__driver_attach+0x8c/0x90) [<8038152c>] (__driver_attach) from [<8037f9d4>] (bus_for_each_dev+0x68/0x9c) [<8037f9d4>] (bus_for_each_dev) from [<80380a68>] (bus_add_driver+0xf4/0x1e8) [<80380a68>] (bus_add_driver) from [<80381b38>] (driver_register+0x78/0xf4) [<80381b38>] (driver_register) from [<80009738>] (do_one_initcall+0x8c/0x1d0) [<80009738>] (do_one_initcall) from [<80a66dac>] (kernel_init_freeable+0x140/0x1d0) [<80a66dac>] (kernel_init_freeable) from [<8076aa38>] (kernel_init+0x8/0xe8) [<8076aa38>] (kernel_init) from [<8000f468>] (ret_from_fork+0x14/0x2c) ---[ end trace d5f941204ed8cb28 ]--- caam: probe of 30900000.caam failed with error -11 Unable to handle kernel NULL pointer dereference at virtual address 00000004 pgd = 80004000 [00000004] *pgd=00000000 Internal error: Oops: 805 [#1] PREEMPT SMP ARM <snip> [<8055cdf8>] (caam_sm_startup) from [<80aa83c8>] (caam_sm_init+0x50/0x58) [<80aa83c8>] (caam_sm_init) from [<80009738>] (do_one_initcall+0x8c/0x1d0) [<80009738>] (do_one_initcall) from [<80a66dac>] (kernel_init_freeable+0x140/0x1d0) [<80a66dac>] (kernel_init_freeable) from [<8076aa38>] (kernel_init+0x8/0xe8) [<8076aa38>] (kernel_init) from [<8000f468>] (ret_from_fork+0x14/0x2c) Code: e59d300c e2832010 e5843008 e5834068 (e58a2004) ---[ end trace d5f941204ed8cb29 ]--- Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b
Fix: Enable the CAAM correctly by setting CONFIG_ARMV7_BOOT_SEC_DEFAULT=y in the upstream defconfig.
Signed-off-by: Bryan O'Donoghue bryan.odonoghue@linaro.org Cc: Fabio Estevam fabio.estevam@nxp.com Cc: Breno Lima breno.lima@nxp.com Reviewed-by: Fabio Estevam fabio.estevam@nxp.com Tested-by: Breno Lima breno.lima@nxp.com --- configs/warp7_defconfig | 1 + 1 file changed, 1 insertion(+)
diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig index 3901a2741b..540dafd120 100644 --- a/configs/warp7_defconfig +++ b/configs/warp7_defconfig @@ -2,6 +2,7 @@ CONFIG_ARM=y CONFIG_ARCH_MX7=y CONFIG_SYS_TEXT_BASE=0x87800000 CONFIG_TARGET_WARP7=y +CONFIG_ARMV7_BOOT_SEC_DEFAULT=y # CONFIG_ARMV7_VIRT is not set CONFIG_IMX_RDC=y CONFIG_IMX_BOOTAUX=y
This patch removes warp7_secure_defconfig. A previous patch set CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked if I could confirm that the NXP and upstream kernels will boot on the WaRP7 with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case, so there's no need to support the secure defconfig - drop it now.
Signed-off-by: Bryan O'Donoghue bryan.odonoghue@linaro.org Suggested-by: Fabio Estevam fabio.estevam@nxp.com Reviewed-by: Fabio Estevam fabio.estevam@nxp.com --- board/warp7/MAINTAINERS | 1 - 1 file changed, 1 deletion(-)
diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS index 0fc9746606..1d3ee29222 100644 --- a/board/warp7/MAINTAINERS +++ b/board/warp7/MAINTAINERS @@ -4,4 +4,3 @@ S: Maintained F: board/warp7/ F: include/configs/warp7.h F: configs/warp7_defconfig -F: configs/warp7_secure_defconfig
Hi Bryan,
On Tue, Apr 24, 2018 at 2:34 PM, Bryan O'Donoghue bryan.odonoghue@linaro.org wrote:
This patch removes warp7_secure_defconfig. A previous patch set CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked if I could confirm that the NXP and upstream kernels will boot on the WaRP7 with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case, so there's no need to support the secure defconfig - drop it now.
Signed-off-by: Bryan O'Donoghue bryan.odonoghue@linaro.org Suggested-by: Fabio Estevam fabio.estevam@nxp.com Reviewed-by: Fabio Estevam fabio.estevam@nxp.com
board/warp7/MAINTAINERS | 1 - 1 file changed, 1 deletion(-)
diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS index 0fc9746606..1d3ee29222 100644 --- a/board/warp7/MAINTAINERS +++ b/board/warp7/MAINTAINERS @@ -4,4 +4,3 @@ S: Maintained F: board/warp7/ F: include/configs/warp7.h F: configs/warp7_defconfig -F: configs/warp7_secure_defconfig
In v4 you missed to actually remove configs/warp7_secure_defconfig :-)
On 24/04/18 18:37, Fabio Estevam wrote:
Hi Bryan,
On Tue, Apr 24, 2018 at 2:34 PM, Bryan O'Donoghue bryan.odonoghue@linaro.org wrote:
This patch removes warp7_secure_defconfig. A previous patch set CONFIG_ARMV7_BOOT_SEC_DEFAULT=y on the unsecure WaRP7 config. Fabio asked if I could confirm that the NXP and upstream kernels will boot on the WaRP7 with CONFIG_ARMV7_BOOT_SEC_DEFAULT=y. I can confirm that this is the case, so there's no need to support the secure defconfig - drop it now.
Signed-off-by: Bryan O'Donoghue bryan.odonoghue@linaro.org Suggested-by: Fabio Estevam fabio.estevam@nxp.com Reviewed-by: Fabio Estevam fabio.estevam@nxp.com
board/warp7/MAINTAINERS | 1 - 1 file changed, 1 deletion(-)
diff --git a/board/warp7/MAINTAINERS b/board/warp7/MAINTAINERS index 0fc9746606..1d3ee29222 100644 --- a/board/warp7/MAINTAINERS +++ b/board/warp7/MAINTAINERS @@ -4,4 +4,3 @@ S: Maintained F: board/warp7/ F: include/configs/warp7.h F: configs/warp7_defconfig -F: configs/warp7_secure_defconfig
In v4 you missed to actually remove configs/warp7_secure_defconfig :-)
ah damnit i had a merge conflict ... I remember now
From: Pierre-Jean TEXIER texier.pj2@gmail.com
This enable generic file system commands (load, ls).
Signed-off-by: Pierre-Jean TEXIER texier.pj2@gmail.com Acked-by: Bryan O'Donoghue bryan.odonoghue@linaro.org Reviewed-by: Fabio Estevam fabio.estevam@nxp.com Signed-off-by: Bryan O'Donoghue bryan.odonoghue@linaro.org --- configs/warp7_defconfig | 1 + 1 file changed, 1 insertion(+)
diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig index 540dafd120..73f3f09dc1 100644 --- a/configs/warp7_defconfig +++ b/configs/warp7_defconfig @@ -27,6 +27,7 @@ CONFIG_CMD_EXT2=y CONFIG_CMD_EXT4=y CONFIG_CMD_EXT4_WRITE=y CONFIG_CMD_FAT=y +CONFIG_CMD_FS_GENERIC=y CONFIG_NET_RANDOM_ETHADDR=y CONFIG_DFU_MMC=y CONFIG_FSL_ESDHC=y
participants (2)
-
Bryan O'Donoghue -
Fabio Estevam