Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32376 )
Change subject: superio/fintek/f71808a: Add more optional ramstage registers ......................................................................
Patch Set 4:
(2 comments)
Thank you for your patch.
You should not score your own changes.
https://review.coreboot.org/#/c/32376/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32376/4//COMMIT_MSG@9 PS4, Line 9: More registers added and made optional so they keep untouched/default if omitted. Please wrap the line after 72/75 characters.
https://review.coreboot.org/#/c/32376/4//COMMIT_MSG@9 PS4, Line 9: added and made Summaries can be written in present tense.
Add more registers and make them optional, so they keep untouched/ their default if omitted.