[RESEND 0/7] Refactor generic fastboot_set_reboot_flag implementation
Eugeniu Rosca
erosca at de.adit-jv.com
Thu Dec 10 08:30:36 CET 2020
Dear U-Boot maintainers,
On Fri, Oct 23, 2020 at 11:52:18AM +0300, Roman Kovalivskyi wrote:
> 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.
>
> 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 | 12 ++++-
> include/bcb.h | 20 ++++++++
> include/fastboot.h | 9 ----
> 7 files changed, 101 insertions(+), 84 deletions(-)
> delete mode 100644 drivers/fastboot/fb_bcb_impl.c
> create mode 100644 include/bcb.h
Any chance the patches will be applied?
--
Best regards,
Eugeniu Rosca
More information about the U-Boot
mailing list