On Tue, 22 Aug 2017, Kever Yang wrote:
U-Boot fastboot, kernel may reboot with parameter which require bootloader to get into different boot mode, detect it and enter proper mode.
Signed-off-by: Kever Yang kever.yang@rock-chips.com
Reviewed-by: Philipp Tomsich philipp.tomsich@theobroma-systems.com
Requested changes below.
Changes in v2:
- use env_set() instead of setenv()
arch/arm/mach-rockchip/rk3399/Kconfig | 1 + arch/arm/mach-rockchip/rk3399/rk3399.c | 42 +++++++++++++++++++++++++++++++++- 2 files changed, 42 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-rockchip/rk3399/Kconfig b/arch/arm/mach-rockchip/rk3399/Kconfig index 415466a..c4a6d46 100644 --- a/arch/arm/mach-rockchip/rk3399/Kconfig +++ b/arch/arm/mach-rockchip/rk3399/Kconfig @@ -5,6 +5,7 @@ choice
config TARGET_EVB_RK3399 bool "RK3399 evaluation board"
- select BOARD_LATE_INIT
With the changes below, we can make this a default for all RK3399 boards. So let's imply BOARD_LATE_INIT from the RK3399 selection.
help RK3399evb is a evaluation board for Rockchp rk3399, with full function and phisical connectors support like type-C ports, diff --git a/arch/arm/mach-rockchip/rk3399/rk3399.c b/arch/arm/mach-rockchip/rk3399/rk3399.c index dbc248f..3a5e4f9 100644 --- a/arch/arm/mach-rockchip/rk3399/rk3399.c +++ b/arch/arm/mach-rockchip/rk3399/rk3399.c @@ -5,9 +5,13 @@ */
#include <common.h> +#include <syscon.h> #include <asm/armv8/mmu.h> -#include <asm/io.h> +#include <asm/arch/boot_mode.h> +#include <asm/arch/clock.h> +#include <asm/arch/grf_rk3399.h> #include <asm/arch/hardware.h> +#include <asm/io.h>
DECLARE_GLOBAL_DATA_PTR;
@@ -55,3 +59,39 @@ int arch_cpu_init(void)
return 0; }
+static void setup_boot_mode(void) +{
- struct rk3399_pmugrf_regs *pmugrf;
- int boot_mode;
- pmugrf = syscon_get_first_range(ROCKCHIP_SYSCON_PMUGRF);
- boot_mode = readl(&pmugrf->os_reg0);
- debug("boot mode %x\n", boot_mode);
- /* Clear boot mode */
- writel(BOOT_NORMAL, &pmugrf->os_reg0);
- switch (boot_mode) {
- case BOOT_FASTBOOT:
printf("enter fastboot!\n");
I would use a debug() here...
env_set("preboot", "setenv preboot; fastboot usb0");
Could we just set a more neutral (than 'preboot') environment variable that can then be run by bootscripts? This would allow custom boot scripts to either run this or not (while 'preboot' is unconditional).
Alternatively, you could just set an environment variable indicating that the 'fastboot'-flag was set and then a bootscript could test this and forward into fastboot mode.
break;- case BOOT_UMS:
printf("enter UMS!\n");env_set("preboot", "setenv preboot; if mmc dev 0;""then ums mmc 0; else ums mmc 1;fi");
Same.
break;- case BOOT_LOADER:
printf("enter Rockusb!\n");env_set("preboot", "setenv preboot; rockusb 0 mmc 0");
Same.
break;- }
+}
+int board_late_init(void) +{
- setup_boot_mode();
- return 0;
+}