From mboxrd@z Thu Jan  1 00:00:00 1970
From: Reuben Dowle <reuben.dowle@4rf.com>
Date: Thu, 13 May 2021 03:46:33 +0000
Subject: [PATCH v2] mmc: Update environment variable with active partition
In-Reply-To: <3883c6ac-ef2f-1f9c-e0e2-1104cafd14dd@samsung.com>
References: <CGME20210510001459epcas1p21b617abad9c1318127c27e187907f11d@epcas1p2.samsung.com>
 <1620605662-64907-1-git-send-email-reuben.dowle@4rf.com>
 <cc63a279-5008-a438-9c6c-9958d15f4dea@samsung.com>
 <3afe0c7e8f604ffc8d46063d22bfdb50@4rf.com>
 <3883c6ac-ef2f-1f9c-e0e2-1104cafd14dd@samsung.com>
Message-ID: <f497d8dff6134467b333c398cc7e4c48@4rf.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Jaehoon,

> <dev> is mandatory, and [<boot_ack> <boot_partition> <partition_access>] is optional.
> If optional variable will be used, those variables can be mandatory, right?
> 
> mmc partconf <dev> is displayed even without other variables.
> 
> "mmc partconf <dev> [[varname] | [<boot_ack> <boot_partition> <partition_access>]]" is correct usage for yours.
> 
> It means varname or other options variable can be chosen.

I agree, this seems like a better way to format the help. I have sent a new patch with it changed to your suggestion.

Regards,
Reuben