All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX
@ 2020-07-29 15:42 Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 1/6] efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system Heinrich Schuchardt
                   ` (7 more replies)
  0 siblings, 8 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

For RISC-V testing on real hardware the Sipeed MAIX boards provides an
affordable solution.

UEFI testing fails on the MAIX BiT with Mic because the UEFI sub-system
reserves 16 MiB for the stack though the board only has 8 MiB.

The first two patches make the size of the memory reserved for the stack
customizable on all boards defaulting to 16 MiB.

The value on the the MAIX BiT with Mic is set to 1 MiB.

Missing default load addresses for the MAIX BiT with Mic are added.

The sysreset driver is fixed to output the full reset message.

The UEFI network and block device unit tests are only built if the
configuration provides the prerequisites.


Heinrich Schuchardt (6):
  efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system
  configs: reduce stack size of Sipeed MAIX
  riscv: load addresses for Sipeed MAIX
  dm: sysreset: wait after reset message
  efi_selftest: SNP test depends on network
  efi_selftest: block device test requires CONFIG_DOS_PARTITION

 Kconfig                            | 10 ++++++++++
 arch/arm/Kconfig                   | 17 -----------------
 arch/microblaze/Kconfig            |  8 --------
 configs/sipeed_maix_bitm_defconfig |  1 +
 drivers/sysreset/sysreset-uclass.c |  1 +
 include/configs/sipeed-maix.h      |  9 +++++++++
 lib/efi_loader/efi_memory.c        |  2 +-
 lib/efi_selftest/Kconfig           |  2 ++
 lib/efi_selftest/Makefile          |  5 +++--
 9 files changed, 27 insertions(+), 28 deletions(-)

--
2.27.0

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 1/6] efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 2/6] configs: reduce stack size of Sipeed MAIX Heinrich Schuchardt
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

The Kconfig symbol CONFIG_STACK_SIZE is used both by ARM and Microblaze
with the same meaning. Move it to menu 'General setup' so that we can use
it for all architectures.

Use the value of CONFIG_STACK_SIZE instead of a hard coded 16 MiB value for
reserving memory in the UEFI sub-system.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 Kconfig                     | 10 ++++++++++
 arch/arm/Kconfig            | 17 -----------------
 arch/microblaze/Kconfig     |  8 --------
 lib/efi_loader/efi_memory.c |  2 +-
 4 files changed, 11 insertions(+), 26 deletions(-)

diff --git a/Kconfig b/Kconfig
index e6308f30e5..fda35f9e45 100644
--- a/Kconfig
+++ b/Kconfig
@@ -350,6 +350,16 @@ config PLATFORM_ELFENTRY
 	default "__start" if MIPS
 	default "_start"

+config STACK_SIZE
+	hex "Define max stack size that can be used by U-Boot"
+	default 0x4000000 if ARCH_VERSAL || ARCH_ZYNQMP
+	default 0x200000 if MICROBLAZE
+	default 0x1000000
+	help
+	  Define Max stack size that can be used by U-Boot. This value is used
+	  by the UEFI sub-system. On some boards initrd_high is calculated as
+	  base stack pointer minus this stack size.
+
 endmenu		# General setup

 menu "Boot images"
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index ded8cfee09..e01196e724 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -501,23 +501,6 @@ config TPL_USE_ARCH_MEMSET
 	  Such an implementation may be faster under some conditions
 	  but may increase the binary size.

-config SET_STACK_SIZE
-	bool "Enable an option to set max stack size that can be used"
-	default y if ARCH_VERSAL || ARCH_ZYNQMP || ARCH_ZYNQ
-	help
-	  This will enable an option to set max stack size that can be
-	  used by U-Boot.
-
-config STACK_SIZE
-	hex "Define max stack size that can be used by U-Boot"
-	depends on SET_STACK_SIZE
-	default 0x4000000 if ARCH_VERSAL || ARCH_ZYNQMP
-	default 0x1000000 if ARCH_ZYNQ
-	help
-	  Define Max stack size that can be used by U-Boot so that the
-	  initrd_high will be calculated as base stack pointer minus this
-	  stack size.
-
 config ARM64_SUPPORT_AARCH32
 	bool "ARM64 system support AArch32 execution state"
 	depends on ARM64
diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
index 2bd260e5d7..ff6b3c7e3d 100644
--- a/arch/microblaze/Kconfig
+++ b/arch/microblaze/Kconfig
@@ -20,14 +20,6 @@ config TARGET_MICROBLAZE_GENERIC

 endchoice

-config STACK_SIZE
-	hex "Define max stack size that can be used by u-boot"
-	default 0x200000
-	help
-	  Defines Max stack size that can be used by u-boot so that the
-	  initrd_high will be calculated as base stack pointer minus this
-	  stack size.
-
 source "board/xilinx/microblaze-generic/Kconfig"

 endmenu
diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
index 44b8a2e09f..7be756e370 100644
--- a/lib/efi_loader/efi_memory.c
+++ b/lib/efi_loader/efi_memory.c
@@ -762,7 +762,7 @@ static void add_u_boot_and_runtime(void)
 	unsigned long runtime_start, runtime_end, runtime_pages;
 	unsigned long runtime_mask = EFI_PAGE_MASK;
 	unsigned long uboot_start, uboot_pages;
-	unsigned long uboot_stack_size = 16 * 1024 * 1024;
+	unsigned long uboot_stack_size = CONFIG_STACK_SIZE;

 	/* Add U-Boot */
 	uboot_start = ((uintptr_t)map_sysmem(gd->start_addr_sp, 0) -
--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 2/6] configs: reduce stack size of Sipeed MAIX
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 1/6] efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-08-01 13:32   ` Sean Anderson
  2020-07-29 15:42 ` [PATCH 3/6] riscv: load addresses for " Heinrich Schuchardt
                   ` (5 subsequent siblings)
  7 siblings, 1 reply; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

The K210 has only 8 MiB RAM thereof 2 MiB reserved for AI.

Allow only 1 MiB for the stack.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 configs/sipeed_maix_bitm_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/sipeed_maix_bitm_defconfig b/configs/sipeed_maix_bitm_defconfig
index 96651f0268..459bf0d530 100644
--- a/configs/sipeed_maix_bitm_defconfig
+++ b/configs/sipeed_maix_bitm_defconfig
@@ -1,6 +1,7 @@
 CONFIG_RISCV=y
 CONFIG_TARGET_SIPEED_MAIX=y
 CONFIG_ARCH_RV64I=y
+CONFIG_STACK_SIZE=0x100000
 # CONFIG_NET is not set
 # CONFIG_INPUT is not set
 # CONFIG_DM_ETH is not set
--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 3/6] riscv: load addresses for Sipeed MAIX
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 1/6] efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 2/6] configs: reduce stack size of Sipeed MAIX Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-08-01 10:04   ` Heinrich Schuchardt
       [not found]   ` <752D002CFF5D0F4FA35C0100F1D73F3FA473151D@ATCPCS16.andestech.com>
  2020-07-29 15:42 ` [PATCH 4/6] dm: sysreset: wait after reset message Heinrich Schuchardt
                   ` (4 subsequent siblings)
  7 siblings, 2 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

Define default load addresses and the device tree name for the Sipeed MAIX.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 include/configs/sipeed-maix.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/include/configs/sipeed-maix.h b/include/configs/sipeed-maix.h
index a46473fc78..978db6fb00 100644
--- a/include/configs/sipeed-maix.h
+++ b/include/configs/sipeed-maix.h
@@ -21,4 +21,13 @@
 /* For early init */
 #define K210_SYSCTL_BASE 0x50440000

+#ifndef CONFIG_EXTRA_ENV_SETTINGS
+#define CONFIG_EXTRA_ENV_SETTINGS \
+	"loadaddr=0x80040000\0" \
+	"fdt_addr_r=0x80010000\0" \
+	"scriptaddr=0x80008000\0" \
+	"kernel_addr_r=0x80040000\0" \
+	"fdtfile=kendryte/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0"
+#endif
+
 #endif /* CONFIGS_SIPEED_MAIX_H */
--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 4/6] dm: sysreset: wait after reset message
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
                   ` (2 preceding siblings ...)
  2020-07-29 15:42 ` [PATCH 3/6] riscv: load addresses for " Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-08-04  2:00   ` Simon Glass
  2020-07-29 15:42 ` [PATCH 5/6] efi_selftest: SNP test depends on network Heinrich Schuchardt
                   ` (3 subsequent siblings)
  7 siblings, 1 reply; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

In our Python tests we expect to see the word "resetting". It may be
truncated if we reset before the serial console buffer is transferred.

Wait for 100 ms between the "resetting ..." message and the actual reset
like we do when powering off.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 drivers/sysreset/sysreset-uclass.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/sysreset/sysreset-uclass.c b/drivers/sysreset/sysreset-uclass.c
index 995240f0cb..3f5414ed1f 100644
--- a/drivers/sysreset/sysreset-uclass.c
+++ b/drivers/sysreset/sysreset-uclass.c
@@ -117,6 +117,7 @@ void reset_cpu(ulong addr)
 int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
 {
 	printf("resetting ...\n");
+	mdelay(100);

 	sysreset_walk_halt(SYSRESET_COLD);

--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 5/6] efi_selftest: SNP test depends on network
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
                   ` (3 preceding siblings ...)
  2020-07-29 15:42 ` [PATCH 4/6] dm: sysreset: wait after reset message Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-07-29 15:42 ` [PATCH 6/6] efi_selftest: block device test requires CONFIG_DOS_PARTITION Heinrich Schuchardt
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

If CONFIG_NET=n, testing the simple network protocol makes no sense.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 lib/efi_selftest/Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/efi_selftest/Makefile b/lib/efi_selftest/Makefile
index e9baa64135..7f849032ed 100644
--- a/lib/efi_selftest/Makefile
+++ b/lib/efi_selftest/Makefile
@@ -32,7 +32,6 @@ efi_selftest_memory.o \
 efi_selftest_open_protocol.o \
 efi_selftest_register_notify.o \
 efi_selftest_set_virtual_address_map.o \
-efi_selftest_snp.o \
 efi_selftest_textinput.o \
 efi_selftest_textinputex.o \
 efi_selftest_textoutput.o \
@@ -42,6 +41,8 @@ efi_selftest_variables.o \
 efi_selftest_variables_runtime.o \
 efi_selftest_watchdog.o

+obj-$(CONFIG_NET) += efi_selftest_snp.o
+
 obj-$(CONFIG_EFI_DEVICE_PATH_TO_TEXT) += efi_selftest_devicepath.o
 obj-$(CONFIG_EFI_UNICODE_COLLATION_PROTOCOL2) += \
 efi_selftest_unicode_collation.o
--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 6/6] efi_selftest: block device test requires CONFIG_DOS_PARTITION
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
                   ` (4 preceding siblings ...)
  2020-07-29 15:42 ` [PATCH 5/6] efi_selftest: SNP test depends on network Heinrich Schuchardt
@ 2020-07-29 15:42 ` Heinrich Schuchardt
  2020-07-29 16:00 ` [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
  2020-08-01 13:29 ` Sean Anderson
  7 siblings, 0 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 15:42 UTC (permalink / raw)
  To: u-boot

Do not execute the block device test if CONFIG_DOS_PARTITION=n.

Imply CONFIG_DOS_PARTITION in Kconfig.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 lib/efi_selftest/Kconfig  | 2 ++
 lib/efi_selftest/Makefile | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/lib/efi_selftest/Kconfig b/lib/efi_selftest/Kconfig
index 478140330b..ca62436108 100644
--- a/lib/efi_selftest/Kconfig
+++ b/lib/efi_selftest/Kconfig
@@ -1,6 +1,8 @@
 config CMD_BOOTEFI_SELFTEST
 	bool "UEFI unit tests"
 	depends on CMD_BOOTEFI
+	imply PARTITIONS
+	imply DOS_PARTITION
 	imply FAT
 	imply FAT_WRITE
 	imply CMD_POWEROFF if PSCI_RESET || SYSRESET_PSCI
diff --git a/lib/efi_selftest/Makefile b/lib/efi_selftest/Makefile
index 7f849032ed..45ce6859b8 100644
--- a/lib/efi_selftest/Makefile
+++ b/lib/efi_selftest/Makefile
@@ -57,7 +57,7 @@ ifeq ($(CONFIG_GENERATE_ACPI_TABLE),)
 obj-y += efi_selftest_fdt.o
 endif

-ifeq ($(CONFIG_BLK)$(CONFIG_PARTITIONS),yy)
+ifeq ($(CONFIG_BLK)$(CONFIG_DOS_PARTITION),yy)
 obj-y += efi_selftest_block_device.o
 endif

--
2.27.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
                   ` (5 preceding siblings ...)
  2020-07-29 15:42 ` [PATCH 6/6] efi_selftest: block device test requires CONFIG_DOS_PARTITION Heinrich Schuchardt
@ 2020-07-29 16:00 ` Heinrich Schuchardt
  2020-08-01 13:29 ` Sean Anderson
  7 siblings, 0 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-07-29 16:00 UTC (permalink / raw)
  To: u-boot

On 29.07.20 17:42, Heinrich Schuchardt wrote:
> For RISC-V testing on real hardware the Sipeed MAIX boards provides an
> affordable solution.
>
> UEFI testing fails on the MAIX BiT with Mic because the UEFI sub-system
> reserves 16 MiB for the stack though the board only has 8 MiB.
>
> The first two patches make the size of the memory reserved for the stack
> customizable on all boards defaulting to 16 MiB.
>
> The value on the the MAIX BiT with Mic is set to 1 MiB.
>
> Missing default load addresses for the MAIX BiT with Mic are added.
>
> The sysreset driver is fixed to output the full reset message.
>
> The UEFI network and block device unit tests are only built if the
> configuration provides the prerequisites.
>
>
> Heinrich Schuchardt (6):
>   efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system
>   configs: reduce stack size of Sipeed MAIX
>   riscv: load addresses for Sipeed MAIX
>   dm: sysreset: wait after reset message
>   efi_selftest: SNP test depends on network
>   efi_selftest: block device test requires CONFIG_DOS_PARTITION

This patch is prerequisite for the series:

https://lists.denx.de/pipermail/u-boot/2020-July/421925.html
x86: rename CONFIG_STACK_SIZE

Best regards

Heinrich


>
>  Kconfig                            | 10 ++++++++++
>  arch/arm/Kconfig                   | 17 -----------------
>  arch/microblaze/Kconfig            |  8 --------
>  configs/sipeed_maix_bitm_defconfig |  1 +
>  drivers/sysreset/sysreset-uclass.c |  1 +
>  include/configs/sipeed-maix.h      |  9 +++++++++
>  lib/efi_loader/efi_memory.c        |  2 +-
>  lib/efi_selftest/Kconfig           |  2 ++
>  lib/efi_selftest/Makefile          |  5 +++--
>  9 files changed, 27 insertions(+), 28 deletions(-)
>
> --
> 2.27.0
>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 3/6] riscv: load addresses for Sipeed MAIX
  2020-07-29 15:42 ` [PATCH 3/6] riscv: load addresses for " Heinrich Schuchardt
@ 2020-08-01 10:04   ` Heinrich Schuchardt
       [not found]   ` <752D002CFF5D0F4FA35C0100F1D73F3FA473151D@ATCPCS16.andestech.com>
  1 sibling, 0 replies; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-08-01 10:04 UTC (permalink / raw)
  To: u-boot

On 7/29/20 5:42 PM, Heinrich Schuchardt wrote:
> Define default load addresses and the device tree name for the Sipeed MAIX.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  include/configs/sipeed-maix.h | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/include/configs/sipeed-maix.h b/include/configs/sipeed-maix.h
> index a46473fc78..978db6fb00 100644
> --- a/include/configs/sipeed-maix.h
> +++ b/include/configs/sipeed-maix.h
> @@ -21,4 +21,13 @@
>  /* For early init */
>  #define K210_SYSCTL_BASE 0x50440000
>
> +#ifndef CONFIG_EXTRA_ENV_SETTINGS
> +#define CONFIG_EXTRA_ENV_SETTINGS \
> +	"loadaddr=0x80040000\0" \
> +	"fdt_addr_r=0x80010000\0" \
> +	"scriptaddr=0x80008000\0" \
> +	"kernel_addr_r=0x80040000\0" \
> +	"fdtfile=kendryte/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0"
> +#endif
> +
>  #endif /* CONFIGS_SIPEED_MAIX_H */
> --
> 2.27.0
>

Probably we should choose the addresses here higher to allow for
OpenSBI. I am still trying to make OpenSBI load U-Boot on the board.

Best regards

Heinrich

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX
  2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
                   ` (6 preceding siblings ...)
  2020-07-29 16:00 ` [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
@ 2020-08-01 13:29 ` Sean Anderson
  7 siblings, 0 replies; 15+ messages in thread
From: Sean Anderson @ 2020-08-01 13:29 UTC (permalink / raw)
  To: u-boot

On 7/29/20 11:42 AM, Heinrich Schuchardt wrote:
> For RISC-V testing on real hardware the Sipeed MAIX boards provides an
> affordable solution.
> 
> UEFI testing fails on the MAIX BiT with Mic because the UEFI sub-system
> reserves 16 MiB for the stack though the board only has 8 MiB.
> 
> The first two patches make the size of the memory reserved for the stack
> customizable on all boards defaulting to 16 MiB.
> 
> The value on the the MAIX BiT with Mic is set to 1 MiB.
> 
> Missing default load addresses for the MAIX BiT with Mic are added.
> 
> The sysreset driver is fixed to output the full reset message.
> 
> The UEFI network and block device unit tests are only built if the
> configuration provides the prerequisites.
> 
> 
> Heinrich Schuchardt (6):
>   efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system
>   configs: reduce stack size of Sipeed MAIX
>   riscv: load addresses for Sipeed MAIX
>   dm: sysreset: wait after reset message
>   efi_selftest: SNP test depends on network
>   efi_selftest: block device test requires CONFIG_DOS_PARTITION
> 
>  Kconfig                            | 10 ++++++++++
>  arch/arm/Kconfig                   | 17 -----------------
>  arch/microblaze/Kconfig            |  8 --------
>  configs/sipeed_maix_bitm_defconfig |  1 +
>  drivers/sysreset/sysreset-uclass.c |  1 +
>  include/configs/sipeed-maix.h      |  9 +++++++++
>  lib/efi_loader/efi_memory.c        |  2 +-
>  lib/efi_selftest/Kconfig           |  2 ++
>  lib/efi_selftest/Makefile          |  5 +++--
>  9 files changed, 27 insertions(+), 28 deletions(-)
> 
> --
> 2.27.0

When testing this series, after completing the self test the board does
not reset automatically, even after pressing a key. Here is an
(abbreviated) log

	=> bootefi selftest

	Testing EFI API implementation

	Number of tests to execute: 32

	<snip>

	Summary: 0 failures

	Preparing for reset. Press any key...

And after this I need to manually reset the board. Running the reset
command from within the U-Boot shell works fine. Is this intended
behavior?

In addition, the self test changes the default terminal colors and does
not change them back. This is most noticeable when using a terminal which
does not default to white text on a black background. As an example, I
have configured my terminal to use black text on a white background in
[1]. However, once the test has completed, the terminal colors are not
returned to their defaults [2]. (in this example, I manually reset the
board). Is this expected behavior?

[1] https://imgur.com/AUuVLFI.png
[2] https://imgur.com/6YGtK9n.png

--Sean

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 2/6] configs: reduce stack size of Sipeed MAIX
  2020-07-29 15:42 ` [PATCH 2/6] configs: reduce stack size of Sipeed MAIX Heinrich Schuchardt
@ 2020-08-01 13:32   ` Sean Anderson
  0 siblings, 0 replies; 15+ messages in thread
From: Sean Anderson @ 2020-08-01 13:32 UTC (permalink / raw)
  To: u-boot


On 7/29/20 11:42 AM, Heinrich Schuchardt wrote:
> The K210 has only 8 MiB RAM thereof 2 MiB reserved for AI.
> 
> Allow only 1 MiB for the stack.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  configs/sipeed_maix_bitm_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/configs/sipeed_maix_bitm_defconfig b/configs/sipeed_maix_bitm_defconfig
> index 96651f0268..459bf0d530 100644
> --- a/configs/sipeed_maix_bitm_defconfig
> +++ b/configs/sipeed_maix_bitm_defconfig
> @@ -1,6 +1,7 @@
>  CONFIG_RISCV=y
>  CONFIG_TARGET_SIPEED_MAIX=y
>  CONFIG_ARCH_RV64I=y
> +CONFIG_STACK_SIZE=0x100000
>  # CONFIG_NET is not set
>  # CONFIG_INPUT is not set
>  # CONFIG_DM_ETH is not set
> --
> 2.27.0
> 

Why is the stack size set so large? On Linux they use 16K stacks with
64-bit RISC-V. Does EFI need an especially large stack?

--Sean

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 4/6] dm: sysreset: wait after reset message
  2020-07-29 15:42 ` [PATCH 4/6] dm: sysreset: wait after reset message Heinrich Schuchardt
@ 2020-08-04  2:00   ` Simon Glass
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Glass @ 2020-08-04  2:00 UTC (permalink / raw)
  To: u-boot

Hi Heinrich,

On Wed, 29 Jul 2020 at 09:43, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> In our Python tests we expect to see the word "resetting". It may be
> truncated if we reset before the serial console buffer is transferred.
>
> Wait for 100 ms between the "resetting ..." message and the actual reset
> like we do when powering off.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  drivers/sysreset/sysreset-uclass.c | 1 +
>  1 file changed, 1 insertion(+)

Please add a comment as to the reason for the delay. and how you chose
the value.

Reviewed-by: Simon Glass <sjg@chromium.org>


>
> diff --git a/drivers/sysreset/sysreset-uclass.c b/drivers/sysreset/sysreset-uclass.c
> index 995240f0cb..3f5414ed1f 100644
> --- a/drivers/sysreset/sysreset-uclass.c
> +++ b/drivers/sysreset/sysreset-uclass.c
> @@ -117,6 +117,7 @@ void reset_cpu(ulong addr)
>  int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
>  {
>         printf("resetting ...\n");
> +       mdelay(100);
>
>         sysreset_walk_halt(SYSRESET_COLD);
>
> --
> 2.27.0
>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 3/6] riscv: load addresses for Sipeed MAIX
       [not found]   ` <752D002CFF5D0F4FA35C0100F1D73F3FA473151D@ATCPCS16.andestech.com>
@ 2020-08-06  7:07     ` Rick Chen
  2020-08-07 17:50       ` Heinrich Schuchardt
  0 siblings, 1 reply; 15+ messages in thread
From: Rick Chen @ 2020-08-06  7:07 UTC (permalink / raw)
  To: u-boot

> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Heinrich Schuchardt
> Sent: Wednesday, July 29, 2020 11:43 PM
> To: Sean Anderson
> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; u-boot at lists.denx.de; Heinrich Schuchardt
> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
>
> Define default load addresses and the device tree name for the Sipeed MAIX.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  include/configs/sipeed-maix.h | 9 +++++++++
>  1 file changed, 9 insertions(+)

Acked-by: Rick Chen <rick@andestech.com>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 3/6] riscv: load addresses for Sipeed MAIX
  2020-08-06  7:07     ` Rick Chen
@ 2020-08-07 17:50       ` Heinrich Schuchardt
  2020-08-12  7:09         ` Rick Chen
  0 siblings, 1 reply; 15+ messages in thread
From: Heinrich Schuchardt @ 2020-08-07 17:50 UTC (permalink / raw)
  To: u-boot

On 8/6/20 9:07 AM, Rick Chen wrote:
>> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Heinrich Schuchardt
>> Sent: Wednesday, July 29, 2020 11:43 PM
>> To: Sean Anderson
>> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; u-boot at lists.denx.de; Heinrich Schuchardt
>> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
>>
>> Define default load addresses and the device tree name for the Sipeed MAIX.
>>
>> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>> ---
>>  include/configs/sipeed-maix.h | 9 +++++++++
>>  1 file changed, 9 insertions(+)
>
> Acked-by: Rick Chen <rick@andestech.com>
>

Thanks for reviewing.

I have reassigned the patch to you in patchwork. The rest of the series
was already merged via the last EFI pull request.

Best regards

Heinrich

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 3/6] riscv: load addresses for Sipeed MAIX
  2020-08-07 17:50       ` Heinrich Schuchardt
@ 2020-08-12  7:09         ` Rick Chen
  0 siblings, 0 replies; 15+ messages in thread
From: Rick Chen @ 2020-08-12  7:09 UTC (permalink / raw)
  To: u-boot

Hi Heinrich

> On 8/6/20 9:07 AM, Rick Chen wrote:
> >> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Heinrich Schuchardt
> >> Sent: Wednesday, July 29, 2020 11:43 PM
> >> To: Sean Anderson
> >> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; u-boot at lists.denx.de; Heinrich Schuchardt
> >> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
> >>
> >> Define default load addresses and the device tree name for the Sipeed MAIX.
> >>
> >> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> >> ---
> >>  include/configs/sipeed-maix.h | 9 +++++++++
> >>  1 file changed, 9 insertions(+)
> >
> > Acked-by: Rick Chen <rick@andestech.com>
> >
>
> Thanks for reviewing.
>
> I have reassigned the patch to you in patchwork. The rest of the series
> was already merged via the last EFI pull request.

It is better to be pulled with the whole patch series together via EFI tree.
Because the title of this patch sets is [PATCH 0/6] efi_selftest:
allow UEFI testing on MAIX.

And you said that you still try the address for OpenSBI in previous mail.

Thanks,
Rick

>
> Best regards
>
> Heinrich

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2020-08-12  7:09 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-29 15:42 [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
2020-07-29 15:42 ` [PATCH 1/6] efi_loader: use CONFIG_STACK_SIZE in the UEFI sub-system Heinrich Schuchardt
2020-07-29 15:42 ` [PATCH 2/6] configs: reduce stack size of Sipeed MAIX Heinrich Schuchardt
2020-08-01 13:32   ` Sean Anderson
2020-07-29 15:42 ` [PATCH 3/6] riscv: load addresses for " Heinrich Schuchardt
2020-08-01 10:04   ` Heinrich Schuchardt
     [not found]   ` <752D002CFF5D0F4FA35C0100F1D73F3FA473151D@ATCPCS16.andestech.com>
2020-08-06  7:07     ` Rick Chen
2020-08-07 17:50       ` Heinrich Schuchardt
2020-08-12  7:09         ` Rick Chen
2020-07-29 15:42 ` [PATCH 4/6] dm: sysreset: wait after reset message Heinrich Schuchardt
2020-08-04  2:00   ` Simon Glass
2020-07-29 15:42 ` [PATCH 5/6] efi_selftest: SNP test depends on network Heinrich Schuchardt
2020-07-29 15:42 ` [PATCH 6/6] efi_selftest: block device test requires CONFIG_DOS_PARTITION Heinrich Schuchardt
2020-07-29 16:00 ` [PATCH 0/6] efi_selftest: allow UEFI testing on MAIX Heinrich Schuchardt
2020-08-01 13:29 ` Sean Anderson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.