From mboxrd@z Thu Jan 1 00:00:00 1970 From: Reuben Dowle Date: Mon, 10 May 2021 04:38:52 +0000 Subject: [PATCH v2] mmc: Update environment variable with active partition In-Reply-To: References: <1620605662-64907-1-git-send-email-reuben.dowle@4rf.com> Message-ID: <3afe0c7e8f604ffc8d46063d22bfdb50@4rf.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 this patch, > > It seems that "mmc partconf [varname] [boot_ack boot_partition > partition_access]" is correct way. mmc partconf has two forms currently - one where you set the values and one where you read the values. It would not make sense to write the environment variable when you are writing the MMC values. When setting the values you need 4 parameters (dev + three values to set). In this case all three values are mandatory, and this is not changing. When reading the values you currently just need 1 parameter (dev). This patch adds a second optional parameter to store the current active partition. > > Best Regards, > Jaehoon Chung >