[PATCH] Correct sun8i-v3s SRAM size

[PATCH] Correct sun8i-v3s SRAM size

[Benedikt-Alexander Mokroß]

According to the Datasheet, the V3s has a 32KiB SRAM.
This patch corrects CONFIG_SPL_MAX_SIZE and LOW_LEVEL_SRAM_STACK
accordingly.

Signed-off-by: Benedikt-Alexander Mokro? <u-boot@bamkrs.de>
---
  include/configs/sunxi-common.h | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
index 0ef289fd64..623501edce 100644
--- a/include/configs/sunxi-common.h
+++ b/include/configs/sunxi-common.h
@@ -185,8 +185,11 @@
  #define CONFIG_SPL_MAX_SIZE		0x7fa0		/* 32 KiB */
  /* end of SRAM A2 on H6 for now */
  #define LOW_LEVEL_SRAM_STACK		0x00118000
+#elif defined CONFIG_MACH_SUN8I_V3S
+#define CONFIG_SPL_MAX_SIZE		0x7fa0		/* 32 KiB */
+#define LOW_LEVEL_SRAM_STACK		0x0000F000	/* End of sram */
  #else
  #define CONFIG_SPL_MAX_SIZE		0x5fa0		/* 24KB on sun4i/sun7i */
  #define LOW_LEVEL_SRAM_STACK		0x00008000	/* End of sram */
  #endif

-- 
2.20.1

[PATCH] Correct sun8i-v3s SRAM size

[Jagan Teki]

On Tue, Apr 28, 2020 at 9:26 PM Benedikt-Alexander Mokro?
<catchall@bamkrs.de> wrote:
>
> According to the Datasheet, the V3s has a 32KiB SRAM.
> This patch corrects CONFIG_SPL_MAX_SIZE and LOW_LEVEL_SRAM_STACK
> accordingly.

Look like the existing value has taken from allwinner BSP, but did you
find any issues with the existing one?

Jagan.

[PATCH] Correct sun8i-v3s SRAM size

[Benedikt-Alexander Mokroß]

Hi Jagan,

I run into some size-related issues regarding my SPL configuration for a custom V3s board (mainly due to using fit images). In fact, the V3s has even more SRAM that could be used for the SPL. However the BROM won't boot anything larger than 32KiB. So I use this 32KiB limit in production now and found no issues with my 29KiB SPL whatsoever. And - if I read the BROM ASM correctly - there is no performance drawback for default (and smaller) SPL configurations since the BROM only loads the actual SPL size amount of pages/bytes.

I think it would be much less frustrating for other users if they could use these 32KiB from the start since the current limit is hit pretty fast and there is no reason why it should be this 24k limit for the V3s. It's another poorly or even plain wrongly documented aspect of the Allwinner BSP.

Benedikt

Am 1. Juni 2020 19:00:30 MESZ schrieb Jagan Teki <jagan@amarulasolutions.com>:
On Tue, Apr 28, 2020 at 9:26 PM Benedikt-Alexander Mokro?
<catchall@bamkrs.de> wrote:

 According to the Datasheet, the V3s has a 32KiB SRAM.
 This patch corrects CONFIG_SPL_MAX_SIZE and LOW_LEVEL_SRAM_STACK
 accordingly.

Look like the existing value has taken from allwinner BSP, but did you
find any issues with the existing one?

Jagan.

[PATCH] Correct sun8i-v3s SRAM size

[Maxime Ripard]

On Tue, Jun 02, 2020 at 09:10:08AM +0200, Benedikt-Alexander Mokro? wrote:
> Hi Jagan,
> 
> I run into some size-related issues regarding my SPL configuration for
> a custom V3s board (mainly due to using fit images). In fact, the V3s
> has even more SRAM that could be used for the SPL. However the BROM
> won't boot anything larger than 32KiB. So I use this 32KiB limit in
> production now and found no issues with my 29KiB SPL whatsoever. And -
> if I read the BROM ASM correctly - there is no performance drawback
> for default (and smaller) SPL configurations since the BROM only loads
> the actual SPL size amount of pages/bytes.
> 
> I think it would be much less frustrating for other users if they
> could use these 32KiB from the start since the current limit is hit
> pretty fast and there is no reason why it should be this 24k limit for
> the V3s. It's another poorly or even plain wrongly documented aspect
> of the Allwinner BSP.

I agree

Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200602/fbdc9a5f/attachment.sig>

[PATCH] Correct sun8i-v3s SRAM size

[Jagan Teki]

On Tue, Jun 2, 2020 at 12:40 PM Benedikt-Alexander Mokro?
<u-boot@bamkrs.de> wrote:
>
> Hi Jagan,
>
> I run into some size-related issues regarding my SPL configuration for a custom V3s board (mainly due to using fit images). In fact, the V3s has even more SRAM that could be used for the SPL. However the BROM won't boot anything larger than 32KiB. So I use this 32KiB limit in production now and found no issues with my 29KiB SPL whatsoever. And - if I read the BROM ASM correctly - there is no performance drawback for default (and smaller) SPL configurations since the BROM only loads the actual SPL size amount of pages/bytes.
>
> I think it would be much less frustrating for other users if they could use these 32KiB from the start since the current limit is hit pretty fast and there is no reason why it should be this 24k limit for the V3s. It's another poorly or even plain wrongly documented aspect of the Allwinner BSP.

Can you rebase on master and send it with the proper commit head.

"sunxi: Correct sun8i-v3s SRAM size"