diff mbox series

[v2,2/4] arm: mvebu: Espressobin: Change default loadaddr

Message ID 20200827130128.22389-2-pali@kernel.org
State Accepted
Commit 996ecfd3ec4e86be9dbee17b2223061e16627f71
Delegated to: Stefan Roese
Headers show
Series [v2,1/4] arm: mvebu: Espressobin: Move env load addresses | expand

Commit Message

Pali Rohár Aug. 27, 2020, 1:01 p.m. UTC
Current loadaddr is not writable on Espressobin and 'loadb' command cause
rebooting board. Marvell's U-Boot already set default loadaddr to value
0x06000000 [1] and after this change 'loadb' is working fine.

[1] - https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/0a5b159806baa0746b6449920e450498bc269ba7

Signed-off-by: Pali Rohár <pali@kernel.org>
---
 include/configs/mvebu_armada-37xx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stefan Roese Aug. 27, 2020, 3:31 p.m. UTC | #1
On 27.08.20 15:01, Pali Rohár wrote:
> Current loadaddr is not writable on Espressobin and 'loadb' command cause
> rebooting board. Marvell's U-Boot already set default loadaddr to value
> 0x06000000 [1] and after this change 'loadb' is working fine.
> 
> [1] - https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/0a5b159806baa0746b6449920e450498bc269ba7
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>

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

Thanks,
Stefan

> ---
>   include/configs/mvebu_armada-37xx.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/mvebu_armada-37xx.h b/include/configs/mvebu_armada-37xx.h
> index a9ed0a838f..93c05e30cc 100644
> --- a/include/configs/mvebu_armada-37xx.h
> +++ b/include/configs/mvebu_armada-37xx.h
> @@ -37,7 +37,7 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
> +#define CONFIG_SYS_LOAD_ADDR	0x06000000	/* default load adr */
>   #define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
> 


Viele Grüße,
Stefan
Andre Heider Aug. 27, 2020, 4:04 p.m. UTC | #2
On 27/08/2020 15:01, Pali Rohár wrote:
> Current loadaddr is not writable on Espressobin and 'loadb' command cause
> rebooting board. Marvell's U-Boot already set default loadaddr to value
> 0x06000000 [1] and after this change 'loadb' is working fine.
> 
> [1] - https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/0a5b159806baa0746b6449920e450498bc269ba7
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>

Tested-by: Andre Heider <a.heider@gmail.com>
Stefan Roese Aug. 31, 2020, 1:06 p.m. UTC | #3
On 27.08.20 15:01, Pali Rohár wrote:
> Current loadaddr is not writable on Espressobin and 'loadb' command cause
> rebooting board. Marvell's U-Boot already set default loadaddr to value
> 0x06000000 [1] and after this change 'loadb' is working fine.
> 
> [1] - https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/0a5b159806baa0746b6449920e450498bc269ba7
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>

Applied to u-boot-marvell/master

Thanks,
Stefan

> ---
>   include/configs/mvebu_armada-37xx.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/mvebu_armada-37xx.h b/include/configs/mvebu_armada-37xx.h
> index a9ed0a838f..93c05e30cc 100644
> --- a/include/configs/mvebu_armada-37xx.h
> +++ b/include/configs/mvebu_armada-37xx.h
> @@ -37,7 +37,7 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
> +#define CONFIG_SYS_LOAD_ADDR	0x06000000	/* default load adr */
>   #define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
> 


Viele Grüße,
Stefan
diff mbox series

Patch

diff --git a/include/configs/mvebu_armada-37xx.h b/include/configs/mvebu_armada-37xx.h
index a9ed0a838f..93c05e30cc 100644
--- a/include/configs/mvebu_armada-37xx.h
+++ b/include/configs/mvebu_armada-37xx.h
@@ -37,7 +37,7 @@ 
 /*
  * Other required minimal configurations
  */
-#define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
+#define CONFIG_SYS_LOAD_ADDR	0x06000000	/* default load adr */
 #define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
 #define CONFIG_SYS_MAXARGS	32	/* max number of command args */