Attention is currently required from: Felix Singer, Jason Glenesk, Martin L Roth.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71590 )
Change subject: Documentation/releases: Start getting ready for the 4.19 release ......................................................................
Patch Set 1: Code-Review+1
(7 comments)
File Documentation/releases/coreboot-4.19-relnotes.md:
https://review.coreboot.org/c/coreboot/+/71590/comment/84382603_c063b93d PS1, Line 13: relevent typo: relev*a*nt
https://review.coreboot.org/c/coreboot/+/71590/comment/84999c3b_d42392bc PS1, Line 42: optoins typo: options
https://review.coreboot.org/c/coreboot/+/71590/comment/d2e1cccc_0079cde6 PS1, Line 46: nit: remove one space
https://review.coreboot.org/c/coreboot/+/71590/comment/c26a9c26_aafd8033 PS1, Line 64: * AMD: Mayan for Morgana SoC : * GIGABYTE: GA-H61M-DS2 : * Google: Crystaldrift : * Google: Gladios : * Google: Marasov : * Google: Voltorb : * Intel: Meteorlake-P RVP : * Siemens: MC_EHL3 : * Star Labs: StarBook Mk VI (i3-1220P and i7-1260P) : * System76: darp8 : * System76: galp6 nit: Please be consistent with the spacing after the colon: some lines have one space, whereas others have two.
https://review.coreboot.org/c/coreboot/+/71590/comment/5af84833_a58bf6ff PS1, Line 114: * src/soc/intel/xeon_sp/lbg This isn't a new platform, it's just a result of restructuring the code as part of CB:70009. It'd be better to drop it.
https://review.coreboot.org/c/coreboot/+/71590/comment/d8655c73_da253576 PS1, Line 110: New SoCs : ------------------- : : * src/soc/amd/glinda : * src/soc/intel/xeon_sp/lbg : : : Removed processors : ------------------- : : * src/cpu/amd/agesa/family14 : * src/cpu/amd/agesa/family15tn : * src/cpu/amd/agesa/family16kb nit: drop the `src/` prefix?
https://review.coreboot.org/c/coreboot/+/71590/comment/c9656524_e3435097 PS1, Line 124: Also list dropped northbridges/southbridges?