[PATCH V5 05/12] iot2050: Add CONFIG_ENV_FLAGS_LIST_STATIC
Tom Rini
trini at konsulko.com
Fri Feb 3 19:52:28 CET 2023
On Fri, Feb 03, 2023 at 01:26:34PM +0100, Jan Kiszka wrote:
> From: Jan Kiszka <jan.kiszka at siemens.com>
>
> Will be needed when CONFIG_ENV_WRITEABLE_LIST is enabled. The listed
> variables shall remain writable, for informational purposes - they have
> to be considered untrusted because the persistent U-Boot env is not
> protected.
>
> Signed-off-by: Jan Kiszka <jan.kiszka at siemens.com>
> ---
> include/configs/iot2050.h | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/include/configs/iot2050.h b/include/configs/iot2050.h
> index 5186dfd8ff8..52094e18ea8 100644
> --- a/include/configs/iot2050.h
> +++ b/include/configs/iot2050.h
> @@ -56,4 +56,12 @@
>
> #include <configs/ti_armv7_common.h>
>
> +#ifdef CONFIG_ENV_WRITEABLE_LIST
> +/* relevant for secure boot with CONFIG_ENV_WRITEABLE_LIST=y */
> +#define CONFIG_ENV_FLAGS_LIST_STATIC \
> + "board_uuid:sw,board_name:sw,board_serial:sw,board_a5e:sw," \
> + "mlfb:sw,fw_version:sw,seboot_version:sw," \
> + "eth1addr:mw,eth2addr:mw,watchdog_timeout_ms:dw,boot_targets:sw"
> +#endif
> +
> #endif /* __CONFIG_IOT2050_H */
I don't think you've tested the whole series on top of current master,
this needs to be CFG_ENV_FLAGS_LIST_STATIC. If this is the only thing
that needs changing, I can just correct this while applying, otherwise a
v6, and I'll try my best to not forget to grab this before -rc2, I know
this whole series has been waiting a while so I thank you for your
patience and persistence here.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230203/c88747c1/attachment.sig>
More information about the U-Boot
mailing list