Attention is currently required from: Kangheui Won, Rizwan Qureshi, Tim Wawrzynczak. Rizwan Qureshi has uploaded a new patch set (#5) to the change originally created by Krishna P Bhat D. ( https://review.coreboot.org/c/coreboot/+/61764 )
Change subject: mb/google/nissa: Set half_populated true ......................................................................
mb/google/nissa: Set half_populated true
Alder Lake N has single memory controller with 64-bit bus width. Alder Lake common meminit block driver considers bus width to be 128-bit and populates the meminit data accordingly. By setting half_populated to true, only the bottom half is populated.
Ideally, half_populated is used in platforms with multiple channels to enable only one half of the channel. Alder Lake N has single channel, and it would require for new structures to be defined in meminit block driver for LPx memory configurations. In order to avoid adding new structures, set half_populated to true. This has the same effect as having single channel with 64-bit width.
Change-Id: I414e5dc82caf47b6b96c474b3ef6e01c2ce0226e Signed-off-by: Krishna Prasad Bhat krishna.p.bhat.d@intel.com Signed-off-by: Rizwan Qureshi rizwan.qureshi@intel.com --- M src/mainboard/google/brya/variants/baseboard/nissa/memory.c 1 file changed, 9 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/61764/5