Attention is currently required from: Marc Jones, Anjaneya "Reddy" Chagam, Johnny Lin, Paul Menzel, Stefan Reinauer, Angel Pons, ron minnich, Tim Chu. Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51824 )
Change subject: Documentation/mb/ocp: Update Delta Lake documentation ......................................................................
Patch Set 5:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/51824/comment/0479c4de_e705dbb5 PS5, Line 10: OSF (Open System Firmware) solution.
I have no idea, what the second part means. […]
Done
File Documentation/mainboard/ocp/deltalake.md:
https://review.coreboot.org/c/coreboot/+/51824/comment/0b82c016_382f5093 PS5, Line 31: Intel.
Or even move it to the previous line (it should fit)
Done
https://review.coreboot.org/c/coreboot/+/51824/comment/8aee1d76_8f71dae0 PS5, Line 66: mapped to verbose, 0 to 4 and 9 would be mapped to quiet.
Use two spaces?
Done
https://review.coreboot.org/c/coreboot/+/51824/comment/2633f288_3704ca29 PS5, Line 103: - KM/BPM signing
Yeah, I'd use 8 spaces
Done
https://review.coreboot.org/c/coreboot/+/51824/comment/5beee6d8_d11fa9cc PS5, Line 151: CLTT
CLTT = Closed Loop Thermal Throttling? I'd add the full form in parentheses: […]
Done
https://review.coreboot.org/c/coreboot/+/51824/comment/db03c1e1_a9afefb7 PS5, Line 152: - ProcHot
PROCHOT# is a signal that is used to report "Processor Hot", and activate thermal protections. […]
CLTT is thermal protection for DIMM modules, PROCHOT is thermal protection for processor. There are thermal sensors on them. When overheating is detected, the DIMM/processor frequency will be reduced to protect the hardware.