random-mirrors/smaeul-u-boot
1
0
Fork 0
mirror of https://github.com/smaeul/u-boot.git synced 2025-10-14 04:46:01 +01:00
Code Issues Packages Projects Releases Wiki Activity

ddr: fsl: Allow to detect 4 GB DDR modules in 32-bit mode

Browse Source 
U-Boot core code already handles the case when RAM size is bigger than
CONFIG_MAX_MEM_MAPPED. So there is no need to do duplicate check in fsl ddr
driver for CONFIG_MAX_MEM_MAPPED. Instead simplify code to just check if
RAM size can be representable in phys_size_t type. And avoid printing
warning if phys_size_t is just 1 byte smaller than RAM size, which is the
typical situation with 4 GB DDR module.

Signed-off-by: Pali Rohár <pali@kernel.org>
This commit is contained in:
Pali Rohár 2022-09-09 17:32:44 +02:00 committed by Tom Rini
parent 922460be0b
commit 2721997654
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
drivers/ddr/fsl/main.c
View File

@ -857,15 +857,18 @@ phys_size_t __fsl_ddr_sdram(fsl_ddr_info_t *pinfo)
debug("total_memory by %s = %llu\n", __func__, total_memory); debug("total_memory by %s = %llu\n", __func__, total_memory);
#if !defined(CONFIG_PHYS_64BIT) #if !defined(CONFIG_PHYS_64BIT)
/* Check for more than max memory. Bad. */ /*
if ((first_ctrl == 0) && (total_memory > CONFIG_MAX_MEM_MAPPED)) { * Show warning about big DDR moodules. But avoid warning for 4 GB DDR
* modules when U-Boot supports RAM of maximal size 4 GB - 1 byte.
*/
if ((first_ctrl == 0) && (total_memory - 1 > (phys_size_t)~0ULL)) {
puts("Detected "); puts("Detected ");
print_size(total_memory, " of memory\n"); print_size(total_memory, " of memory\n");
#ifndef CONFIG_SPL_BUILD #ifndef CONFIG_SPL_BUILD
puts(" "); /* re-align to match init_dram print */ puts(" "); /* re-align to match init_dram print */
#endif #endif
puts("This U-Boot only supports <= "); puts("This U-Boot only supports <= ");
print_size(CONFIG_MAX_MEM_MAPPED, " of DDR\n"); print_size((unsigned long long)((phys_size_t)~0ULL)+1, " of DDR\n");
#ifndef CONFIG_SPL_BUILD #ifndef CONFIG_SPL_BUILD
puts(" "); /* re-align to match init_dram print */ puts(" "); /* re-align to match init_dram print */
#endif #endif
@ -873,10 +876,13 @@ phys_size_t __fsl_ddr_sdram(fsl_ddr_info_t *pinfo)
#ifndef CONFIG_SPL_BUILD #ifndef CONFIG_SPL_BUILD
puts(" "); /* re-align to match init_dram print */ puts(" "); /* re-align to match init_dram print */
#endif #endif
total_memory = CONFIG_MAX_MEM_MAPPED;
} }
#endif #endif
/* Ensure that total_memory does not overflow on return */
if (total_memory > (phys_size_t)~0ULL)
total_memory = (phys_size_t)~0ULL;
return total_memory; return total_memory;
} }