Hi Heinrich, On Sat, 29 Jan 2022 at 14:40, Heinrich Schuchardt <heinrich.schucha...@canonical.com> wrote: > > > > On 1/29/22 22:09, Simon Glass wrote: > > Hi Heinrich, > > > > On Fri, 28 Jan 2022 at 11:42, Heinrich Schuchardt > > <heinrich.schucha...@canonical.com> wrote: > >> > >> On RISC-V a new UEFI protocol has been introduced. Support printing > >> its GUID using %pUs. > >> > >> Cc: Sunil V L <suni...@ventanamicro.com> > >> Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com> > >> --- > >> cf. > >> [RFC PATCH V2 1/2] efi_loader: Enable RISCV_EFI_BOOT_PROTOCOL support > >> https://lists.denx.de/pipermail/u-boot/2022-January/473505.html > >> --- > >> lib/uuid.c | 6 ++++++ > >> 1 file changed, 6 insertions(+) > >> > >> diff --git a/lib/uuid.c b/lib/uuid.c > >> index 24571ef5fb..284f8113ff 100644 > >> --- a/lib/uuid.c > >> +++ b/lib/uuid.c > >> @@ -220,7 +220,13 @@ static const struct { > >> "TCG2 Final Events Table", > >> EFI_TCG2_FINAL_EVENTS_TABLE_GUID, > >> }, > >> +#ifdef CONFIG_EFI_RISCV_BOOT_PROTOCOL > >> + { > >> + "RISC-V Boot", > >> + RISCV_EFI_BOOT_PROTOCOL_GUID, > >> + }, > >> #endif > >> +#endif /* CONFIG_CMD_EFIDEBUG */ > >> #ifdef CONFIG_CMD_NVEDIT_EFI > >> /* signature database */ > >> { > >> -- > >> 2.33.1 > >> > > > > Why the #ifdef? > > We want a small size of the binary. Why should we add a text for a > RISC-V GUID on ARM?
If we wanted a small binary size we would not be using UUIDs. I was thinking that if you load an image for another arch it would be very confusing if it were not recognised. Regards, Simon
