Hi Heinrich,
On Fri, 31 Mar 2023 at 11:38, Heinrich Schuchardt <xypron.g...@gmx.de> wrote:
>
>
>
> Am 30. März 2023 23:32:07 MESZ schrieb Simon Glass <s...@chromium.org>:
> >The CMD_EFIDEBUG option enables debugging so it is reasonable to assume
> >that all effects should be made to decode the dreaded UUIDs favoured by
> >UEFI.
> >
> >Update the table to show them all when CONFIG_CMD_EFIDEBUG is enabled.
> >
> >Signed-off-by: Simon Glass <s...@chromium.org>
> >---
> >
> > lib/uuid.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/lib/uuid.c b/lib/uuid.c
> >index 96e1af3c8b00..ab30fbf9152f 100644
> >--- a/lib/uuid.c
> >+++ b/lib/uuid.c
> >@@ -255,7 +255,7 @@ static const struct {
> > EFI_CERT_TYPE_PKCS7_GUID,
> > },
> > #endif
> >-#ifdef CONFIG_EFI
> >+#if defined(CONFIG_CMD_EFIDEBUG) || defined(CONFIG_EFI)
> > { "EFI_LZMA_COMPRESSED", EFI_LZMA_COMPRESSED },
> > { "EFI_DXE_SERVICES", EFI_DXE_SERVICES },
> > { "EFI_HOB_LIST", EFI_HOB_LIST },
>
> None of these are used when not building the EFI app.
So you think we should disable them? As I said above, enabling
debugging seems like a good reason to allow decoding of all of them.
Regards,
SImon
