[PATCH 1/2] doc: imx: psb: Fix PERSIST_SECONDARY_BOOT bit location in GPR10

The PERSIST_SECONDARY_BOOT is in GPR10 address 0x30390098, adjust the text which currently says it is in GPR0 while using the correct address of GPR10.
Signed-off-by: Marek Vasut marex@denx.de Cc: Marcel Ziswiler marcel.ziswiler@toradex.com Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com --- doc/imx/misc/psb.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/imx/misc/psb.rst b/doc/imx/misc/psb.rst index 71ac09fac8d..3533c916405 100644 --- a/doc/imx/misc/psb.rst +++ b/doc/imx/misc/psb.rst @@ -159,7 +159,7 @@ WARM reset into B-copy using WDT ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
To perform a reboot into B-copy, the PERSIST_SECONDARY_BOOT must be set -in SRC_GPR0 register. Example on iMX8MM:: +in SRC_GPR10 register. Example on iMX8MM::
=> mw 0x30390098 0x40000000

On iMX8MM with Fast Boot fuse blown, the SIT and A-copy image are placed at different offset than on iMX8MM with Fast Boot fuse NOT blown. List both options and both offsets to avoid confusion.
Signed-off-by: Marek Vasut marex@denx.de Cc: Marcel Ziswiler marcel.ziswiler@toradex.com Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com --- doc/imx/misc/psb.rst | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/doc/imx/misc/psb.rst b/doc/imx/misc/psb.rst index 3533c916405..2eccdcfccd9 100644 --- a/doc/imx/misc/psb.rst +++ b/doc/imx/misc/psb.rst @@ -75,13 +75,15 @@ offset 0x1 means 512 Bytes from the start of SD/eMMC card data partition). For details on the addition of two numbers in recommended B-copy offset, see SIT format below.
-+----------+--------------------+-----------------------+-----------------------------+ -| SoC | SIT offset (fixed) | A-copy offset (fixed) | B-copy offset (recommended) | -+----------+--------------------+-----------------------+-----------------------------+ -| iMX7D | 0x1 | 0x2 | 0x800+0x2 | -+----------+--------------------+-----------------------+-----------------------------+ -| iMX8MM | 0x41 | 0x42 | 0x1000+0x42 | -+----------+--------------------+-----------------------+-----------------------------+ ++----------+-----------------------------+--------------------+-----------------------+-----------------------------+ +| SoC | Boot Device Type | SIT offset (fixed) | A-copy offset (fixed) | B-copy offset (recommended) | ++----------+-----------------------------+--------------------+-----------------------+-----------------------------+ +| iMX7D | | 0x1 | 0x2 | 0x800+0x2 | ++----------+-----------------------------+--------------------+-----------------------+-----------------------------+ +| iMX8MM | SD/eSD/MMC/eMMC normal boot | 0x41 | 0x42 | 0x1000+0x42 | ++----------+-----------------------------+--------------------+-----------------------+-----------------------------+ +| iMX8MM | eMMC Fast boot fuse blown | 0x1 | 0x2 | 0x1000+0x2 | ++----------+-----------------------------+--------------------+-----------------------+-----------------------------+
SIT format ~~~~~~~~~~

On Tue, 2021-08-10 at 21:12 +0200, Marek Vasut wrote:
On iMX8MM with Fast Boot fuse blown, the SIT and A-copy image are placed at different offset than on iMX8MM with Fast Boot fuse NOT blown. List both options and both offsets to avoid confusion.
Signed-off-by: Marek Vasut marex@denx.de
Reviewed-by: Marcel Ziswiler marcel.ziswiler@toradex.com
Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com
doc/imx/misc/psb.rst | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/doc/imx/misc/psb.rst b/doc/imx/misc/psb.rst index 3533c916405..2eccdcfccd9 100644 --- a/doc/imx/misc/psb.rst +++ b/doc/imx/misc/psb.rst @@ -75,13 +75,15 @@ offset 0x1 means 512 Bytes from the start of SD/eMMC card data partition). For details on the addition of two numbers in recommended B-copy offset, see SIT format below. -+----------+--------------------+-----------------------+-----------------------------+ -| SoC | SIT offset (fixed) | A-copy offset (fixed) | B-copy offset (recommended) | -+----------+--------------------+-----------------------+-----------------------------+ -| iMX7D | 0x1 | 0x2 | 0x800+0x2 | -+----------+--------------------+-----------------------+-----------------------------+ -| iMX8MM | 0x41 | 0x42 | 0x1000+0x42 | -+----------+--------------------+-----------------------+-----------------------------+ ++----------+-----------------------------+--------------------+-----------------------+--------------------- --------+ +| SoC | Boot Device Type | SIT offset (fixed) | A-copy offset (fixed) | B-copy offset (recommended) | ++----------+-----------------------------+--------------------+-----------------------+--------------------- --------+ +| iMX7D | | 0x1 | 0x2 | 0x800+0x2 | ++----------+-----------------------------+--------------------+-----------------------+--------------------- --------+ +| iMX8MM | SD/eSD/MMC/eMMC normal boot | 0x41 | 0x42 | 0x1000+0x42 | ++----------+-----------------------------+--------------------+-----------------------+--------------------- --------+ +| iMX8MM | eMMC Fast boot fuse blown | 0x1 | 0x2 | 0x1000+0x2 | ++----------+-----------------------------+--------------------+-----------------------+--------------------- --------+ SIT format ~~~~~~~~~~

On iMX8MM with Fast Boot fuse blown, the SIT and A-copy image are placed at different offset than on iMX8MM with Fast Boot fuse NOT blown. List both options and both offsets to avoid confusion. Signed-off-by: Marek Vasut marex@denx.de Cc: Marcel Ziswiler marcel.ziswiler@toradex.com Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com Reviewed-by: Marcel Ziswiler marcel.ziswiler@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic

On Tue, 2021-08-10 at 21:12 +0200, Marek Vasut wrote:
The PERSIST_SECONDARY_BOOT is in GPR10 address 0x30390098, adjust the text which currently says it is in GPR0 while using the correct address of GPR10.
Signed-off-by: Marek Vasut marex@denx.de
Reviewed-by: Marcel Ziswiler marcel.ziswiler@toradex.com
Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com
doc/imx/misc/psb.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/imx/misc/psb.rst b/doc/imx/misc/psb.rst index 71ac09fac8d..3533c916405 100644 --- a/doc/imx/misc/psb.rst +++ b/doc/imx/misc/psb.rst @@ -159,7 +159,7 @@ WARM reset into B-copy using WDT ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ To perform a reboot into B-copy, the PERSIST_SECONDARY_BOOT must be set -in SRC_GPR0 register. Example on iMX8MM:: +in SRC_GPR10 register. Example on iMX8MM:: => mw 0x30390098 0x40000000

The PERSIST_SECONDARY_BOOT is in GPR10 address 0x30390098, adjust the text which currently says it is in GPR0 while using the correct address of GPR10. Signed-off-by: Marek Vasut marex@denx.de Cc: Marcel Ziswiler marcel.ziswiler@toradex.com Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de Cc: Ye Li ye.li@nxp.com Cc: uboot-imx uboot-imx@nxp.com Reviewed-by: Marcel Ziswiler marcel.ziswiler@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
participants (3)
-
Marcel Ziswiler
-
Marek Vasut
-
sbabic@denx.de