[PATCH v2 0/7] Refactor generic fastboot_set_reboot_flag implementation

Roman Kovalivskyi roman.kovalivskyi at globallogic.com
Tue Jan 26 21:54:49 CET 2021


Current generic implementation of fastboot_set_reboot_flag is somewhat
messy and requires some additional configuration option to be enabled
besides CMD_BCB, so it reverts that implementtion in order to bring a
new cleaner one.

New function called bcb_set_reboot_reason should be exposed by BCB
commands, so that all of the details could be put there instead of
dealing with it in fastboot implementation directly. After that,
fastboot_set_reboot_flag could simply pass correct reboot reason
string to the BCB implementation. If CMD_BCB is disabled then the
whole operation would return error code, which is no different
behaviour than the current implementation.

Changes since v1[1]
* Fixed compilation issue for sandbox platform caused by undefined
  FASTBOOT_FLASH_MMC_DEV Kconfig option.
* Fixed compilation issue in bcb.h caused by missing common.h header
  for sandbox platform

[1]: https://patchwork.ozlabs.org/project/uboot/cover/cover.1603442753.git.roman.kovalivskyi@globallogic.com/

Eugeniu Rosca (5):
  cmd: bcb: Extract '__bcb_load' from 'do_bcb_load' for internal needs
  cmd: bcb: Extract '__bcb_set' from 'do_bcb_set' for internal needs
  cmd: bcb: Extract '__bcb_store' from 'do_bcb_store' for internal needs
  cmd: bcb: Expose 'bcb_write_reboot_reason' to external callers
  cmd: bcb: Add support for processing const string literals in
    bcb_set()

Roman Kovalivskyi (2):
  Revert "fastboot: Add default fastboot_set_reboot_flag implementation"
  fastboot: Implement generic fastboot_set_reboot_flag

 cmd/bcb.c                      | 88 +++++++++++++++++++++++++++-------
 drivers/fastboot/Kconfig       | 12 -----
 drivers/fastboot/Makefile      |  1 -
 drivers/fastboot/fb_bcb_impl.c | 43 -----------------
 drivers/fastboot/fb_common.c   | 16 ++++++-
 include/bcb.h                  | 22 +++++++++
 include/fastboot.h             |  9 ----
 7 files changed, 107 insertions(+), 84 deletions(-)
 delete mode 100644 drivers/fastboot/fb_bcb_impl.c
 create mode 100644 include/bcb.h

-- 
2.17.1



More information about the U-Boot mailing list