From 4a73bf8378d8e36f0bae1488aa0b17cb5d5bd4ce Mon Sep 17 00:00:00 2001 From: Andrey Petrov Date: Thu, 1 Aug 2019 15:09:37 -0700 Subject: [PATCH] soc/intel/fsp_broadwell_de: Populate SMBIOS tables with memory information Add code to read SPD data, parse it and save into SMBIOS table. This is implemented for socketed DDR4 chips only. For soldered-down memory this is not implemented and probably won't be ever needed. TEST=tested on OCP Monolake mainboard, and found dmidecode -t memory to work. The stack has also been tested on an out-of-tree board. Signed-off-by: Andrey Petrov Change-Id: I1162eb4484dab46f1ab9fe3426eecc4d9378e8e2 Reviewed-on: https://review.coreboot.org/c/coreboot/+/34681 Tested-by: build bot (Jenkins) Reviewed-by: Matt DeVillier --- src/soc/intel/fsp_broadwell_de/Kconfig | 4 ++ .../fsp_broadwell_de/include/soc/memory.h | 30 +++++++++ .../fsp_broadwell_de/include/soc/pci_devs.h | 5 ++ .../fsp_broadwell_de/romstage/Makefile.inc | 1 + .../intel/fsp_broadwell_de/romstage/memory.c | 61 +++++++++++++++++++ .../fsp_broadwell_de/romstage/romstage.c | 4 ++ 6 files changed, 105 insertions(+) create mode 100644 src/soc/intel/fsp_broadwell_de/include/soc/memory.h create mode 100644 src/soc/intel/fsp_broadwell_de/romstage/memory.c diff --git a/src/soc/intel/fsp_broadwell_de/Kconfig b/src/soc/intel/fsp_broadwell_de/Kconfig index 7783f78599..fef7a822d9 100644 --- a/src/soc/intel/fsp_broadwell_de/Kconfig +++ b/src/soc/intel/fsp_broadwell_de/Kconfig @@ -88,6 +88,10 @@ config SERIRQ_CONTINUOUS_MODE If you set this option to y, the serial IRQ machine will be operated in continuous mode. +config DIMM_SPD_SIZE + int + default 512 + ## Broadwell-DE Specific FSP Kconfig source src/soc/intel/fsp_broadwell_de/fsp/Kconfig diff --git a/src/soc/intel/fsp_broadwell_de/include/soc/memory.h b/src/soc/intel/fsp_broadwell_de/include/soc/memory.h new file mode 100644 index 0000000000..3bdba2ef56 --- /dev/null +++ b/src/soc/intel/fsp_broadwell_de/include/soc/memory.h @@ -0,0 +1,30 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2019 Facebook, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#ifndef _SOC_MEMORY_H_ +#define _SOC_MEMORY_H_ + +/* EDS vol 2, 9.2.24 */ +#define REG_MC_BIOS_REQ 0x98 +#define REG_MC_BIOS_REQ_FREQ_MSK ((1u << 6) - 1) +#define REG_MC_MULTIPLIER 133.33f + +#define IMC_MAX_CHANNELS 2 + +#define SPD_SLAVE_ADDR(chan, slot) (2 * chan + slot) + +void save_dimm_info(void); + +#endif diff --git a/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h b/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h index ee0b80e49b..973f55983b 100644 --- a/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h +++ b/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h @@ -131,4 +131,9 @@ #define IMC_DEV PCI_DEV(QPI_BUS, IMC_DEV0, IMC_FUNC0) +#define PCU1_DEV 30 +#define PCU1_FUNC 01 +#define UBOX_DEV 16 +#define UBOX_FUNC 7 + #endif /* _SOC_PCI_DEVS_H_ */ diff --git a/src/soc/intel/fsp_broadwell_de/romstage/Makefile.inc b/src/soc/intel/fsp_broadwell_de/romstage/Makefile.inc index 5ef9a9251d..5c332018e0 100644 --- a/src/soc/intel/fsp_broadwell_de/romstage/Makefile.inc +++ b/src/soc/intel/fsp_broadwell_de/romstage/Makefile.inc @@ -1,3 +1,4 @@ romstage-y += romstage.c +romstage-y += memory.c $(obj)/soc/intel/fsp_broadwell_de/romstage/romstage.romstage.o : $(obj)/build.h diff --git a/src/soc/intel/fsp_broadwell_de/romstage/memory.c b/src/soc/intel/fsp_broadwell_de/romstage/memory.c new file mode 100644 index 0000000000..7574c5fb66 --- /dev/null +++ b/src/soc/intel/fsp_broadwell_de/romstage/memory.c @@ -0,0 +1,61 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2019 Facebook, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include +#include +#include +#include +#include +#include + +static uint32_t get_memory_dclk(void) +{ + uint32_t reg32 = + pci_mmio_read_config32(PCI_DEV(QPI_BUS, PCU1_DEV, PCU1_FUNC), REG_MC_BIOS_REQ); + return (reg32 & REG_MC_BIOS_REQ_FREQ_MSK) * REG_MC_MULTIPLIER; +} + +void save_dimm_info(void) +{ + int index = 0; + uint32_t dclk_mhz = 0; + + /* + * When talking to SPD chips through IMC slave offset of 0x50 is automagically added + * by hardware. Real-world slave numbers translate to: 0xa0, 0xa2, 0xa4, 0xa6. + */ + struct spd_block blk = {.addr_map = {SPD_SLAVE_ADDR(0, 0), SPD_SLAVE_ADDR(0, 1), + SPD_SLAVE_ADDR(1, 0), SPD_SLAVE_ADDR(1, 1)} }; + + get_spd_smbus(&blk); + dump_spd_info(&blk); + + dclk_mhz = get_memory_dclk(); + + /* + * The platform is limited to 2 channels and max 2 dimms per channel. + * It doesn't look like DDR3 is supported so we assume memory is all DDR4. + */ + + for (int channel = 0; channel < IMC_MAX_CHANNELS; channel++) { + for (int slot = 0; slot < CONFIG_DIMM_MAX / IMC_MAX_CHANNELS; slot++) { + dimm_attr dimm = {0}; + u8 *spd_data = blk.spd_array[index]; + if (spd_decode_ddr4(&dimm, spd_data) == SPD_STATUS_OK) + spd_add_smbios17_ddr4(channel, index, dclk_mhz, &dimm); + index++; + } + } +} diff --git a/src/soc/intel/fsp_broadwell_de/romstage/romstage.c b/src/soc/intel/fsp_broadwell_de/romstage/romstage.c index 085395e9a8..3b74a1cb69 100644 --- a/src/soc/intel/fsp_broadwell_de/romstage/romstage.c +++ b/src/soc/intel/fsp_broadwell_de/romstage/romstage.c @@ -29,6 +29,7 @@ #include #include #include +#include #include #include #include @@ -165,6 +166,9 @@ void romstage_main_continue(EFI_STATUS status, void *hob_list_ptr) die("Could not allocate cbmem for HOB pointer"); *(u32 *)cbmem_hob_ptr = (u32)hob_list_ptr; + if (!CONFIG(FSP_MEMORY_DOWN)) + save_dimm_info(); + /* Load the ramstage. */ post_code(0x4e); run_ramstage();