Attention is currently required from: melvyn2. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/58774 )
Change subject: chipset_enable.c: Mark Intel Z390 as DEP ......................................................................
Patch Set 3: Code-Review+1
(3 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/58774/comment/e0771009_10192c29 PS3, Line 9: me_cleaner Minor: Our guidelines state that commit message lines wrap at 72 characters [1]. `me_cleaner` goes past the limit, so it should go in a new line.
[1]: https://flashrom.org/Development_Guidelines#Commit_message
https://review.coreboot.org/c/flashrom/+/58774/comment/9611dc8c_d84dcd52 PS3, Line 12: melvyn2 As per our Sign-off procedure [1], I'm afraid you have to use your real name here.
[1]: https://flashrom.org/Development_Guidelines#Sign-off_Procedure
Patchset:
PS3: Welcome! The change itself looks good, but the sign-off needs to be addressed.