From: Reuben Dowle <reuben.dowle@4rf.com>
To: u-boot@lists.denx.de
Subject: [PATCH v2] mmc: Update environment variable with active partition
Date: Mon, 10 May 2021 04:38:52 +0000 [thread overview]
Message-ID: <3afe0c7e8f604ffc8d46063d22bfdb50@4rf.com> (raw)
In-Reply-To: <cc63a279-5008-a438-9c6c-9958d15f4dea@samsung.com>
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 <dev>". Then it's showed which
> partition is used.
>
> If you have to apply this patch,
>
> It seems that "mmc partconf <dev> [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
>
next prev parent reply other threads:[~2021-05-10 4:38 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20210510001459epcas1p21b617abad9c1318127c27e187907f11d@epcas1p2.samsung.com>
2021-05-10 0:14 ` [PATCH v2] mmc: Update environment variable with active partition Reuben Dowle
2021-05-10 4:30 ` Jaehoon Chung
2021-05-10 4:38 ` Reuben Dowle [this message]
2021-05-10 6:35 ` Jaehoon Chung
2021-05-13 3:46 ` Reuben Dowle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3afe0c7e8f604ffc8d46063d22bfdb50@4rf.com \
--to=reuben.dowle@4rf.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox