Hello Damien Zammit, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45424
to look at the new patch set (#2).
Change subject: nb/intel/x4x: Place raminit definitions in raminit.h ......................................................................
nb/intel/x4x: Place raminit definitions in raminit.h
There's no need to have implementation details in a public header.
Tested with BUILD_TIMELESS=1, Asus P5QL PRO remains identical.
Change-Id: I04d8c610d3e52adecfe96cc435f0523bedf3060a Signed-off-by: Angel Pons th3fanbus@gmail.com --- M src/northbridge/intel/x4x/dq_dqs.c M src/northbridge/intel/x4x/raminit.c A src/northbridge/intel/x4x/raminit.h M src/northbridge/intel/x4x/raminit_ddr23.c M src/northbridge/intel/x4x/raminit_tables.c M src/northbridge/intel/x4x/rcven.c M src/northbridge/intel/x4x/romstage.c M src/northbridge/intel/x4x/x4x.h 8 files changed, 259 insertions(+), 245 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/45424/2