Re: [PATCH] fastboot: read fastboot buffer address from fastboot_addr_r

[Mattijs Korpershoek <mkorpershoek@kernel.org>]

Hi Łukasz,

Thank you for the patch.

Somehow, I was not in copy of this email so I could have missed it.
Please make sure to run get_maintainer.pl next time you submit something.

$ ./scripts/get_maintainer.pl -f drivers/fastboot/

See:
https://docs.u-boot.org/en/latest/CONTRIBUTE.html#contributions

On Thu, Apr 30, 2026 at 22:14, Łukasz Stelmach <l.stelmach@samsung.com> wrote:

> Following the previous advice (link below) check fastbooot_addr_r

fastbooot -> fastboot

> environment variable for fastboot buffer address.
>
> Link: https://lore.kernel.org/u-boot/aWiWFrdwvy7-IK63@sumit-xelite/
> Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
> ---
>  drivers/fastboot/fb_common.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/fastboot/fb_common.c b/drivers/fastboot/fb_common.c
> index 68f92c4b887..4579f30b52e 100644
> --- a/drivers/fastboot/fb_common.c
> +++ b/drivers/fastboot/fb_common.c
> @@ -226,7 +226,9 @@ void fastboot_set_progress_callback(void (*progress)(const char *msg))
>   */
>  void fastboot_init(void *buf_addr, u32 buf_size)
>  {
> -	fastboot_buf_addr = buf_addr ? buf_addr :
> +	void *_buf_addr = (void *)env_get_hex("fastboot_addr_r", 0);
> +
> +	fastboot_buf_addr = buf_addr ? buf_addr : _buf_addr ? _buf_addr :
>  				       (void *)CONFIG_FASTBOOT_BUF_ADDR;

Please also have a look at Quentin's suggestion:
https://lore.kernel.org/all/453e3e7f-e975-42a6-9f44-5aaa9a5fd03f@cherry.de/

>  	fastboot_buf_size = buf_size ? buf_size : CONFIG_FASTBOOT_BUF_SIZE;
>  	fastboot_set_progress_callback(NULL);
> -- 
> 2.47.3