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

[Heinrich Schuchardt]

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?

Best regards

Heinrich


Regards,
Simon