Attention is currently required from: Edward O'Callaghan.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/66674 )
Change subject: chipset_enable.c: plumb programmer_cfg into sb600 and ich init entry-points ......................................................................
Patch Set 9: Code-Review+2
(1 comment)
Commit Message:
https://review.coreboot.org/c/flashrom/+/66674/comment/2ed21b0d_035e9662 PS9, Line 7: chipset_enable.c: plumb programmer_cfg into sb600 and ich init entry-points This seems longer than 72 chars (you can see it is wrapped in gerrit). Maybe drop "entry-points" at the end?
If you want to explain more, you can add one paragraph in commit message. Although IMO commit title is already clear (even without "entry-points").