arch/Kconfig.nxp: Re-organize slightly

Make all of the CHAIN_OF_TRUST options be under a single menu and add a
comment for the rest, so the resulting config file reads more clearly.
Remove duplicate CHAIN_OF_TRUST options from
board/congatec/common/Kconfig.  Remove duplicate NXP_ESBC config
questions and move to arch/Kconfig.nxp.

Signed-off-by: Tom Rini <trini@konsulko.com>
diff --git a/arch/Kconfig.nxp b/arch/Kconfig.nxp
index 22ae186..273e8e8 100644
--- a/arch/Kconfig.nxp
+++ b/arch/Kconfig.nxp
@@ -1,7 +1,13 @@
-config CHAIN_OF_TRUST
+config NXP_ESBC
+	bool "NXP ESBC (secure boot) functionality"
+	help
+	  Enable Freescale Secure Boot feature. Normally selected by defconfig.
+	  If unsure, do not change.
+
+menu "Chain of trust / secure boot options"
 	depends on !FIT_SIGNATURE && NXP_ESBC
-	imply CMD_BLOB
-	imply CMD_HASH if ARM
+
+config CHAIN_OF_TRUST
 	select FSL_CAAM
 	select ARCH_MISC_INIT
 	select FSL_SEC_MON
@@ -12,12 +18,12 @@
 	select ENV_IS_NOWHERE
 	select CMD_EXT4 if ARM
 	select CMD_EXT4_WRITE if ARM
-	bool
-	default y
+	imply CMD_BLOB
+	imply CMD_HASH if ARM
+	def_bool y
 
 config CMD_ESBC_VALIDATE
 	bool "Enable the 'esbc_validate' and 'esbc_halt' commands"
-	depends on CHAIN_OF_TRUST
 	default y
 	help
 	  This option enables two commands used for secure booting:
@@ -36,15 +42,14 @@
 
 config SYS_FSL_SFP_BE
 	def_bool y
-	depends on CHAIN_OF_TRUST && (PPC || FSL_LSCH2 || ARCH_LS1021A)
+	depends on PPC || FSL_LSCH2 || ARCH_LS1021A
 
 config SYS_FSL_SFP_LE
 	def_bool y
-	depends on CHAIN_OF_TRUST && !SYS_FSL_SFP_BE
+	depends on !SYS_FSL_SFP_BE
 
 choice
 	prompt "SFP IP revision"
-	depends on CHAIN_OF_TRUST
 	default SYS_FSL_SFP_VER_3_0 if PPC
 	default SYS_FSL_SFP_VER_3_4
 
@@ -61,11 +66,14 @@
 
 config SYS_FSL_SRK_LE
 	def_bool y
-	depends on CHAIN_OF_TRUST && ARM
+	depends on ARM
 
 config KEY_REVOCATION
 	def_bool y
-	depends on CHAIN_OF_TRUST
+
+endmenu
+
+comment "Other functionality shared between NXP SoCs"
 
 config DEEP_SLEEP
 	bool "Enable SoC deep sleep feature"