Re: [PATCH 0/3] use environment partitions in GPT

[Marco Felsch <m.felsch@pengutronix.de>]

On 24-02-19, Sascha Hauer wrote:
> We have specified a GUID for GPTs to be used as barebox environment
> partition, but so far we haven't made any use of that. This series
> changes that. We now start using environment partitions when we find
> them, provided the board code hasn't specified any other partition.
> 
> This likely doesn't change much for existing boards because most of
> them actually have an environment partition specified. For new boards
> however it becomes feasible to not register a partition explicitly.
> 
> Something that has hold me back from doing this earlier was that barebox
> was not able to create an environment partition itself, so barebox
> couldn't persist the environment on an unpartitioned device. Now with
> parted support this has changed, so I think it's worth it to give it a
> try now.
> 
> Sascha

LGTM, feel free to add my:

Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>

> 
> Sascha Hauer (3):
>   partitions: efi: Allow to create barebox environment partition
>   bootsource: add function to get device_node we booted from
>   environment: use barebox environment from GPT partitions
> 
>  commands/parted.c       |  2 +-
>  common/bootsource.c     | 14 +++++++++
>  common/environment.c    | 65 ++++++++++++++++++++++++++++++++++++++++-
>  common/oftree.c         |  7 +----
>  common/partitions/efi.c |  3 ++
>  include/bootsource.h    |  1 +
>  include/efi/partition.h |  3 ++
>  7 files changed, 87 insertions(+), 8 deletions(-)
> 
> -- 
> 2.39.2
> 
> 
>