public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jagan Teki <jagan@amarulasolutions.com>
To: u-boot@lists.denx.de
Subject: [PATCH v4 2/5] rk3399: spl: Print SPL banner after relocation
Date: Thu, 18 Jun 2020 21:09:45 +0530	[thread overview]
Message-ID: <20200618153948.218506-3-jagan@amarulasolutions.com> (raw)
In-Reply-To: <20200618153948.218506-1-jagan@amarulasolutions.com>

Usually printing the SPL banner varies between architecture
or board codes.
- Some would print before relocation at the end board_init_f
? for making sure all initialization prior to this would happen
? properly. if at all there is a requirement for serial init,
? that happens properly since it prints all after that.
- Some would print after relocation at the spl_board_init for
? making sure all initialization prior to relocation would
  happen properly. Also debug uart on these cases would be
  available before relocation. So debug support is available
  in before and after relocation.

Rockchip SPL is following formar step to print the banner at
the end of board_init_f.

To support various custom use cases in SPL like leds, environment,
board detections later options like printing the banner after
relocation would be a better option. Printing banner would also
help to support debugging availability between relocation codes,
like debug uart available before relocation and banner availability
after relation.

By demonstrating all the above use cases, this patch is trying
to print the SPL banner after relocation.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Tested-by: Suniel Mahesh <sunil@amarulasolutions.com>
---
Changes for v4:
- none

 arch/arm/mach-rockchip/spl.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-rockchip/spl.c b/arch/arm/mach-rockchip/spl.c
index d4c83a1119..6d5c058548 100644
--- a/arch/arm/mach-rockchip/spl.c
+++ b/arch/arm/mach-rockchip/spl.c
@@ -147,7 +147,6 @@ void board_init_f(ulong dummy)
 	gd->ram_top = gd->ram_base + get_effective_memsize();
 	gd->ram_top = board_get_usable_ram_top(gd->ram_size);
 #endif
-	preloader_console_init();
 }
 
 __weak void rk_spl_board_init(void)
@@ -158,6 +157,8 @@ void spl_board_init(void)
 {
 	/* board specific spl init */
 	rk_spl_board_init();
+
+	preloader_console_init();
 }
 
 #ifdef CONFIG_SPL_LOAD_FIT
-- 
2.25.1

  parent reply	other threads:[~2020-06-18 15:39 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-18 15:39 [PATCH v4 0/5] roc-rk3399-pc: Custom SPL init Jagan Teki
2020-06-18 15:39 ` [PATCH v4 1/5] rockchip: spl: Add spl_board_init Jagan Teki
2020-06-28  3:06   ` Kever Yang
2020-06-18 15:39 ` Jagan Teki [this message]
2020-06-28  2:56   ` [PATCH v4 2/5] rk3399: spl: Print SPL banner after relocation Kever Yang
2020-07-10 10:19     ` Jagan Teki
2020-06-18 15:39 ` [PATCH v4 3/5] roc-rk3399-pc: Move leds setup in SPL Jagan Teki
2020-06-28  2:50   ` Kever Yang
2020-06-18 15:39 ` [PATCH v4 4/5] rockchip: Separate the reset cause from display cpuinfo Jagan Teki
2020-06-28  2:46   ` Kever Yang
2020-07-13 19:57     ` Jagan Teki
2020-06-18 15:39 ` [PATCH v4 5/5] roc-rk3399-pc: Set LED only during POR and pwr_key=y Jagan Teki

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=20200618153948.218506-3-jagan@amarulasolutions.com \
    --to=jagan@amarulasolutions.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