Thanks!
29 comments:
File configs/config.ocp_tiogapass:
CONFIG_CPU_MICROCODE_CBFS_LOC=0xfff0fdc0
CONFIG_CPU_MICROCODE_CBFS_LEN=0x7C00
No need to specify these, they are the defaults as per src/soc/intel/xeon_sp/Kconfig
Done
File src/mainboard/ocp/tiogapass/Kconfig:
select HAVE_IFD_BIN
select HAVE_ME_BIN
Remove these and coreboot will not try to add an IFD or ME image. […]
Done
File src/mainboard/ocp/tiogapass/Makefile.inc:
Patch Set #38, Line 18: romstage-y += boardid.c
remove (see comment in tiogapass_boardid. […]
Done
Patch Set #38, Line 21: ramstage-y += boardid.c
same as previous comment
Done
File src/mainboard/ocp/tiogapass/acpi/platform.asl:
Please indent this file uniformly using tabs.
Done
Please indent this file uniformly using tabs.
Done
File src/mainboard/ocp/tiogapass/acpi_tables.c:
Patch Set #38, Line 20: #include <soc/nvs.h>
nit: this is not needed
Done
File src/mainboard/ocp/tiogapass/boardid.c:
This is currently unused, let's get rid of it for now and reintroduce it later if needed.
Done
File src/mainboard/ocp/tiogapass/devicetree.cb:
Please indent this file uniformly using tabs.
Done
File src/mainboard/ocp/tiogapass/romstage.c:
Patch Set #38, Line 18: #include "tiogapass_boardid.h"
remove (see comment in tiogapass_boardid. […]
Done
Patch Set #38, Line 20: #include <fsp/soc_binding.h>
unnecessary #include
Done
Patch Set #38, Line 21: #include <FspmUpd.h>
#include <soc/romstage.h> […]
Done
void mainboard_config_gpios(FSPM_UPD * mupd);
void mainboard_config_iio(FSPM_UPD *mupd);
void mainboard_memory_init_params(FSPM_UPD *mupd);
I'm surprised that the compiler doesn't err out on these... […]
Done
Patch Set #38, Line 33: void mainboard_config_gpios(FSPM_UPD *mupd)
I think this is only called from mainboard_memory_init_params() in this file, so it can be static. […]
Done
Patch Set #38, Line 40: void mainboard_config_iio(FSPM_UPD *mupd)
this can be static since it's only called from mainboard_memory_init_params()
Done
Patch Set #38, Line 45: sizeof(tp_iio_bifur_table)/sizeof(UPD_IIO_BIFURCATION_DATA_ENTRY);
ARRAY_SIZE(tp_iio_bifur_table)
Done
Patch Set #38, Line 50: sizeof(tp_iio_pci_port_skt0)/sizeof(UPD_PCI_PORT_CONFIG);
ARRAY_SIZE(tp_iio_pci_port_skt0);
Done
Patch Set #38, Line 55: sizeof(tp_pch_pci_port_skt0)/sizeof(UPD_PCH_PCIE_PORT);
ARRAY_SIZE(tp_pch_pci_port_skt0);
Done
File src/mainboard/ocp/tiogapass/skxsp_tp_gpio.h:
nit: extra asterix
Done
nit: extra asterix
Done
Needs an #endif
Done
File src/mainboard/ocp/tiogapass/skxsp_tp_iio.h:
nit: Use /* (without the extra asterix)
Done
Nit: Use */ (without the extra asterix)
Done
Nit: extra asterix
Done
Nit: extra asterix
Done
indent
Done
Nit: extra asterix
Done
Nit: extra asterix
Done
File src/mainboard/ocp/tiogapass/tiogapass_boardid.h:
This seems copy+pasted from harcuvar. […]
Done
To view, visit change 38549. To unsubscribe, or for help writing mail filters, visit settings.