From: Nishanth Menon <nm@ti.com>
To: u-boot@lists.denx.de
Subject: [PATCH V2 4/4] configs: am65x_evm: Use DEFAULT_LINUX_BOOT_ENV and remove overlayaddr
Date: Wed, 7 Oct 2020 21:48:01 -0500 [thread overview]
Message-ID: <20201008024801.11583-5-nm@ti.com> (raw)
In-Reply-To: <20201008024801.11583-1-nm@ti.com>
Use DEFAULT_LINUX_BOOT_ENV to define the standard addresses used in rest
of TI platforms as defined in ti_armv7_common.h
This avoids the standard pitfalls we've had with kernel images and fdt
addresses stomping on each other.
As part of this process, redefine overlayaddr to be dtboaddr (defined
in ti_armv7_common.h for this very purpose) and get rid of the
definition of overlayaddr..
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Changes since V1:
- Squashed patch of #1 and #4, no functional changes
V1:
https://patchwork.ozlabs.org/project/uboot/patch/20201006190507.13346-2-nm at ti.com/
https://patchwork.ozlabs.org/project/uboot/patch/20201006190507.13346-5-nm at ti.com/
include/configs/am65x_evm.h | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/include/configs/am65x_evm.h b/include/configs/am65x_evm.h
index df0605657aef..9eed0ea203a9 100644
--- a/include/configs/am65x_evm.h
+++ b/include/configs/am65x_evm.h
@@ -69,9 +69,6 @@
"findfdt=" \
"setenv name_fdt k3-am654-base-board.dtb;" \
"setenv fdtfile ${name_fdt}\0" \
- "loadaddr=0x80080000\0" \
- "fdtaddr=0x82000000\0" \
- "overlayaddr=0x83000000\0" \
"name_kern=Image\0" \
"console=ttyS2,115200n8\0" \
"stdin=serial,usbkbd\0" \
@@ -93,8 +90,8 @@
"fdt resize 0x100000;" \
"for overlay in $name_overlays;" \
"do;" \
- "load mmc ${bootpart} ${overlayaddr} ${bootdir}/${overlay};" \
- "fdt apply ${overlayaddr};" \
+ "load mmc ${bootpart} ${dtboaddr} ${bootdir}/${overlay};" \
+ "fdt apply ${dtboaddr};" \
"done;\0" \
"get_kern_mmc=load mmc ${bootpart} ${loadaddr} " \
"${bootdir}/${name_kern}\0" \
@@ -133,6 +130,7 @@
/* Incorporate settings into the U-Boot environment */
#define CONFIG_EXTRA_ENV_SETTINGS \
+ DEFAULT_LINUX_BOOT_ENV \
DEFAULT_MMC_TI_ARGS \
DEFAULT_FIT_TI_ARGS \
EXTRA_ENV_AM65X_BOARD_SETTINGS \
--
2.28.0
next prev parent reply other threads:[~2020-10-08 2:48 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-08 2:47 [PATCH V2 0/4] configs: ti: am65x/j721e: Use DEFAULT_LINUX_BOOT_ENV Nishanth Menon
2020-10-08 2:47 ` [PATCH V2 1/4] configs: j721e_evm: " Nishanth Menon
2020-10-08 2:47 ` [PATCH V2 2/4] env: ti: ufs: Use dtboaddr instead of overlayaddr Nishanth Menon
2020-10-08 2:48 ` [PATCH V2 3/4] configs: j721e_evm: Get rid " Nishanth Menon
2020-10-08 2:48 ` Nishanth Menon [this message]
2020-10-12 9:09 ` [PATCH V2 0/4] configs: ti: am65x/j721e: Use DEFAULT_LINUX_BOOT_ENV Lokesh Vutla
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=20201008024801.11583-5-nm@ti.com \
--to=nm@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