diff mbox series

[4/4] cmd: booti: convert the debug print about image move to printf

Message ID 20200612124121.13846-5-t-kristo@ti.com
State Accepted
Commit 9b83f9c594af6033dd507c3ecad1231421063949
Delegated to: Tom Rini
Headers show
Series Some reserved memory hardenings | expand

Commit Message

Tero Kristo June 12, 2020, 12:41 p.m. UTC
Moving of the OS image may have some nasty side effects like corrupting
DTB. Convert the current debug print to printf so that the relocation of
the OS is always obvious to the user.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
---
 cmd/booti.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Tom Rini July 17, 2020, 8:57 p.m. UTC | #1
On Fri, Jun 12, 2020 at 03:41:21PM +0300, Tero Kristo wrote:

> Moving of the OS image may have some nasty side effects like corrupting
> DTB. Convert the current debug print to printf so that the relocation of
> the OS is always obvious to the user.
> 
> Signed-off-by: Tero Kristo <t-kristo@ti.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/cmd/booti.c b/cmd/booti.c
index 6153b229cf..ad04ebd8c3 100644
--- a/cmd/booti.c
+++ b/cmd/booti.c
@@ -79,7 +79,8 @@  static int booti_start(struct cmd_tbl *cmdtp, int flag, int argc,
 
 	/* Handle BOOTM_STATE_LOADOS */
 	if (relocated_addr != ld) {
-		debug("Moving Image from 0x%lx to 0x%lx\n", ld, relocated_addr);
+		printf("Moving Image from 0x%lx to 0x%lx, end=%lx\n", ld,
+		       relocated_addr, relocated_addr + image_size);
 		memmove((void *)relocated_addr, (void *)ld, image_size);
 	}