diff mbox series

[U-Boot,v2] arm: mvebu: theadorable: Add test for ctrl-c in PCIe PEX switch test

Message ID 20190311125614.18804-1-sr@denx.de
State Accepted
Commit 54055ff7a6e66e7206db6f5e350f3880983afdd8
Delegated to: Stefan Roese
Headers show
Series [U-Boot,v2] arm: mvebu: theadorable: Add test for ctrl-c in PCIe PEX switch test | expand

Commit Message

Stefan Roese March 11, 2019, 12:56 p.m. UTC
The check for the missing PEX switch can lead to an infinite loop, when
the PCIe device is not found. It is helpful to enable the user to break
out of this boot-loop, to enable booting to the prompt for test cases.
This patch adds a 3 second delay in the error case before rebooting.
The user can press Ctrl-C in this time to abort the boot-loop.

This patch also calls bootcount_inc() before rebooting in the error
case. This is needed to increment the bootcounter, since this function
is called earlier than the main bootcounter increment. Otherwise the
bootcounter will not be incremented in the error case at all.

Signed-off-by: Stefan Roese <sr@denx.de>
---
v2:
- Rebase on current master

 board/theadorable/theadorable.c | 32 ++++++++++++++++++++++++++++++--
 1 file changed, 30 insertions(+), 2 deletions(-)

Comments

Stefan Roese April 11, 2019, 12:01 p.m. UTC | #1
On 11.03.19 13:56, Stefan Roese wrote:
> The check for the missing PEX switch can lead to an infinite loop, when
> the PCIe device is not found. It is helpful to enable the user to break
> out of this boot-loop, to enable booting to the prompt for test cases.
> This patch adds a 3 second delay in the error case before rebooting.
> The user can press Ctrl-C in this time to abort the boot-loop.
> 
> This patch also calls bootcount_inc() before rebooting in the error
> case. This is needed to increment the bootcounter, since this function
> is called earlier than the main bootcounter increment. Otherwise the
> bootcounter will not be incremented in the error case at all.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> ---
> v2:
> - Rebase on current master

Applied to u-boot-marvell/master.

Thanks,
Stefan
diff mbox series

Patch

diff --git a/board/theadorable/theadorable.c b/board/theadorable/theadorable.c
index acd46b0026..65e90381e1 100644
--- a/board/theadorable/theadorable.c
+++ b/board/theadorable/theadorable.c
@@ -1,11 +1,15 @@ 
 // SPDX-License-Identifier: GPL-2.0+
 /*
- * Copyright (C) 2015-2016 Stefan Roese <sr@denx.de>
+ * Copyright (C) 2015-2019 Stefan Roese <sr@denx.de>
  */
 
 #include <common.h>
+#include <console.h>
 #include <i2c.h>
 #include <pci.h>
+#if !defined(CONFIG_SPL_BUILD)
+#include <bootcount.h>
+#endif
 #include <asm/gpio.h>
 #include <asm/io.h>
 #include <asm/arch/cpu.h>
@@ -42,6 +46,7 @@  DECLARE_GLOBAL_DATA_PTR;
 #define STM_I2C_BUS	1
 #define STM_I2C_ADDR	0x27
 #define REBOOT_DELAY	1000		/* reboot-delay in ms */
+#define ABORT_TIMEOUT	3000		/* 3 seconds reboot abort timeout */
 
 /* DDR3 static configuration */
 static MV_DRAM_MC_INIT ddr3_theadorable[MV_MAX_DDR3_STATIC_SIZE] = {
@@ -222,7 +227,7 @@  int board_eth_init(bd_t *bis)
 }
 #endif
 
-#ifdef CONFIG_BOARD_LATE_INIT
+#if !defined(CONFIG_SPL_BUILD) && defined(CONFIG_BOARD_LATE_INIT)
 int board_late_init(void)
 {
 	pci_dev_t bdf;
@@ -236,6 +241,7 @@  int board_late_init(void)
 	 */
 	bdf = pci_find_device(PCI_VENDOR_ID_PLX, 0x8619, 0);
 	if (bdf == -1) {
+		unsigned long start_time = get_timer(0);
 		u8 i2c_buf[8];
 		int ret;
 
@@ -243,6 +249,28 @@  int board_late_init(void)
 		bootcount = bootcount_load();
 		printf("Failed to find PLX PEX-switch (bootcount=%ld)\n",
 		       bootcount);
+
+		/*
+		 * The user can exit this boot-loop in the error case by
+		 * hitting Ctrl-C. So wait some time for this key here.
+		 */
+		printf("Continue booting with Ctrl-C, otherwise rebooting\n");
+		do {
+			/* Handle control-c and timeouts */
+			if (ctrlc()) {
+				printf("PEX error boot-loop aborted!\n");
+				return 0;
+			}
+		} while (get_timer(start_time) < ABORT_TIMEOUT);
+
+
+		/*
+		 * At this stage the bootcounter has not been incremented
+		 * yet. We need to do this manually here to get an actually
+		 * working bootcounter in this error case.
+		 */
+		bootcount_inc();
+
 		if (bootcount > PEX_SWITCH_NOT_FOUNT_LIMIT) {
 			printf("Issuing power-switch via uC!\n");