[PATCH v2] cmd: gpt: Add option to write GPT partitions to environment variable

[Heinrich Schuchardt]

On 5/3/21 9:29 PM, Farhan Ali wrote:
> Hi Heinrich,
>
> I was wondering if there are any other changes necessary?

Nothing on my side.

The patch was reviewed by Simon and now is is assigned to Tom.

https://patchwork.ozlabs.org/project/uboot/patch/20210226181733.19307-1-farhan.ali@broadcom.com/

He should be able to tell you why it wasn't merged yet.

Best regards

Heinrich
>
> Thanks
>
> Farhan
>
> On Thu, Mar 4, 2021 at 8:08 PM Simon Glass <sjg at chromium.org
> <mailto:sjg at chromium.org>> wrote:
>
>     On Fri, 26 Feb 2021 at 13:17, Farhan Ali <farhan.ali at broadcom.com
>     <mailto:farhan.ali at broadcom.com>> wrote:
>     >
>     > This change would enhance the existing 'gpt read' command to allow
>     > (optionally) writing of the read GPT partitions to an environment
>     > variable in the UBOOT partitions layout format. This would allow users
>     > to easily change the overall partition settings by editing said
>     variable
>     > and then using the variable in the 'gpt write' and 'gpt verify'
>     commands.
>     >
>     > Signed-off-by: Farhan Ali <farhan.ali at broadcom.com
>     <mailto:farhan.ali at broadcom.com>>
>     > Cc: Simon Glass <sjg at chromium.org <mailto:sjg at chromium.org>>
>     > Cc: Heinrich Schuchardt <xypron.glpk at gmx.de
>     <mailto:xypron.glpk at gmx.de>>
>     > Cc: Corneliu Doban <cdoban at broadcom.com <mailto:cdoban at broadcom.com>>
>     > Cc: Rayagonda Kokatanur <rayagonda.kokatanur at broadcom.com
>     <mailto:rayagonda.kokatanur at broadcom.com>>
>     > Cc: Rasmus Villemoes <rasmus.villemoes at prevas.dk
>     <mailto:rasmus.villemoes at prevas.dk>>
>     >
>     > ---
>     > Changes for v2:
>     >    - Checked for argv[4] existence before calling do_get_gpt_info
>     >    - Added missing update to doc/README.gpt
>     > ---
>     >  cmd/gpt.c      | 46 ++++++++++++++++++++++++++++++++++++++--------
>     >  doc/README.gpt | 17 +++++++++++++++++
>     >  2 files changed, 55 insertions(+), 8 deletions(-)
>     >
>
>     Reviewed-by: Simon Glass <sjg at chromium.org <mailto:sjg at chromium.org>>
>