[PATCH v2 0/3] efi: Minimal revert to rodata change

Heinrich Schuchardt xypron.glpk at gmx.de
Thu Aug 5 17:24:12 CEST 2021



On 02.08.21 16:44, Simon Glass wrote:
> The changes to move from devicetree to rodata take things in the wrong
> direction for various reasons:
>
> - devicetree is where config should be stored

We are not talking about configuration here at all.

> - it provides no memory production in any case, particularly when U-Boot

No clue what you mean by "memory production".

>    is relocated
> - testing becomes harder, with the suggestion of adding an entire new
>    sandbox build just for this

Having an extra config is not required when putting the certificate into
.rodata.

Best regards

Heinrich

>
> Revert this until a new direction can be established.
>
> Changes in v2:
> - Also revert two other patches, based on comment from Takahiro
>
> Simon Glass (3):
>    Revert "doc: Update CapsuleUpdate READMEs"
>    Revert "mkeficapsule: Remove dtb related options"
>    Revert "efi_capsule: Move signature from DTB to .rodata"
>
>   board/emulation/common/Makefile             |   1 +
>   board/emulation/common/qemu_capsule.c       |  43 ++++
>   doc/board/emulation/qemu_capsule_update.rst | 203 +++++++++++++++++
>   doc/develop/uefi/uefi.rst                   | 125 -----------
>   include/asm-generic/sections.h              |   2 -
>   lib/efi_loader/Kconfig                      |   7 -
>   lib/efi_loader/Makefile                     |   8 -
>   lib/efi_loader/efi_capsule.c                |  18 +-
>   lib/efi_loader/efi_capsule_key.S            |  17 --
>   tools/mkeficapsule.c                        | 229 +++++++++++++++++++-
>   10 files changed, 472 insertions(+), 181 deletions(-)
>   create mode 100644 board/emulation/common/qemu_capsule.c
>   create mode 100644 doc/board/emulation/qemu_capsule_update.rst
>   delete mode 100644 lib/efi_loader/efi_capsule_key.S
>


More information about the U-Boot mailing list