Re: [PATCH] cmd: mmc: update the mmc command's usage about argument

2020-12-14 Thread Simon Glass
Hi Jaehoon, On Mon, 14 Dec 2020 at 21:17, Jaehoon Chung wrote: > > Hi Simon, > > On 12/15/20 1:00 PM, Simon Glass wrote: > > Hi Jaehoon, > > > > On Mon, 14 Dec 2020 at 03:09, Jaehoon Chung wrote: > >> > >> It's confused what is optional or mandantary argument. > > > > how about: It's confusing w

Re: [PATCH] cmd: mmc: update the mmc command's usage about argument

2020-12-14 Thread Jaehoon Chung
Hi Simon, On 12/15/20 1:00 PM, Simon Glass wrote: > Hi Jaehoon, > > On Mon, 14 Dec 2020 at 03:09, Jaehoon Chung wrote: >> >> It's confused what is optional or mandantary argument. > > how about: It's confusing whether arguments are optional or mandatory Okay, will update. > >> Update the com

Re: [PATCH] cmd: mmc: update the mmc command's usage about argument

2020-12-14 Thread Simon Glass
Hi Jaehoon, On Mon, 14 Dec 2020 at 03:09, Jaehoon Chung wrote: > > It's confused what is optional or mandantary argument. how about: It's confusing whether arguments are optional or mandatory > Update the command's usage to clarify how to use. Are you using <> to indicate optional? I see [] us

[PATCH] cmd: mmc: update the mmc command's usage about argument

2020-12-14 Thread Jaehoon Chung
It's confused what is optional or mandantary argument. Update the command's usage to clarify how to use. Signed-off-by: Jaehoon Chung --- cmd/mmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/mmc.c b/cmd/mmc.c index 1529a3e05ddd..cb6b59f36a43 100644 --- a/cmd/mm