[PATCH master] fastboot: populate is-logical:$partition variable

[Uwe Kleine-König]

Hello Ahmad,

On Tue, Jul 22, 2025 at 04:05:34PM +0200, Ahmad Fatoum wrote:
> Uwe reports on IRC that fastboot tries to read some default variables
> that barebox doesn't define, which triggers host-side error output:
> 
>   ERROR: fastboot: no such variable: is-logical:kernel
> 
> Flashing didn't seem impacted though.
> 
> Referring to the documentation[1], is-logical is related to a number of
> commands that create/delete/resize logical partitions.
> 
> We have support for modifying partition tables in barebox, but we don't expose
> it over Fastboot. Until we do, let's report all partitions as physical
> partitions, which is the default value.
> 
> This issue has always existed, but only became apparent once we started
> returning an error for non-existent variables.
> 
> [1]: https://android.googlesource.com/platform/system/core/+show/refs/heads/main/fastboot/README.md#237
> 
> Fixes: dd377c937f8b ("common: fastboot: send FAIL if variable does not exist")
> Fixes: 3d5080aae90d ("USB: gadget: Add Android fastboot support")
> Reported-by: Uwe Kleine-König <uwe at kleine-koenig.org>
> Cc: Andrei Lalaev <andrey.lalaev at gmail.com>
> Signed-off-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
> ---
> @Uwe, can you give this a test?

I did, and with it the error messages I wailed about are gone.

Thanks, 🌟🌟🌟🌟🌟
Uwe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/barebox/attachments/20250722/e46e9441/attachment.sig>