[U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig
B, Ravi
ravibabu at ti.com
Thu Jul 28 10:23:55 CEST 2016
Masahiro-san
>>#if !defined(CONFIG_PPC_P5020) && !defined(CONFIG_PPC_P5040) #error Must call Cyrus CONFIG with a specific CPU enabled.
>>#endif
>>diff --git a/include/configs/dra7xx_evm.h
>>b/include/configs/dra7xx_evm.h index 0d51aeb..a11b692 100644
>>--- a/include/configs/dra7xx_evm.h
>>+++ b/include/configs/dra7xx_evm.h
>>@@ -20,7 +20,6 @@
>> #endif
>>#define CONFIG_VERY_BIG_RAM
>>-#define CONFIG_PHYS_64BIT
>Applied this patch on mainline branch, I see warning. It seems like CONFIG_PHYS_64BIT is not taking effect, the sizeof phys_addr_t is 4 bytes instead of 8 byte.
Due to removal of CONFIG_PHYS_64BIT in dra7xx_evm.h, phys_addr_t set to 4, and due to SDRAM init would have failed and u-boot prompt did not appear.
I have defined CONFIG_PHYS_64BIT in dra7xx_evm.h, I did not see any warning and I could see u-boot prompt, worked fine.
>>board/ti/dra7xx/evm.c: In function 'dram_init_banksize':
>>board/ti/dra7xx/evm.c:408:30: warning: large integer implicitly truncated to unsigned type [-Woverflow]
>> gd->bd->bi_dram[1].start = 0x200000000;
^
>>board/ti/dra7xx/evm.c: In function 'dram_init_banksize':
>>board/ti/dra7xx/evm.c:408:30: warning: large integer implicitly truncated to unsigned type [-Woverflow]
>> gd->bd->bi_dram[1].start = 0x200000000;
^
Regards
Ravi
More information about the U-Boot
mailing list