Attention is currently required from: Martin Roth, Marshall Dawson, Felix Held. ritul guru has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50901 )
Change subject: mb/amd/bilby: updating EC FW specific options for bilby ......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/50901/comment/d6fee177_1a615c58 PS1, Line 7: \
for path separators we use / and not . […]
Ack
File src/mainboard/amd/bilby/Kconfig:
https://review.coreboot.org/c/coreboot/+/50901/comment/5679d595_231a759a PS1, Line 50: if BOARD_AMD_BILBY
since there aren't multiple variants, you can remove this condition; the condition will be always tr […]
Ack
https://review.coreboot.org/c/coreboot/+/50901/comment/98865358_80bf0bb5 PS1, Line 68: config AMD_FWM_POSITION_INDEX
this option can be removed with the EC firmware section being removed, since now the default positio […]
Ack