29 comments:
Patch Set #13, Line 7: src/mainboard/libretrend/lt1000: Initial commit
Please use just `mb/libretrend/lt1000:`, and use a statement by adding a verb in imperative mood. […]
Done
Please add a note, how you created that board. […]
Yes it was contracted, but only partially. Upstreaming this is my/3mdeb own initiative.
File Documentation/mainboard/libretrend/lt1000.md:
Patch Set #13, Line 23: Kabylake
Kaby Lake
Done
Patch Set #13, Line 25: *3rdparty/fsp* submodule.
Mark up as code `3rdparty/fsp`.
Done
Patch Set #13, Line 27: microcode
Microcode updates are …
Done
Patch Set #13, Line 28: from the *3rdparty/intel-microcode* submodule.
Mark up as code/monospace.
Done
, whose datasheet
can be found [here][W25Q64FV]
Just do [datasheet][W25Q64FV].
Done
Patch Set #13, Line 61: onboard
Capitalize this: "Onboard"
Done
Patch Set #13, Line 67: InfraRed
infrared
Done
Patch Set #13, Line 102: Super I/O, EC
Kind of. It is Super I/O with integrated Environmental Controller (EC). […]
Removed EC.
File src/mainboard/libretrend/lt1000/Kconfig:
select MAINBOARD_HAS_LPC_TPM
select INTEL_GMA_HAVE_VBT
select MAINBOARD_HAS_LIBGFXINIT
any reason not to alphabetize these?
Done
File src/mainboard/libretrend/lt1000/acpi/mainboard.asl:
IIRC there was a discussion some time ago about correctness of adding power button in ACPI. […]
Noticed some board also have LIDs and sleep buttons. Some mainboards do not have power button (physical button) so I wonder if it should be included in common location.
Patch Set #13, Line 21: Name (PBST, One)
Probably nothing. Copy-paste error.
Removed.
File src/mainboard/libretrend/lt1000/acpi_tables.c:
Definitely.
Done
File src/mainboard/libretrend/lt1000/bootblock.c:
Patch Set #13, Line 38: devictree
devicetree
Done
Patch Set #13, Line 41: devictree
here too
Done
File src/mainboard/libretrend/lt1000/devicetree.cb:
There's an enum for this
Done
File src/mainboard/libretrend/lt1000/dsdt.asl:
Patch Set #13, Line 28: // Some generic macros
This comment got dropped
Done
Patch Set #13, Line 34: // CPU
This comment is not useful
Done
Scope (\_SB) {
Device (PCI0)
Device (\_SB. […]
Done
Patch Set #13, Line 45: // Chipset specific sleep states
This got dropped as well.
Done
File src/mainboard/libretrend/lt1000/mainboard.c:
this can be dropped/replaced by selection of DRIVERS_GENERIC_CBFS_SERIAL in Kconfig
dropped as it is not needed
Please add spaces around the 0.
File was removed.
Patch Set #13, Line 47: MAX_SERIAL_LENGTH);
Fits in 96 characters.
File was removed.
File src/mainboard/libretrend/lt1000/ramstage.c:
Remove. […]
Done
File src/mainboard/libretrend/lt1000/romstage.c:
Patch Set #13, Line 38: /* DQS CPU<>DRAM map
https://doc.coreboot.org/coding_style. […]
Done
Patch Set #13, Line 50: /* Rcomp resistor */
Unneeded comment.
Done
Patch Set #13, Line 53: sizeof(RcompResistor));
One line.
Done
https://doc.coreboot.org/coding_style. […]
Done
To view, visit change 30360. To unsubscribe, or for help writing mail filters, visit settings.