Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31841 )
Change subject: Documentation/soc/intel: Add MP Initialization document ......................................................................
Patch Set 3:
(6 comments)
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... File Documentation/soc/intel/mp_init/mp_init.md:
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 22: -
No need for a hyphen (-), should be a space instead
Done
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 26: Considering these facts
Maybe add a comma after this
Done
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 34:
please put a comma here
Done
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 35: the same
the same task/job
Done
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 35: make most
These two words can be removed
Done
https://review.coreboot.org/#/c/31841/3/Documentation/soc/intel/mp_init/mp_i... PS3, Line 36: Today
This can stop being true at some point. […]
Done