RE: [PATCH v2] mmc: Update environment variable with active partition

2021-05-12 Thread Reuben Dowle
Hi Jaehoon, > is mandatory, and [ ] is > optional. > If optional variable will be used, those variables can be mandatory, right? > > mmc partconf is displayed even without other variables. > > "mmc partconf [[varname] | [ > ]]" is correct usage for yours. > > It means varname or other op

Re: [PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Jaehoon Chung
Hi Reuben, On 5/10/21 1:38 PM, Reuben Dowle wrote: > Hi Jaehoon, > >> >> boot_ack / boot_partition / partition_access are optional variable, not >> mandatory. >> And it's confused that different variable is used with same command. >> >> I think that it's enough to use "mmc partconf ". Then it's s

RE: [PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Reuben Dowle
Hi Jaehoon, > > boot_ack / boot_partition / partition_access are optional variable, not > mandatory. > And it's confused that different variable is used with same command. > > I think that it's enough to use "mmc partconf ". Then it's showed which > partition is used. > > If you have to apply t

Re: [PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Jaehoon Chung
Hi Reuben, On 5/10/21 9:14 AM, Reuben Dowle wrote: > This patch allows uboot scripts make choices about where to boot from based > on the active mmc boot partition. This allows having two copies of kernel, > filesystems etc, and choosing which to boot from based off of the active > bootloader part

[PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Reuben Dowle
This patch allows uboot scripts make choices about where to boot from based on the active mmc boot partition. This allows having two copies of kernel, filesystems etc, and choosing which to boot from based off of the active bootloader partition. Signed-off-by: Reuben Dowle --- cmd/mmc.c | 18 +++