[PATCH u-boot-mvebu] tools: kwboot: Document information about NOR XIP

[PATCH u-boot-mvebu] tools: kwboot: Document information about NOR XIP

[Pali Rohár]

Signed-off-by: Pali Rohár <pali@kernel.org>
---
 tools/kwboot.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tools/kwboot.c b/tools/kwboot.c
index 2b92966919da..db917708a8e8 100644
--- a/tools/kwboot.c
+++ b/tools/kwboot.c
@@ -61,7 +61,9 @@
  *   SPI-NOR or parallel-NOR. Despite the type name it really can be stored on
  *   parallel-NOR and cannot be stored on other SPI devices, like SPI-NAND.
  *   So it should have been named NOR image, not SPI image. This image type
- *   supports XIP - Execute In Place directly from NOR memory.
+ *   supports XIP - Execute In Place directly from NOR memory. Destination
+ *   address of the XIP image is set to 0xFFFFFFFF and execute address to the
+ *   absolute offset in bytes from the beginning of NOR memory.
  *
  * - IBR_HDR_NAND_ID (0x8B):
  *   NAND image can be stored either at any 2 MB aligned offset in the first
-- 
2.20.1

Re: [PATCH u-boot-mvebu] tools: kwboot: Document information about NOR XIP

[Stefan Roese]

On 3/23/23 21:00, Pali Rohár wrote:
> Signed-off-by: Pali Rohár <pali@kernel.org>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
>   tools/kwboot.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/kwboot.c b/tools/kwboot.c
> index 2b92966919da..db917708a8e8 100644
> --- a/tools/kwboot.c
> +++ b/tools/kwboot.c
> @@ -61,7 +61,9 @@
>    *   SPI-NOR or parallel-NOR. Despite the type name it really can be stored on
>    *   parallel-NOR and cannot be stored on other SPI devices, like SPI-NAND.
>    *   So it should have been named NOR image, not SPI image. This image type
> - *   supports XIP - Execute In Place directly from NOR memory.
> + *   supports XIP - Execute In Place directly from NOR memory. Destination
> + *   address of the XIP image is set to 0xFFFFFFFF and execute address to the
> + *   absolute offset in bytes from the beginning of NOR memory.
>    *
>    * - IBR_HDR_NAND_ID (0x8B):
>    *   NAND image can be stored either at any 2 MB aligned offset in the first

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de

Re: [PATCH u-boot-mvebu] tools: kwboot: Document information about NOR XIP

[Stefan Roese]

On 3/23/23 21:00, Pali Rohár wrote:
> Signed-off-by: Pali Rohár <pali@kernel.org>

Applied to u-boot-marvell/next

Thanks,
Stefan

> ---
>   tools/kwboot.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/kwboot.c b/tools/kwboot.c
> index 2b92966919da..db917708a8e8 100644
> --- a/tools/kwboot.c
> +++ b/tools/kwboot.c
> @@ -61,7 +61,9 @@
>    *   SPI-NOR or parallel-NOR. Despite the type name it really can be stored on
>    *   parallel-NOR and cannot be stored on other SPI devices, like SPI-NAND.
>    *   So it should have been named NOR image, not SPI image. This image type
> - *   supports XIP - Execute In Place directly from NOR memory.
> + *   supports XIP - Execute In Place directly from NOR memory. Destination
> + *   address of the XIP image is set to 0xFFFFFFFF and execute address to the
> + *   absolute offset in bytes from the beginning of NOR memory.
>    *
>    * - IBR_HDR_NAND_ID (0x8B):
>    *   NAND image can be stored either at any 2 MB aligned offset in the first

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de