Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42947 )
Change subject: doc/mb/ocp: Add documentation for Delta Lake ......................................................................
Patch Set 2:
(13 comments)
Thanks!
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... File Documentation/mainboard/ocp/deltalake.md:
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 11: CooperLake
Cooper Lake
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 17: Linuxboot
LinuxBoot
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 22: CooperLake
Cooper Lake
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 23: MP
Mass Production I believe, it's a standard term of art.
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 29: Linuxboot
LinuxBoot
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 29: Linuxboot
LinuxBoot
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 50: Linuxboot
LinuxBoot
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 51: versions
version
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 50: The solution is developed using Linuxboot payload. The Linuxboot : kernel versions tried is 5.2.9.
The solution is developed using LinuxBoot payload with Linux kernel 5.2. […]
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 62: - Skipping memory training upon subsequent reboots
… by using MRC cache.
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 73: its
that
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 81: with Intel.
Use two spaces for consistency instead of tab?
Done
https://review.coreboot.org/c/coreboot/+/42947/2/Documentation/mainboard/ocp... PS2, Line 86: that
so
Done