public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Andrew F. Davis <afd@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/6] configs: ti: Factor out call to 'args_mmc' into MMC common environment
Date: Wed, 20 Nov 2019 12:40:28 -0500	[thread overview]
Message-ID: <20191120174032.12134-3-afd@ti.com> (raw)
In-Reply-To: <20191120174032.12134-1-afd@ti.com>

Both 'loadfit' and 'mmcloados' start with a call to 'args_mmc' so this
can be factored out to before eithers only call site. This also allows us
to remove that call from 'loadfit', which should not have been calling it
anyway as that command should not be MMC specific. Without the call to
'args_mmc' the command 'loadfit' becomes just a call to 'run_fit' so
remove the indirection and call 'run_fit' directly, this removes the need
for 'loadfit' command (which was misnamed anyway). Drop it.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 include/configs/ti_armv7_common.h | 1 -
 include/environment/ti/mmc.h      | 5 +++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
index 6d15304a65..2ec1dea26e 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -61,7 +61,6 @@
 		"setenv overlaystring ${overlaystring}'#'${overlay};" \
 		"done;\0" \
 	"run_fit=bootm ${addr_fit}#${fdtfile}${overlaystring}\0" \
-	"loadfit=run args_mmc; run run_fit;\0" \
 
 /*
  * DDR information.  If the CONFIG_NR_DRAM_BANKS is not defined,
diff --git a/include/environment/ti/mmc.h b/include/environment/ti/mmc.h
index bb4af0a3d5..1c8e49a8b3 100644
--- a/include/environment/ti/mmc.h
+++ b/include/environment/ti/mmc.h
@@ -41,7 +41,7 @@
 				"fi;" \
 			"fi;" \
 		"fi;\0" \
-	"mmcloados=run args_mmc; " \
+	"mmcloados=" \
 		"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
 			"if run loadfdt; then " \
 				"bootz ${loadaddr} - ${fdtaddr}; " \
@@ -61,8 +61,9 @@
 		"if mmc rescan; then " \
 			"echo SD/MMC found on device ${mmcdev};" \
 			"if run loadimage; then " \
+				"run args_mmc; " \
 				"if test ${boot_fit} -eq 1; then " \
-					"run loadfit; " \
+					"run run_fit; " \
 				"else " \
 					"run mmcloados;" \
 				"fi;" \
-- 
2.17.1

  parent reply	other threads:[~2019-11-20 17:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-20 17:40 [U-Boot] [PATCH 0/6] J721e HS device support Andrew F. Davis
2019-11-20 17:40 ` [U-Boot] [PATCH 1/6] configs: j721e_evm.h: Sync J721e environment configuration with AM65x Andrew F. Davis
2019-11-20 17:40 ` Andrew F. Davis [this message]
2019-11-20 17:40 ` [U-Boot] [PATCH 3/6] arm: K3: Fix header comment match AM6 specific file function Andrew F. Davis
2019-11-20 17:40 ` [U-Boot] [PATCH 4/6] arm: K3: Disable ROM configured firewalls Andrew F. Davis
2019-11-20 17:40 ` [U-Boot] [PATCH 5/6] arm: K3: Increase default SYSFW image size allocation Andrew F. Davis
2019-11-20 17:40 ` [U-Boot] [PATCH 6/6] configs: Add configs for J721e High Security EVM Andrew F. Davis
2019-11-21  3:59 ` [U-Boot] [PATCH 0/6] J721e HS device support Lokesh Vutla
2019-12-16 20:50   ` Andrew F. Davis
2020-01-10 18:38     ` Andrew F. Davis
2020-01-10 19:28       ` Andrew F. Davis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191120174032.12134-3-afd@ti.com \
    --to=afd@ti.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox