[PATCH] Fix flash and erase of EMMC_BOOT2 with fastboot

[Sean Anderson]

Hi,

Have you considered trying to flash using e.g.

flashboot flash 0.1 foo.img

instead of

fastboot flash boot0

? I would like to remove these MMC_BOOT2 options but I haven't gotten around to it.

On 5/12/21 6:43 PM, Oleh Kravchenko wrote:
> The current U-Boot version has the next matches for boot partitions:
>> mmc0boot0 to EMMC_BOOT1
>> mmc0boot1 to EMMC_BOOT1 (should be EMMC_BOOT2)
> This patch fixes a typo for the boot partition number.
> 
> Signed-off-by: Oleh Kravchenko <oleg at kaa.org.ua>
> Cc: Pantelis Antoniou <panto at antoniou-consulting.com>
> Cc: Marek Vasut <marex at denx.de>
> ---
> 
>   drivers/fastboot/fb_mmc.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/fastboot/fb_mmc.c b/drivers/fastboot/fb_mmc.c
> index 8e74e50e91..1827ce5d12 100644
> --- a/drivers/fastboot/fb_mmc.c
> +++ b/drivers/fastboot/fb_mmc.c
> @@ -525,7 +525,7 @@ void fastboot_mmc_flash_write(const char *cmd, void *download_buffer,
>   	if (strcmp(cmd, CONFIG_FASTBOOT_MMC_BOOT2_NAME) == 0) {
>   		dev_desc = fastboot_mmc_get_dev(response);
>   		if (dev_desc)
> -			fb_mmc_boot_ops(dev_desc, download_buffer, 1,
> +			fb_mmc_boot_ops(dev_desc, download_buffer, 2,
>   					download_bytes, response);
>   		return;
>   	}
> @@ -655,7 +655,7 @@ void fastboot_mmc_erase(const char *cmd, char *response)
>   		/* erase EMMC boot2 */
>   		dev_desc = fastboot_mmc_get_dev(response);
>   		if (dev_desc)
> -			fb_mmc_boot_ops(dev_desc, NULL, 1, 0, response);
> +			fb_mmc_boot_ops(dev_desc, NULL, 2, 0, response);
>   		return;
>   	}
>   #endif
> 

In any case,

Reviewed-by: Sean Anderson <sean.anderson at seco.com>