[U-Boot] [PATCH] fastboot: Add "is-userspace" variable

[Igor Opaniuk]

Hi Sam,

On Thu, Jun 20, 2019 at 4:40 PM Sam Protsenko
<semen.protsenko at linaro.org> wrote:
>
> As per documentation [1], Android-Q requires for bootloader to provide
> "is-userspace" variable. "no" value should be always returned. This
> patch implements this fastboot variable.
>
> [1] https://android.googlesource.com/platform/system/core/+/master/fastboot/README.md#client-variables
>
> Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
> ---
>  drivers/fastboot/fb_getvar.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/drivers/fastboot/fb_getvar.c b/drivers/fastboot/fb_getvar.c
> index 9ee5054485..fd0823b2bf 100644
> --- a/drivers/fastboot/fb_getvar.c
> +++ b/drivers/fastboot/fb_getvar.c
> @@ -28,6 +28,7 @@ static void getvar_partition_type(char *part_name, char *response);
>  #if CONFIG_IS_ENABLED(FASTBOOT_FLASH)
>  static void getvar_partition_size(char *part_name, char *response);
>  #endif
> +static void getvar_is_userspace(char *var_parameter, char *response);
>
>  static const struct {
>         const char *variable;
> @@ -78,6 +79,9 @@ static const struct {
>                 .variable = "partition-size",
>                 .dispatch = getvar_partition_size
>  #endif
> +       }, {
> +               .variable = "is-userspace",
> +               .dispatch = getvar_is_userspace
>         }
>  };
>
> @@ -243,6 +247,11 @@ static void getvar_partition_size(char *part_name, char *response)
>  }
>  #endif
>
> +static void getvar_is_userspace(char *var_parameter, char *response)
> +{
> +       fastboot_okay("no", response);
> +}
> +
>  /**
>   * fastboot_getvar() - Writes variable indicated by cmd_parameter to response.
>   *
> --
> 2.20.1
>

Reviewed-by: Igor Opaniuk <igor.opaniuk at toradex.com>

-- 
Best regards - Freundliche Grüsse - Meilleures salutations

Igor Opaniuk

mailto: igor.opaniuk at gmail.com
skype: igor.opanyuk
+380 (93) 836 40 67
http://ua.linkedin.com/in/iopaniuk