Hello Simon, I've tried out successfully the option (with BOOTSTD_FULL, with BOOTSTD_DEFAULT it's not possible to boot) and will provide a changed patch set soon.
Mit freundlichen Grüßen / Best regards Thomas Mittelstaedt Cross-Domain Computing Solutions > -----Ursprüngliche Nachricht----- > Von: Simon Glass <s...@chromium.org> > Gesendet: Dienstag, 2. Mai 2023 19:12 > An: Heinrich Schuchardt <xypron.g...@gmx.de> > Cc: Mittelstaedt Thomas (XC-CT/EBV3) <thomas.mittelsta...@de.bosch.com>; > u-boot@lists.denx.de; Niel Armstrong <neil.armstr...@linaro.org>; Patrick > Delaunay <patrick.delau...@foss.st.com>; Ramon Fried > <rfried....@gmail.com>; Marek Vasut <ma...@denx.de>; Manuel Traut > <manuel.tr...@mt.com>; Bin Meng <bmeng...@gmail.com> > Betreff: Re: [PATCH v2 2/3] X86: Add support for distro boot > > Hi, > > On Tue, 2 May 2023 at 09:41, Heinrich Schuchardt <xypron.g...@gmx.de> wrote: > > > > > > > > Am 2. Mai 2023 17:21:29 MESZ schrieb thomas.mittelsta...@de.bosch.com: > > >From: Thomas Mittelstaedt <thomas.mittelsta...@de.bosch.com> > > > > > >Enable distro boot feature for U-Boot at VirtualBox described at > > >https://sou/ > > >rce.denx.de%2Fu-boot%2Fu-boot%2F- > %2Fblob%2Fmaster%2Fdoc%2Fdevelop%2Fd > > > >istro.rst&data=05%7C01%7Cthomas.mittelstaedt%40de.bosch.com%7Ca52180e > > > >e6e8f4087da4d08db4b306953%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0% > 7C0% > > > >7C638186443515155378%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAw > MDAiLCJQ > > > >IjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata > =4TSv > > >yIZMiz4Tt7Wt36NxQR576tg%2F9cS%2FVyZS83xwN50%3D&reserved=0 > > > > > >Signed-off-by: Thomas Mittelstaedt <thomas.mittelsta...@bosch.com> > > >--- > > > configs/efi-x86_payload64_defconfig | 12 +----------- > > > include/configs/efi-x86_payload.h | 11 +++++++++++ > > > 2 files changed, 12 insertions(+), 11 deletions(-) > > > > > >diff --git a/configs/efi-x86_payload64_defconfig > > >b/configs/efi-x86_payload64_defconfig > > >index 30a7f31dac..a4cfe95890 100644 > > >--- a/configs/efi-x86_payload64_defconfig > > >+++ b/configs/efi-x86_payload64_defconfig > > >@@ -8,33 +8,23 @@ CONFIG_TARGET_EFI_PAYLOAD=y CONFIG_FIT=y > > >CONFIG_FIT_SIGNATURE=y CONFIG_LEGACY_IMAGE_FORMAT=y > > >+CONFIG_DISTRO_DEFAULTS=y > > > CONFIG_SHOW_BOOT_PROGRESS=y > > > CONFIG_USE_BOOTARGS=y > > > CONFIG_BOOTARGS="root=/dev/sdb3 init=/sbin/init rootwait ro" > > >-CONFIG_USE_BOOTCOMMAND=y > > > CONFIG_PRE_CONSOLE_BUFFER=y > > > CONFIG_SYS_CONSOLE_INFO_QUIET=y > > > CONFIG_DISPLAY_BOARDINFO_LATE=y > > > CONFIG_LAST_STAGE_INIT=y > > >-CONFIG_HUSH_PARSER=y > > > CONFIG_SYS_PBSIZE=532 > > > CONFIG_CMD_IDE=y > > > CONFIG_CMD_MMC=y > > >-CONFIG_CMD_PART=y > > > CONFIG_CMD_USB=y > > >-CONFIG_CMD_DHCP=y > > > CONFIG_BOOTP_BOOTFILESIZE=y > > > # CONFIG_CMD_NFS is not set > > >-CONFIG_CMD_PING=y > > > CONFIG_CMD_TIME=y > > >-CONFIG_CMD_EXT2=y > > >-CONFIG_CMD_EXT4=y > > > CONFIG_CMD_EXT4_WRITE=y > > >-CONFIG_CMD_FAT=y > > >-CONFIG_CMD_FS_GENERIC=y > > > CONFIG_MAC_PARTITION=y > > >-CONFIG_ISO_PARTITION=y > > >-CONFIG_EFI_PARTITION=y > > > CONFIG_ENV_OVERWRITE=y > > > CONFIG_ENV_IS_IN_FAT=y > > > CONFIG_ENV_FAT_INTERFACE="scsi" > > >diff --git a/include/configs/efi-x86_payload.h > > >b/include/configs/efi-x86_payload.h > > >index c72b067c36..e1cd8eb316 100644 > > >--- a/include/configs/efi-x86_payload.h > > >+++ b/include/configs/efi-x86_payload.h > > >@@ -6,6 +6,17 @@ > > > /* > > > * board/config.h - configuration options, board specific > > > */ > > >+#ifndef CONFIG_SPL_BUILD > > >+ > > >+#define BOOT_TARGET_SCSI(func) \ > > > > Shouldn't NVMe be added here too? > > This is automatic if you use bootstd. > > > > > Best regards > > > > Heinrich > > > > >+ func(SCSI, scsi, 0) > > >+ > > >+#define BOOT_TARGET_DEVICES(func) \ > > >+ BOOT_TARGET_SCSI(func) > > >+ > > >+#include <config_distro_bootcmd.h> > > >+ > > >+#endif > > > > > > #ifndef __CONFIG_H > > > #define __CONFIG_H > > Please can we use bootstd instead? > > You should just need to enable BOOTSTD_DEFAULTS and it will work. If not, > please let me know. > > Please also see the various improvements in[1] available at [2]. There is > also a > new video driver [2]. > > Regards, > Simon > > [1] > https://patchwork.o/ > zlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D351440&data=05%7C01 > %7Cthomas.mittelstaedt%40de.bosch.com%7Ca52180ee6e8f4087da4d08db4b306 > 953%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C0%7C6381864435151553 > 78%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL > CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=yjqyizWXqb > YZmSuzfvqq4EfYvJYqXUf6irbLYhLo4qk%3D&reserved=0 > [2] > https://source.denx/ > .de%2Fu-boot%2Fcustodians%2Fu-boot-dm%2F-%2Ftree%2Fbryc- > working&data=05%7C01%7Cthomas.mittelstaedt%40de.bosch.com%7Ca52180ee > 6e8f4087da4d08db4b306953%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C > 0%7C638186443515155378%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA > wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C > %7C&sdata=5OF%2Bmc%2FVOSFFTzLp3JLrWBxvuZCfqwQDb3GzDZGlvws%3 > D&reserved=0 > [3] > https://patchwork.o/ > zlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D352929&data=05%7C01 > %7Cthomas.mittelstaedt%40de.bosch.com%7Ca52180ee6e8f4087da4d08db4b306 > 953%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C0%7C6381864435151553 > 78%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL > CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=VthFwiSeQt > 7c%2B%2F0twxRZTYu4smkJJDL18CkrST4mhiQ%3D&reserved=0