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-farh...

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@chromium.org mailto:sjg@chromium.org> wrote:

On Fri, 26 Feb 2021 at 13:17, Farhan Ali <farhan.ali@broadcom.com
<mailto:farhan.ali@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@broadcom.com
<mailto:farhan.ali@broadcom.com>>
> Cc: Simon Glass <sjg@chromium.org <mailto:sjg@chromium.org>>
> Cc: Heinrich Schuchardt <xypron.glpk@gmx.de
<mailto:xypron.glpk@gmx.de>>
> Cc: Corneliu Doban <cdoban@broadcom.com <mailto:cdoban@broadcom.com>>
> Cc: Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com
<mailto:rayagonda.kokatanur@broadcom.com>>
> Cc: Rasmus Villemoes <rasmus.villemoes@prevas.dk
<mailto:rasmus.villemoes@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@chromium.org <mailto:sjg@chromium.org>>