CONFIG_STATIC_DDR definition breaking ATF compilation on LS1028ARDB

[lukipedio]

I am trying to bring up a custom board based on LS1028ARDB.

Our DDR configuration is different since we use two DDR chips without an SDP EEPROM, instead of four chips and SPD as in LS1028RDB.

So we would like to use a static DDR configuration.

I have followed the documentation and I have defined CONFIG_STATIC_DDR in platform_def.h

But doing this breaks the compilation!

I have a clean Yocto installation by using the following steps:

repo init -u https://github.com/nxp-qoriq/yocto-sdk -b scarthgap

repo_sync

./-setup-env -m ls1028ardb

source build_ls1028ardb

bitbake qoriq-atf

## Hack code to insert #define CONFIG_STATIC_DDR in build_ls1028ardb/tmp/work/ls1028ardb-fsl-linux/qoriq-atf/2.10+/git/plat/nxp/soc-ls1028a/ls1028ardb/platform_def.h

bitbake -c compile -f qoriq-atf

Compilation errors.....

I have seen many posts regarding static DDR configuration, so I wonder what could be wrong in my case by starting from a new and clean yocto installation.

I hope it is a trivial issue.

Thanks,

Luca

[lukipedio]

Anyone replicated this issue.

I am really stuck since hacking code by writing DDR registsres only withiout all the support from the NXP ddr driver is quite error-prone.

For example, I am setting all DDR registers right, but then IU-boot reports an incorrect RAM size.

[Oswalag]

Please follow the recommendation in the following post:

https://community.nxp.com/t5/Layerscape/how-to-config-DDR-init-in-BL2/m-p/1544952

[lukipedio]

 I have first compiles without any modification to the default ddr_init.c implementation.

First error comes from board_static_ddr() definition:
I need to change from:
unsigned long long board_static_ddr(struct ddr_info *priv)
to
long long board_static_ddr(struct ddr_info *priv)

Then linker complains about not finding 

ddr_get_ddr_params and 
ddr_board_options.

Looking into code, I see those functions are still called when CONFIG_STATIC_DDR is defined.
Some call could be skipped by nondefining CONFIG_DDR_NODIMM
but even by nondefining it, the same errors appear.

[Oswalag]

Hello.

Please confirm that you don't have any issue when compiling before your modifications, after that please share the file with the exact changes made between the first one and the custom one. Then, please share in a .txt file and screenshot of the issue when building.

[lukipedio]

Hi @Oswalag ,

  I can confirm I have no issue compiling before modification.

  Then only add CONFIG_STATIC_DDR constant definition in the suggested platform_def.h file.

   I edit the platform_def.h file into build_ls1028ardb/tmp/work/ls1028ardb-fsl-linux/qoriq-atf/2.10+/git/plat/nxp/soc-ls1028a/ls1028ardb

  
   Then I force an ATF compilation:
   bitbake -c compile -f qoriq-atf 

 
  and I got a compilation error.