Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/21602 )
Change subject: amd/gardenia: Fix number of memory channels ......................................................................
amd/gardenia: Fix number of memory channels
Gardenia (with Stoney Processor) has a single memory channel, not two. This corrects DMI type 17 reporting and the memory clear functions.
Change-Id: If49b6a9f37b2687ea2f64105fb9e476a89aa87ed Signed-off-by: Marc Jones marcj303@gmail.com Reviewed-on: https://review.coreboot.org/21602 Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/amd/gardenia/OemCustomize.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Marshall Dawson: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/mainboard/amd/gardenia/OemCustomize.c b/src/mainboard/amd/gardenia/OemCustomize.c index 3893e5d..6ca84b4 100644 --- a/src/mainboard/amd/gardenia/OemCustomize.c +++ b/src/mainboard/amd/gardenia/OemCustomize.c @@ -21,7 +21,7 @@ static const PSO_ENTRY DDR4PlatformMemoryConfiguration[] = { DRAM_TECHNOLOGY(ANY_SOCKET, DDR4_TECHNOLOGY), NUMBER_OF_DIMMS_SUPPORTED(ANY_SOCKET, ANY_CHANNEL, 2), - NUMBER_OF_CHANNELS_SUPPORTED(ANY_SOCKET, 2), + NUMBER_OF_CHANNELS_SUPPORTED(ANY_SOCKET, 1), MOTHER_BOARD_LAYERS(LAYERS_6), MEMCLK_DIS_MAP(ANY_SOCKET, ANY_CHANNEL, 0xff, 0xff, 0xff, 0xff, 0x00, 0x00, 0x00, 0x00),