Skip to content

riscv: Enable efi_loader support

Alexander Graf u-boot 6 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest posting 4/40 of 8 patches 0 repliesNew
  2. v3 posting 3/40 of 8 patches 0 repliesSuperseded
  3. v3 posting 2/40 of 8 patches 0 repliesSuperseded
Earlier versions · 3
VersionPostedPatchesRepliesStatus
v3 posting 1/48 patches0 repliesSuperseded
v28 patches0 repliesSuperseded
v18 patches0 repliesSuperseded

Patches in v3 posting 4/4 No applies recorded yet

None of the 8 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.

Read the v3 thread

Cover letter [U-Boot] [PATCH v3 0/8] riscv: Enable efi_loader support

Am 10.05.2018 um 05:27 schrieb Rick Chen <[email protected]>:

Quoted 53 lines
Quoted 53 lines
Quoted 27 lines

Then I think if I don't want enter this auto flow, what can I do ?

1 Remove CONFIG_DISTRO_DEFAULTS=y from nx25-ae250_defconfig But it still enter auto flow.

2 Add #ifdef CONFIG_DISTRO_DEFAULTS to encapsulate the CONFIG_EXTRA_ENV_SETTINGS in nx25-ae250.h And Remove CONFIG_DISTRO_DEFAULTS=y from nx25-ae250_defconfig And it will NOT enter auto flow

#ifdef CONFIG_DISTRO_DEFAULTS
/* Enable distro boot */
#define BOOT_TARGET_DEVICES(func) \
func(MMC, mmc, 0) \
func(DHCP, dhcp, na)

#include <config_distro_bootcmd.h>

#define CONFIG_EXTRA_ENV_SETTINGS       \
                               "kernel_addr_r=0x00080000\0" \
                               "pxefile_addr_r=0x01f00000\0" \
                               "scriptaddr=0x01f00000\0" \
                               "fdt_addr_r=0x02000000\0" \
                               "ramdisk_addr_r=0x02800000\0" \
                               BOOTENV #endif
Quoted 3 lines

3 Add #ifdef CONFIG_DISTRO_DEFAULTS to encapsulate the CONFIG_EXTRA_ENV_SETTINGS in nx25-ae250.h And do NOT Remove CONFIG_DISTRO_DEFAULTS=y from nx25-ae250_defconfig But disable

CONFIG_DISTRO_DEFAULTS from make menuconfig

Quoted 5 lines

General setup --->
[ ] Select defaults suitable for booting general purpose Linux
distributions And it still enter auto flow

Offering this information for you.

This is the same for all distro enabled boards. The basic idea is that U-Boot ships with something that "just boots" for simple use cases.

If you want something more advanced, you usually want to put something very target specific in there anyway, such as "bootm $nor_flash". In that case, you adapt the CONFIG_BOOTCOMMAND parameter to whatever you want.

The boot command can also be overridden using the environment. So if you have working environment store, you can change the U-Boot variable "bootcmd" and U-Boot will run that instead when it boots.

The alternative to this default behavior would be no bootcmd at all which means you just get dropped into the U-Boot prompt. While that's reasonably useful for developers, it isn't for people deploying U-Boot on real target systems :).

Quoted 6 lines
Quoted 1 line

Alex

Thanks for your advises.

After you send those patchs into main trunk.
I will fetch and push them to u-boot-riscv.git :)

The usual flow would be the other way around: You apply them to u-boot-riscv.git with your Signed-off-by and send a pull request for the current merge window :).

Unless I misunderstood your earlier reply and you would prefer to have me propagate them to mainline via the efi-next tree?

Alex