diff mbox series

[v6,22/24] board: sk-am69: Define capsule update firmware info

Message ID 20240607223858.1971290-23-j-humphreys@ti.com
State Changes Requested
Delegated to: Tom Rini
Headers show
Series EFI: ti: Enable EFI capsule updates | expand

Commit Message

Jonathan Humphreys June 7, 2024, 10:38 p.m. UTC
Define the firmware components updatable via EFI capsule update, including
defining capsule GUIDs for the various firmware components for the AM69
SK.

TODO: possibly make the struct's sk specific.
TODO: add doc commit (and make sure doc is sk/NOR specific, and add OSIP
boot mode)
TODO: update doc to show sk defconfig when building

Signed-off-by: Jonathan Humphreys <j-humphreys@ti.com>
---
 board/ti/j784s4/evm.c        | 34 ++++++++++++++++++++++++++++++++++
 include/configs/j784s4_evm.h | 24 ++++++++++++++++++++++++
 2 files changed, 58 insertions(+)

Comments

Ilias Apalodimas June 10, 2024, 8:51 a.m. UTC | #1
On Sat, 8 Jun 2024 at 01:39, Jonathan Humphreys <j-humphreys@ti.com> wrote:
>
> Define the firmware components updatable via EFI capsule update, including
> defining capsule GUIDs for the various firmware components for the AM69
> SK.
>
> TODO: possibly make the struct's sk specific.
> TODO: add doc commit (and make sure doc is sk/NOR specific, and add OSIP
> boot mode)
> TODO: update doc to show sk defconfig when building
>
> Signed-off-by: Jonathan Humphreys <j-humphreys@ti.com>
> ---
>  board/ti/j784s4/evm.c        | 34 ++++++++++++++++++++++++++++++++++
>  include/configs/j784s4_evm.h | 24 ++++++++++++++++++++++++
>  2 files changed, 58 insertions(+)
>
> diff --git a/board/ti/j784s4/evm.c b/board/ti/j784s4/evm.c
> index aed0ea5b949..548dbd5925d 100644
> --- a/board/ti/j784s4/evm.c
> +++ b/board/ti/j784s4/evm.c
> @@ -7,12 +7,46 @@
>   *
>   */
>
> +#include <efi_loader.h>
>  #include <init.h>
>  #include <spl.h>
>  #include "../common/fdt_ops.h"
>
>  DECLARE_GLOBAL_DATA_PTR;
>
> +struct efi_fw_image fw_images[] = {
> +       {
> +               .image_type_id = AM69_SK_TIBOOT3_IMAGE_GUID,
> +               .fw_name = u"AM69_SK_TIBOOT3",
> +               .image_index = 1,
> +       },
> +       {
> +               .image_type_id = AM69_SK_SPL_IMAGE_GUID,
> +               .fw_name = u"AM69_SK_SPL",
> +               .image_index = 2,
> +       },
> +       {
> +               .image_type_id = AM69_SK_UBOOT_IMAGE_GUID,
> +               .fw_name = u"AM69_SK_UBOOT",
> +               .image_index = 3,
> +       }
> +};
> +
> +struct efi_capsule_update_info update_info = {
> +       .dfu_string = "sf 0:0=tiboot3.bin raw 0 80000;"
> +       "tispl.bin raw 80000 200000;u-boot.img raw 280000 400000",
> +       .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/j784s4_evm.h b/include/configs/j784s4_evm.h
> index 4a2ccdce144..53eb2bff221 100644
> --- a/include/configs/j784s4_evm.h
> +++ b/include/configs/j784s4_evm.h
> @@ -9,6 +9,30 @@
>  #ifndef __CONFIG_J784S4_EVM_H
>  #define __CONFIG_J784S4_EVM_H
>
> +/**
> + * define AM69_SK_TIBOOT3_IMAGE_GUID - firmware GUID for AM69 SK tiboot3.bin
> + * define AM69_SK_SPL_IMAGE_GUID     - firmware GUID for AM69 SK SPL
> + * define AM69_SK_UBOOT_IMAGE_GUID   - firmware GUID for AM69 SK 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 AM69_SK_TIBOOT3_IMAGE_GUID \
> +       EFI_GUID(0xadf49ec5, 0x61bb, 0x4dbe, 0x8b, 0x8d,        \
> +                0x39, 0xdf, 0x4d, 0x7e, 0xbf, 0x46)
> +
> +#define AM69_SK_SPL_IMAGE_GUID \
> +       EFI_GUID(0x787f0059, 0x63a1, 0x461c, 0xa1, 0x8e, \
> +                0x9d, 0x83, 0x83, 0x45, 0xfe, 0x8e)
> +
> +#define AM69_SK_UBOOT_IMAGE_GUID \
> +       EFI_GUID(0x9300505d, 0x6ec5, 0x4ff8, 0x99, 0xe4, \
> +                0x54, 0x59, 0xa0, 0x4b, 0xe6, 0x17)
> +
>  /* Now for the remaining common defines */
>  #include <configs/ti_armv7_common.h>
>
> --
> 2.34.1
>

Acked-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
diff mbox series

Patch

diff --git a/board/ti/j784s4/evm.c b/board/ti/j784s4/evm.c
index aed0ea5b949..548dbd5925d 100644
--- a/board/ti/j784s4/evm.c
+++ b/board/ti/j784s4/evm.c
@@ -7,12 +7,46 @@ 
  *
  */
 
+#include <efi_loader.h>
 #include <init.h>
 #include <spl.h>
 #include "../common/fdt_ops.h"
 
 DECLARE_GLOBAL_DATA_PTR;
 
+struct efi_fw_image fw_images[] = {
+	{
+		.image_type_id = AM69_SK_TIBOOT3_IMAGE_GUID,
+		.fw_name = u"AM69_SK_TIBOOT3",
+		.image_index = 1,
+	},
+	{
+		.image_type_id = AM69_SK_SPL_IMAGE_GUID,
+		.fw_name = u"AM69_SK_SPL",
+		.image_index = 2,
+	},
+	{
+		.image_type_id = AM69_SK_UBOOT_IMAGE_GUID,
+		.fw_name = u"AM69_SK_UBOOT",
+		.image_index = 3,
+	}
+};
+
+struct efi_capsule_update_info update_info = {
+	.dfu_string = "sf 0:0=tiboot3.bin raw 0 80000;"
+	"tispl.bin raw 80000 200000;u-boot.img raw 280000 400000",
+	.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/j784s4_evm.h b/include/configs/j784s4_evm.h
index 4a2ccdce144..53eb2bff221 100644
--- a/include/configs/j784s4_evm.h
+++ b/include/configs/j784s4_evm.h
@@ -9,6 +9,30 @@ 
 #ifndef __CONFIG_J784S4_EVM_H
 #define __CONFIG_J784S4_EVM_H
 
+/**
+ * define AM69_SK_TIBOOT3_IMAGE_GUID - firmware GUID for AM69 SK tiboot3.bin
+ * define AM69_SK_SPL_IMAGE_GUID     - firmware GUID for AM69 SK SPL
+ * define AM69_SK_UBOOT_IMAGE_GUID   - firmware GUID for AM69 SK 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 AM69_SK_TIBOOT3_IMAGE_GUID \
+	EFI_GUID(0xadf49ec5, 0x61bb, 0x4dbe, 0x8b, 0x8d,	\
+		 0x39, 0xdf, 0x4d, 0x7e, 0xbf, 0x46)
+
+#define AM69_SK_SPL_IMAGE_GUID \
+	EFI_GUID(0x787f0059, 0x63a1, 0x461c, 0xa1, 0x8e, \
+		 0x9d, 0x83, 0x83, 0x45, 0xfe, 0x8e)
+
+#define AM69_SK_UBOOT_IMAGE_GUID \
+	EFI_GUID(0x9300505d, 0x6ec5, 0x4ff8, 0x99, 0xe4, \
+		 0x54, 0x59, 0xa0, 0x4b, 0xe6, 0x17)
+
 /* Now for the remaining common defines */
 #include <configs/ti_armv7_common.h>