All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [Patch v3] armv8: fsl-layerscape: Set env_loc to ENVL_NOWHERE with CONFIG_ENV_IS_NOWHERE.
@ 2019-04-23  6:06 Udit Agarwal
  2019-05-22 12:33 ` Prabhakar Kushwaha
  0 siblings, 1 reply; 2+ messages in thread
From: Udit Agarwal @ 2019-04-23  6:06 UTC (permalink / raw)
  To: u-boot

ENVL_NOWHERE is dependent on CONFIG_ENV_IS_NOWHERE and not on
CONFIG_CHAIN_OF_TRUST so return ENVL_NOWHERE when CONFIG_ENV_IS_NOWHERE
is enabled

Signed-off-by: Udit Agarwal <udit.agarwal@nxp.com>
---
Changes in V3:
Correct checkpatch warnings.

 arch/arm/cpu/armv8/fsl-layerscape/cpu.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
index 54ea6b3de0..d26305ce2f 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
@@ -799,12 +799,8 @@ enum env_location env_get_location(enum env_operation op, int prio)
 	if (prio)
 		return ENVL_UNKNOWN;
 
-#ifdef CONFIG_CHAIN_OF_TRUST
-	/* Check Boot Mode
-	 * If Boot Mode is Secure, return ENVL_NOWHERE
-	 */
-	if (fsl_check_boot_mode_secure() == 1)
-		goto done;
+#ifdef	CONFIG_ENV_IS_NOWHERE
+	return env_loc;
 #endif
 
 	switch (src) {
@@ -834,9 +830,6 @@ enum env_location env_get_location(enum env_operation op, int prio)
 		break;
 	}
 
-#ifdef CONFIG_CHAIN_OF_TRUST
-done:
-#endif
 	return env_loc;
 }
 #endif	/* CONFIG_TFABOOT */
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [U-Boot] [Patch v3] armv8: fsl-layerscape: Set env_loc to ENVL_NOWHERE with CONFIG_ENV_IS_NOWHERE.
  2019-04-23  6:06 [U-Boot] [Patch v3] armv8: fsl-layerscape: Set env_loc to ENVL_NOWHERE with CONFIG_ENV_IS_NOWHERE Udit Agarwal
@ 2019-05-22 12:33 ` Prabhakar Kushwaha
  0 siblings, 0 replies; 2+ messages in thread
From: Prabhakar Kushwaha @ 2019-05-22 12:33 UTC (permalink / raw)
  To: u-boot


> -----Original Message-----
> From: Udit Agarwal
> Sent: Tuesday, April 23, 2019 11:36 AM
> To: u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; Ruchika Gupta
> <ruchika.gupta@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>; Udit
> Agarwal <udit.agarwal@nxp.com>
> Subject: [Patch v3] armv8: fsl-layerscape: Set env_loc to ENVL_NOWHERE with
> CONFIG_ENV_IS_NOWHERE.
> 
> ENVL_NOWHERE is dependent on CONFIG_ENV_IS_NOWHERE and not on
> CONFIG_CHAIN_OF_TRUST so return ENVL_NOWHERE when
> CONFIG_ENV_IS_NOWHERE is enabled
> 
> Signed-off-by: Udit Agarwal <udit.agarwal@nxp.com>
> ---
> Changes in V3:
> Correct checkpatch warnings.
> 

This patch has been applied to fsl-qoriq master, awaiting upstream.

--pk

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-05-22 12:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-23  6:06 [U-Boot] [Patch v3] armv8: fsl-layerscape: Set env_loc to ENVL_NOWHERE with CONFIG_ENV_IS_NOWHERE Udit Agarwal
2019-05-22 12:33 ` Prabhakar Kushwaha

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.