| Message ID | a04b5926ffb24020b0d953fa0c06f261@4rf.com |
|---|---|
| State | Changes Requested |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | Expose active mmc boot partition to scripts | expand |
On Tue, Sep 01, 2020 at 10:01:18PM +0000, 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 the active > bootloader partition. > > Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com> > --- > cmd/mmc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/cmd/mmc.c b/cmd/mmc.c > index 1529a3e..88de0c4 100644 > --- a/cmd/mmc.c > +++ b/cmd/mmc.c > @@ -784,6 +784,8 @@ static int mmc_partconf_print(struct mmc *mmc) > ack = EXT_CSD_EXTRACT_BOOT_ACK(mmc->part_config); > part = EXT_CSD_EXTRACT_BOOT_PART(mmc->part_config); > > + env_set_hex("activepart", part); > + > printf("EXT_CSD[179], PARTITION_CONFIG:\n" > "BOOT_ACK: 0x%x\n" > "BOOT_PARTITION_ENABLE: 0x%x\n" Rather than hard-code a variable here I would like to see this work like similar commands that take the variable to place the value in to as an argument, thanks!
diff --git a/cmd/mmc.c b/cmd/mmc.c index 1529a3e..88de0c4 100644 --- a/cmd/mmc.c +++ b/cmd/mmc.c @@ -784,6 +784,8 @@ static int mmc_partconf_print(struct mmc *mmc) ack = EXT_CSD_EXTRACT_BOOT_ACK(mmc->part_config); part = EXT_CSD_EXTRACT_BOOT_PART(mmc->part_config); + env_set_hex("activepart", part); + printf("EXT_CSD[179], PARTITION_CONFIG:\n" "BOOT_ACK: 0x%x\n" "BOOT_PARTITION_ENABLE: 0x%x\n"
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 the active bootloader partition. Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com> --- cmd/mmc.c | 2 ++ 1 file changed, 2 insertions(+)