Hi Neil, Neil Armstrong <narmstr...@baylibre.com> writes:
> On 30/07/2021 10:55, Mattijs Korpershoek wrote: >> The SEI-610 and SEI-510 boards are well supported in the >> Android Open Source project via the yukawa [1] platform. >> >> Their U-Boot version, despite being public [2] is not in mainline. >> >> Android has multiple OSes it can boot into: >> * Main android >> * Recovery OS >> * Fastbootd (recovery OS with fastboot mode) >> * Bootloader (U-Boot) with fastboot mode >> >> The decision of which OS to boot into is taken by the bootloader based on >> the "reboot reason". >> >> For amlogic boards, we pass the reboot reason via a custom smc call. >> This support is limited: we don't r >> >> These series reworks the reboot reason handling to use the BCB, which >> is the standard Android way. >> >> [1] https://android.googlesource.com/device/amlogic/yukawa >> [2] >> https://gitlab.com/baylibre/amlogic/atv/u-boot/-/tree/u-boot/v2021.07/integ >> >> Mattijs Korpershoek (5): >> Revert "ARM: meson: Add support for fastboot_set_reboot_flag()" >> configs: meson64_android: use BCB for reboot to bootloader >> configs: meson64_android: implement recovery boot via BCB >> configs: meson64_android: reboot to fastbootd from bootloader >> configs: sei610/sei510: set CONFIG_CMD_BCB=y >> >> arch/arm/mach-meson/board-common.c | 34 ----------------- >> configs/sei510_defconfig | 1 + >> configs/sei610_defconfig | 1 + >> include/configs/meson64_android.h | 60 ++++++++++++++++++++---------- >> 4 files changed, 42 insertions(+), 54 deletions(-) >> > > Looks fine, but I'll move the patch 1 as last. Thanks for your review. > > Can you re-post ? > > Please add: > Acked-by: Neil Armstrong <narmstr...@baylibre.com> > > to all patches if you repost. I've reposted a v2 with your ack. Mattijs > > Neil