[PATCH 1/1] lib: allow printing RISC-V EFI Boot Protocol GUID

[Simon Glass]

Hi Heinrich,

On Sat, 29 Jan 2022 at 14:40, Heinrich Schuchardt
<heinrich.schuchardt at 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.schuchardt at canonical.com> wrote:
> >>
> >> On RISC-V a new UEFI protocol has been introduced. Support printing
> >> its GUID using %pUs.
> >>
> >> Cc: Sunil V L <sunilvl at ventanamicro.com>
> >> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at 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