7 comments:
File src/mainboard/ocp/tiogapass/Kconfig:
Vendor shouldn't be needed here
Patch Set #14, Line 34: TiogaPass
Is it TiogaPass or Tioga Pass?
config MAINBOARD_VENDOR
string
default "OCP"
Set to "Open Compute Project" in src/mb/ocp/Kconfig
File src/mainboard/ocp/tiogapass/Makefile.inc:
Patch Set #14, Line 25: CPPFLAGS_common += -I$(src)/soc/intel/skylake_sp/include
This should not be here
File src/mainboard/ocp/tiogapass/acpi_tables.c:
/* Disable USB ports in S5 */
gnvs->s5u0 = 0;
gnvs->s5u1 = 0;
These can be dropped
Patch Set #14, Line 32: Present
Really?
File src/mainboard/ocp/tiogapass/devicetree.cb:
Patch Set #14, Line 62: device pci 04.0 on end # Intel Corporation Sky Lake-E CBDMA Registers
These comments can be shortened
To view, visit change 38549. To unsubscribe, or for help writing mail filters, visit settings.