Skip to contentSkip to conversation

mx7dsabresd: Remove the mx7dsabresd_secure_defconfig target

PATCH New Fabio Estevam 3 participants 3 messages Apr 19, 2018 – Apr 27, 2018

lore
Share & tools
lore

Fabio Estevam

author
#0 lore raw

PATCH 1/1 mx7dsabresd: Remove the mx7dsabresd_secure_defconfig target

From: Fabio Estevam <[email protected]>

mx7dsabresd_secure_defconfig was introduced to allow booting NXP kernel that has CAAM support and needs to boot in secure mode.

Instead of keeping two different config targets for the same board, remove mx7dsabresd_secure_defconfig and select CONFIG_ARMV7_BOOT_SEC_DEFAULT inside mx7dsabresd_defconfig so that this target could be used to boot both mainline and the vendor kernel.

This makes maintenance task easier and avoid potentially confusion for the end user.

Signed-off-by Fabio Estevam <[email protected]>

---
Hi,

As discussed on a patch series from Bryan we can get rid of the secure_defconfig variant and make things simpler.

 board/freescale/mx7dsabresd/MAINTAINERS |  1 -
 configs/mx7dsabresd_defconfig           |  1 +
 configs/mx7dsabresd_secure_defconfig    | 78 ---------------------------------
 3 files changed, 1 insertion(+), 79 deletions(-)
 delete mode 100644 configs/mx7dsabresd_secure_defconfig
board/freescale/mx7dsabresd/MAINTAINERS +0 -1
index c7a22fc..b96642a 100644
@@ -4,4 +4,3 @@ S:	Maintained
 F:	board/freescale/mx7dsabresd
 F:	include/configs/mx7dsabresd.h
 F:	configs/mx7dsabresd_defconfig
-F:	configs/mx7dsabresd_secure_defconfig
configs/mx7dsabresd_defconfig +1 -0
index da881d9..33851c1 100644
@@ -2,6 +2,7 @@ CONFIG_ARM=y
 CONFIG_ARCH_MX7=y
 CONFIG_SYS_TEXT_BASE=0x87800000
 CONFIG_TARGET_MX7DSABRESD=y
+CONFIG_ARMV7_BOOT_SEC_DEFAULT=y
 # CONFIG_ARMV7_VIRT is not set
 CONFIG_IMX_RDC=y
 CONFIG_IMX_BOOTAUX=y
configs/mx7dsabresd_secure_defconfig deleted +0 -78

78 lines not on this page · raw message

Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/1 mx7dsabresd: Remove the mx7dsabresd_secure_defconfig target

Quoted 10 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

mx7dsabresd_secure_defconfig was introduced to allow booting NXP kernel that has CAAM support and needs to boot in secure mode.

Instead of keeping two different config targets for the same board, remove mx7dsabresd_secure_defconfig and select CONFIG_ARMV7_BOOT_SEC_DEFAULT inside mx7dsabresd_defconfig so that this target could be used to boot both mainline and the vendor kernel.

Quoted 4 lines

This makes maintenance task easier and avoid potentially confusion for the end user.

Signed-off-by Fabio Estevam <[email protected]>
Acked-by Bryan O'Donoghue <[email protected]>
#2 lore raw

PATCH 1/1 mx7dsabresd: Remove the mx7dsabresd_secure_defconfig target

In reply to Fabio Estevam
Quoted 10 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

mx7dsabresd_secure_defconfig was introduced to allow booting NXP kernel that has CAAM support and needs to boot in secure mode.

Instead of keeping two different config targets for the same board, remove mx7dsabresd_secure_defconfig and select CONFIG_ARMV7_BOOT_SEC_DEFAULT inside mx7dsabresd_defconfig so that this target could be used to boot both mainline and the vendor kernel.

Quoted 5 lines

This makes maintenance task easier and avoid potentially confusion for the end user.

Signed-off-by Fabio Estevam <[email protected]>

---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic

Signature · 5 lines
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================