On 1/19/22 19:55, Sughosh Ganu wrote:
The FWU Multi Bank Update feature allows the platform to boot the
firmware images from one of the partitions(banks). The first stage
bootloader(fsbl) passes the value of the boot index, i.e. the bank
from which the firmware images were booted from to U-Boot. On the
STM32MP157C-DK2 board, this value is passed through one of the SoC's
backup register. Add a function to read the boot index value from the
backup register.

Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
---

Changes since V2: None

  board/st/stm32mp1/stm32mp1.c | 7 +++++++
  include/fwu.h                | 1 +
  2 files changed, 8 insertions(+)

diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c
index 66cbe3f798..6e7aaaa8a4 100644
--- a/board/st/stm32mp1/stm32mp1.c
+++ b/board/st/stm32mp1/stm32mp1.c
@@ -1104,6 +1104,13 @@ int fwu_plat_get_blk_desc(struct blk_desc **desc)
        return 0;
  }

+void fwu_plat_get_bootidx(void *boot_idx)
+{
+       u32 *bootidx = boot_idx;
+
+       *bootidx = readl(TAMP_BOOTCOUNT);
+}
+

Please, follow the driver model in your implementation.

https://u-boot.readthedocs.io/en/latest/develop/driver-model/index.html

Best regards

Heinrich

  struct fwu_mdata_ops *get_plat_fwu_mdata_ops(void)
  {
        if (CONFIG_IS_ENABLED(TARGET_ST_STM32MP15x) &&
diff --git a/include/fwu.h b/include/fwu.h
index b23a93ac40..6393a1dbb5 100644
--- a/include/fwu.h
+++ b/include/fwu.h
@@ -63,5 +63,6 @@ int fwu_plat_get_update_index(u32 *update_idx);
  int fwu_plat_get_blk_desc(struct blk_desc **desc);
  int fwu_plat_get_alt_num(void *identifier);
  int fwu_plat_fill_partition_guids(efi_guid_t **part_guid_arr);
+void fwu_plat_get_bootidx(void *boot_idx);

  #endif /* _FWU_H_ */

Reply via email to