On 7/25/19 9:22 AM, Sam Protsenko wrote:
"fastboot flashall" expects "fastboot getvar product" to be "beagle_x15board". Instead, "am57xx" is returned, as it's set in $board env var from SYS_BOARD in board/ti/am57xx/Kconfig file.
Override fastboot product variable and set it to correct value, to fix "fastboot flashall".
Signed-off-by: Sam Protsenko semen.protsenko@linaro.org
board/ti/am57xx/board.c | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/board/ti/am57xx/board.c b/board/ti/am57xx/board.c index 1a903f13a6..c8eac4edde 100644 --- a/board/ti/am57xx/board.c +++ b/board/ti/am57xx/board.c @@ -685,6 +685,11 @@ static int device_okay(const char *path) } #endif
+static void am57x_set_fastboot_vars(void) +{
- env_set("fastboot.product", "beagle_x15board");
This doesn't seem right.. This is common source for all AM57x based boards, the only thing we can return here is "am57xx". Either fastboot needs some sort of conversion on its side, or we set the exact board name the same way we do device-tree name detection.
Andrew
+}
int board_late_init(void) { setup_board_eeprom_env(); @@ -717,6 +722,7 @@ int board_late_init(void)
omap_die_id_serial(); omap_set_fastboot_vars();
am57x_set_fastboot_vars();
am57x_idk_lcd_detect();