Fix SPI read and Enable required configs for Exynos5

From: Michael Pratt <[email protected]>

Since snow has a different memory configuration than peach, split the configuration between the 5250 and 5420. Exynos 5420 supports runtime memory configuration detection, and can make the determination between 4 and 7 banks at runtime.

Include the bank size with the number of banks for context to make the number of banks meaningful.

Signed-off-by Michael Pratt <[email protected]>

Signed-off-by Akshay Saraswat <[email protected]>

Acked-by Simon Glass <[email protected]>

Tested-by Simon Glass <[email protected]>

---
Changes since v2:
- Added "Acked-by" & "Tested-by".
Changes since v1:
- New patch.

 include/configs/exynos5-dt.h    | 2 --
 include/configs/exynos5250-dt.h | 5 +++++
 include/configs/exynos5420.h    | 4 ++++
 3 files changed, 9 insertions(+), 2 deletions(-)

include/configs/exynos5-dt.h +0 -2

index d3ef44c..fd607ee 100644
@@ -161,8 +161,6 @@
 
 #define CONFIG_RD_LVL
 
-#define CONFIG_NR_DRAM_BANKS	8
-#define SDRAM_BANK_SIZE		(256UL << 20UL)	/* 256 MB */
 #define PHYS_SDRAM_1		CONFIG_SYS_SDRAM_BASE
 #define PHYS_SDRAM_1_SIZE	SDRAM_BANK_SIZE
 #define PHYS_SDRAM_2		(CONFIG_SYS_SDRAM_BASE + SDRAM_BANK_SIZE)

include/configs/exynos5250-dt.h +5 -0

index 10b8942..27aa455 100644
@@ -65,4 +65,9 @@
 #define LCD_YRES			1600
 #define LCD_BPP			LCD_COLOR16
 #endif
+
+/* DRAM Memory Banks */
+#define CONFIG_NR_DRAM_BANKS	8
+#define SDRAM_BANK_SIZE		(256UL << 20UL)	/* 256 MB */
+
 #endif  /* __CONFIG_5250_H */

include/configs/exynos5420.h +0 -0

index 2ffe5ee..d2a9556 100644

Dear Akshay Saraswat,

Quoted 2 lines · Akshay Saraswat

From: Michael Pratt <[email protected]>

Quoted 4 lines

Since snow has a different memory configuration than peach, split the configuration between the 5250 and 5420. Exynos 5420 supports runtime memory configuration detection, and can make the determination between 4 and 7 banks at runtime.

I think this patch should be included to your peach-pit patchset.
And I think, the number of banks and the size of bank seems to board specific feature. Can you guarantee if it uses same SoC then have same memory banks?

Hi Minkyu,

Quoted 10 lines · Minkyu Kang

Dear Akshay Saraswat,

Quoted 6 lines · Akshay Saraswat

From: Michael Pratt <[email protected]>

Since snow has a different memory configuration than peach, split the configuration between the 5250 and 5420. Exynos 5420 supports runtime memory configuration detection, and can make the determination between 4 and 7 banks at runtime.

Quoted 3 lines

I think this patch should be included to your peach-pit patchset.
And I think, the number of banks and the size of bank seems to board specific feature. Can you guarantee if it uses same SoC then have same memory banks?

I think this is better than what we have there at present. There is a patch from Chromium that puts this in the device tree, but it is probably best dealt with when more patches have landed.

Quoted 13 lines · Simon Glass

Hi Minkyu,

Quoted 10 lines · Minkyu Kang

Dear Akshay Saraswat,

Quoted 6 lines · Akshay Saraswat

From: Michael Pratt <[email protected]>

Since snow has a different memory configuration than peach, split the configuration between the 5250 and 5420. Exynos 5420 supports runtime memory configuration detection, and can make the determination between 4 and 7 banks at runtime.

Quoted 8 lines

Quoted 3 lines

I think this patch should be included to your peach-pit patchset.
And I think, the number of banks and the size of bank seems to board specific feature. Can you guarantee if it uses same SoC then have same memory banks?

I think this is better than what we have there at present. There is a patch from Chromium that puts this in the device tree, but it is probably best dealt with when more patches have landed.

I didn't deny this patch.
My comment is about present state.
If you can not guarantee that have same memory banks then please split this configuration to each board's configs.

Hi Minkyu,

Quoted 23 lines · Minkyu Kang

Quoted 13 lines · Simon Glass

Hi Minkyu,

Quoted 10 lines · Minkyu Kang

Dear Akshay Saraswat,

Quoted 6 lines · Akshay Saraswat

From: Michael Pratt <[email protected]>

Since snow has a different memory configuration than peach, split the configuration between the 5250 and 5420. Exynos 5420 supports runtime memory configuration detection, and can make the determination between 4 and 7 banks at runtime.

Quoted 8 lines

Quoted 3 lines

I think this patch should be included to your peach-pit patchset.
And I think, the number of banks and the size of bank seems to board specific feature. Can you guarantee if it uses same SoC then have same memory banks?

I think this is better than what we have there at present. There is a patch from Chromium that puts this in the device tree, but it is probably best dealt with when more patches have landed.

Quoted 4 lines

I didn't deny this patch.
My comment is about present state.
If you can not guarantee that have same memory banks then please split this configuration to each board's configs.

For Pit which can support 2GB or 4GB, U-Boot detects the correct size at run-time. So the setting in the config file is the *maximum* memory supported by that SOC, which is indeed fixed by the SOC and has nothing to do with the board.