[PATCH v2] board: freescale: p1_p2_rdb_pc: Add env commands norlowerboot, norupperboot, sd2boot and defboot
Pali Rohár
pali at kernel.org
Sun Jul 3 14:38:36 CEST 2022
PING!
On Thursday 23 June 2022 15:43:45 Pali Rohár wrote:
> On Thursday 16 June 2022 17:01:50 Peng Fan (OSS) wrote:
> > 在 2022/4/25 22:50, Pali Rohár 写道:
> > > All *boot env commands overrides default boot source location via i2c.
> > > After board reset without power off, BootROM then starts booting U-Boot
> > > from this specified location instead of the default one.
> > >
> > > Add new env command defboot which reverts boot location to the default
> > > value, which in most cases is configurable by HW DIP switches.
> > >
> > > And add new env commands norlowerboot, norupperboot, sd2boot to boot from
> > > other locations. norlowerboot would instruct BootROM to boot from lower NOR
> > > bank, norupperboot from upper NOR bank and sd2boot from SD card with
> > > alternative configuration.
> > >
> > > Signed-off-by: Pali Rohár <pali at kernel.org>
> >
> > Not able to apply this patch.
>
> And what is the reason? It applies cleanly on top of next branch, which
> is today at commit 9121478ee6f2aee381f8fe49d8997d43527d351a.
>
> See my steps and output:
>
> $ git clone https://source.denx.de/u-boot/u-boot.git -b next
> Cloning into 'u-boot'...
> remote: Enumerating objects: 861911, done.
> remote: Counting objects: 100% (15749/15749), done.
> remote: Compressing objects: 100% (3457/3457), done.
> remote: Total 861911 (delta 12303), reused 15650 (delta 12234), pack-reused 846162
> Receiving objects: 100% (861911/861911), 171.30 MiB | 38.97 MiB/s, done.
> Resolving deltas: 100% (721111/721111), done.
>
> $ cd u-boot
>
> $ wget https://patchwork.ozlabs.org/project/uboot/patch/20220425145043.9945-1-pali@kernel.org/mbox/ -O patch
> --2022-06-23 15:39:52-- https://patchwork.ozlabs.org/project/uboot/patch/20220425145043.9945-1-pali@kernel.org/mbox/
> Resolving patchwork.ozlabs.org (patchwork.ozlabs.org)... 2401:3900:2:1::2, 203.11.71.1
> Connecting to patchwork.ozlabs.org (patchwork.ozlabs.org)|2401:3900:2:1::2|:443... connected.
> HTTP request sent, awaiting response... 200 OK
> Length: 8916 (8.7K) [text/plain]
> Saving to: 'patch'
>
> patch 100%[===============================>] 8.71K --.-KB/s in 0s
>
> 2022-06-23 15:39:53 (57.2 MB/s) - 'patch' saved [8916/8916]
>
> $ git am patch
> Applying: board: freescale: p1_p2_rdb_pc: Add env commands norlowerboot, norupperboot, sd2boot and defboot
>
> $ git log --oneline | head -4
> b94d5df9eb49 board: freescale: p1_p2_rdb_pc: Add env commands norlowerboot, norupperboot, sd2boot and defboot
> 9121478ee6f2 Merge branch '2022-06-22-platform-updates-and-additions' into next
> 929e581a620f corstone1000: Convert to text file environment
> 781a144a7a7e gxp: Convert to text file environment
>
> > Please new version.
Could you please say something what you want in new version?
> > Thanks,
> > Peng.
> > > ---
> > > Changes in v2:
> > > * Fix commit message
> > > * Adapt code to use p1_p2_bootsrc.h
> > > ---
> > > include/configs/p1_p2_bootsrc.h | 20 ++++++++++++++++++++
> > > include/configs/p1_p2_rdb_pc.h | 13 +++++++++++++
> > > 2 files changed, 33 insertions(+)
> > >
> > > diff --git a/include/configs/p1_p2_bootsrc.h b/include/configs/p1_p2_bootsrc.h
> > > index a274c57786f5..60741ef544c0 100644
> > > --- a/include/configs/p1_p2_bootsrc.h
> > > +++ b/include/configs/p1_p2_bootsrc.h
> > > @@ -30,6 +30,18 @@
> > > #define RST_NOR_CMD(var, ...) ""
> > > #endif
> > > +#ifdef __SW_BOOT_NOR_BANK_LO
> > > +#define RST_NOR_LO_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(__SW_BOOT_NOR_BANK_LO, __SW_BOOT_MASK))
> > > +#else
> > > +#define RST_NOR_LO_CMD(var, ...) ""
> > > +#endif
> > > +
> > > +#ifdef __SW_BOOT_NOR_BANK_UP
> > > +#define RST_NOR_UP_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(__SW_BOOT_NOR_BANK_UP, __SW_BOOT_MASK))
> > > +#else
> > > +#define RST_NOR_UP_CMD(var, ...) ""
> > > +#endif
> > > +
> > > #ifdef __SW_BOOT_SPI
> > > #define RST_SPI_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(__SW_BOOT_SPI, __SW_BOOT_MASK))
> > > #else
> > > @@ -42,6 +54,12 @@
> > > #define RST_SD_CMD(var, ...) ""
> > > #endif
> > > +#ifdef __SW_BOOT_SD2
> > > +#define RST_SD2_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(__SW_BOOT_SD2, __SW_BOOT_MASK))
> > > +#else
> > > +#define RST_SD2_CMD(var, ...) ""
> > > +#endif
> > > +
> > > #ifdef __SW_BOOT_NAND
> > > #define RST_NAND_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(__SW_BOOT_NAND, __SW_BOOT_MASK))
> > > #else
> > > @@ -53,3 +71,5 @@
> > > #else
> > > #define RST_PCIE_CMD(var, ...) ""
> > > #endif
> > > +
> > > +#define RST_DEF_CMD(var, ...) __VAR_CMD_RST(var, __VA_ARGS__ __BOOTSRC_CMD(0x00, 0xff))
> > > diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h
> > > index 47bd20eeeafb..50ce2d9aaed4 100644
> > > --- a/include/configs/p1_p2_rdb_pc.h
> > > +++ b/include/configs/p1_p2_rdb_pc.h
> > > @@ -25,6 +25,9 @@
> > > #define __SW_NOR_BANK_MASK 0xfd
> > > #define __SW_NOR_BANK_UP 0x00
> > > #define __SW_NOR_BANK_LO 0x02
> > > +#define __SW_BOOT_NOR_BANK_UP 0x5c /* (__SW_BOOT_NOR | __SW_NOR_BANK_UP) */
> > > +#define __SW_BOOT_NOR_BANK_LO 0x5e /* (__SW_BOOT_NOR | __SW_NOR_BANK_LO) */
> > > +#define __SW_BOOT_NOR_BANK_MASK 0x01 /* (__SW_BOOT_MASK & __SW_NOR_BANK_MASK) */
> > > #define CONFIG_SYS_L2_SIZE (256 << 10)
> > > #endif
> > > @@ -54,6 +57,9 @@
> > > #define __SW_NOR_BANK_MASK 0xfd
> > > #define __SW_NOR_BANK_UP 0x00
> > > #define __SW_NOR_BANK_LO 0x02
> > > +#define __SW_BOOT_NOR_BANK_UP 0x64 /* (__SW_BOOT_NOR | __SW_NOR_BANK_UP) */
> > > +#define __SW_BOOT_NOR_BANK_LO 0x66 /* (__SW_BOOT_NOR | __SW_NOR_BANK_LO) */
> > > +#define __SW_BOOT_NOR_BANK_MASK 0x01 /* (__SW_BOOT_MASK & __SW_NOR_BANK_MASK) */
> > > #define CONFIG_SYS_L2_SIZE (256 << 10)
> > > /*
> > > * Dynamic MTD Partition support with mtdparts
> > > @@ -73,6 +79,9 @@
> > > #define __SW_NOR_BANK_MASK 0xfd
> > > #define __SW_NOR_BANK_UP 0x00
> > > #define __SW_NOR_BANK_LO 0x02
> > > +#define __SW_BOOT_NOR_BANK_UP 0xc8 /* (__SW_BOOT_NOR | __SW_NOR_BANK_UP) */
> > > +#define __SW_BOOT_NOR_BANK_LO 0xca /* (__SW_BOOT_NOR | __SW_NOR_BANK_LO) */
> > > +#define __SW_BOOT_NOR_BANK_MASK 0x01 /* (__SW_BOOT_MASK & __SW_NOR_BANK_MASK) */
> > > #define CONFIG_SYS_L2_SIZE (512 << 10)
> > > /*
> > > * Dynamic MTD Partition support with mtdparts
> > > @@ -605,10 +614,14 @@ __VSCFW_ADDR \
> > > MAP_NOR_LO_CMD(map_lowernorbank) \
> > > MAP_NOR_UP_CMD(map_uppernorbank) \
> > > RST_NOR_CMD(norboot) \
> > > +RST_NOR_LO_CMD(norlowerboot) \
> > > +RST_NOR_UP_CMD(norupperboot) \
> > > RST_SPI_CMD(spiboot) \
> > > RST_SD_CMD(sdboot) \
> > > +RST_SD2_CMD(sd2boot) \
> > > RST_NAND_CMD(nandboot) \
> > > RST_PCIE_CMD(pciboot) \
> > > +RST_DEF_CMD(defboot) \
> > > ""
> > > #define CONFIG_USB_FAT_BOOT \
> >
More information about the U-Boot
mailing list