On Sat, 8 Jun 2024 at 01:39, Jonathan Humphreys <j-humphr...@ti.com> wrote:
>
> Define the firmware components updatable via EFI capsule update, including
> defining capsule GUIDs for the various firmware components for the
> BeaglePlay.
>
> Note this involved creating BeaglePlay's own beagleplay.h board header file
> instead of reusing am62_evm's.
>
> Signed-off-by: Jonathan Humphreys <j-humphr...@ti.com>
> ---
> board/beagle/beagleplay/Kconfig | 4 +--
> board/beagle/beagleplay/beagleplay.c | 34 +++++++++++++++++++++++
> include/configs/beagleplay.h | 41 ++++++++++++++++++++++++++++
> 3 files changed, 77 insertions(+), 2 deletions(-)
> create mode 100644 include/configs/beagleplay.h
>
> diff --git a/board/beagle/beagleplay/Kconfig b/board/beagle/beagleplay/Kconfig
> index 7dbd833acb4..b0e67dc8ef3 100644
> --- a/board/beagle/beagleplay/Kconfig
> +++ b/board/beagle/beagleplay/Kconfig
> @@ -35,7 +35,7 @@ config SYS_VENDOR
> default "beagle"
>
> config SYS_CONFIG_NAME
> - default "am62x_evm"
> + default "beagleplay"
>
> source "board/ti/common/Kconfig"
>
> @@ -50,7 +50,7 @@ config SYS_VENDOR
> default "beagle"
>
> config SYS_CONFIG_NAME
> - default "am62x_evm"
> + default "beagleplay"
>
> config SPL_LDSCRIPT
> default "arch/arm/mach-omap2/u-boot-spl.lds"
> diff --git a/board/beagle/beagleplay/beagleplay.c
> b/board/beagle/beagleplay/beagleplay.c
> index af36439e2e2..a21f09e3122 100644
> --- a/board/beagle/beagleplay/beagleplay.c
> +++ b/board/beagle/beagleplay/beagleplay.c
> @@ -6,6 +6,7 @@
> * Copyright (C) 2022-2023 Robert Nelson, BeagleBoard.org Foundation
> */
>
> +#include <efi_loader.h>
> #include <cpu_func.h>
> #include <env.h>
> #include <fdt_support.h>
> @@ -15,6 +16,39 @@
>
> DECLARE_GLOBAL_DATA_PTR;
>
> +struct efi_fw_image fw_images[] = {
> + {
> + .image_type_id = BEAGLEPLAY_TIBOOT3_IMAGE_GUID,
> + .fw_name = u"BEAGLEPLAY_TIBOOT3",
> + .image_index = 1,
> + },
> + {
> + .image_type_id = BEAGLEPLAY_SPL_IMAGE_GUID,
> + .fw_name = u"BEAGLEPLAY_SPL",
> + .image_index = 2,
> + },
> + {
> + .image_type_id = BEAGLEPLAY_UBOOT_IMAGE_GUID,
> + .fw_name = u"BEAGLEPLAY_UBOOT",
> + .image_index = 3,
> + }
> +};
> +
> +struct efi_capsule_update_info update_info = {
> + .dfu_string = "mmc 0=tiboot3.bin raw 0 2000 mmcpart 1;"
> + "tispl.bin fat 0 1;u-boot.img fat 0 1",
> + .num_images = ARRAY_SIZE(fw_images),
> + .images = fw_images,
> +};
> +
> +#if IS_ENABLED(CONFIG_SET_DFU_ALT_INFO)
> +void set_dfu_alt_info(char *interface, char *devstr)
> +{
> + if (IS_ENABLED(CONFIG_EFI_HAVE_CAPSULE_SUPPORT))
> + env_set("dfu_alt_info", update_info.dfu_string);
> +}
> +#endif
> +
> int board_init(void)
> {
> return 0;
> diff --git a/include/configs/beagleplay.h b/include/configs/beagleplay.h
> new file mode 100644
> index 00000000000..4baeab664af
> --- /dev/null
> +++ b/include/configs/beagleplay.h
> @@ -0,0 +1,41 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Configuration header file for BeaglePlay
> + *
> + * https://beagleplay.org/
> + *
> + * Copyright (C) 2024 Texas Instruments Incorporated - https://www.ti.com/
> + */
> +
> +#ifndef __CONFIG_BEAGLEPLAY_H
> +#define __CONFIG_BEAGLEPLAY_H
> +
> +/**
> + * define BEAGLEPLAY_TIBOOT3_IMAGE_GUID - firmware GUID for BeaglePlay
> + * tiboot3.bin
> + * define BEAGLEPLAY_SPL_IMAGE_GUID - firmware GUID for BeaglePlay SPL
> + * define BEAGLEPLAY_UBOOT_IMAGE_GUID - firmware GUID for BeaglePlay UBOOT
> + *
> + * These GUIDs are used in capsules updates to identify the corresponding
> + * firmware object.
> + *
> + * Board developers using this as a starting reference should
> + * define their own GUIDs to ensure that firmware repositories (like
> + * LVFS) do not confuse them.
> + */
> +#define BEAGLEPLAY_TIBOOT3_IMAGE_GUID \
> + EFI_GUID(0x0e225a09, 0xf720, 0x4d57, 0x91, 0x20, \
> + 0xe2, 0x8f, 0x73, 0x7f, 0x5a, 0x5e)
> +
> +#define BEAGLEPLAY_SPL_IMAGE_GUID \
> + EFI_GUID(0xb2e7cc49, 0x1a5a, 0x4036, 0xae, 0x01, \
> + 0x33, 0x87, 0xc3, 0xbe, 0xf6, 0x57)
> +
> +#define BEAGLEPLAY_UBOOT_IMAGE_GUID \
> + EFI_GUID(0x92c92b11, 0xa7ee, 0x486f, 0xaa, 0xa2, \
> + 0x71, 0x3d, 0x84, 0x42, 0x5b, 0x0e)
> +
> +/* Now for the remaining common defines */
> +#include <configs/ti_armv7_common.h>
> +
> +#endif /* __CONFIG_BEAGLEPLAY_H */
> --
> 2.34.1
>
Acked-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>
