Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38905 )
Change subject: mainboard: drop 5 broken ROMCC_BOOTBLOCK boards ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38905/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38905/1//COMMIT_MSG@7 PS1, Line 7: broken
unmaintained
"Unmaintained" isn't a good enough reason for removal: there's always a chance someone stumbles upon a board, discovers that it's supported by coreboot (of course if it hasn't been dropped!) and joins us - maybe helping with the common code as well (btw, G505S and AM1I-A technically were unmaintained before I signed up as their maintainer). Meanwhile, "broken" is a much stronger reason: if we couldn't get them built in a reasonable time, means a newcomer would fail as well - so no benefit of keeping them aside for the reference.
https://review.coreboot.org/c/coreboot/+/38905/1//COMMIT_MSG@9 PS1, Line 9: The source code of these 5 boards can't be successfully built after a : simple switch away from ROMCC_BOOTBLOCK which we did for many other : AMD boards. See my failed attempts at CB:37976, CB:37979, CB:37980, : CB:37981 and CB:37982. Unless someone is really interested in fixing : and if their code doesn't hold any other value, they could be dropped.
remove
I needed to provide an explanation why I remove these 5 boards but not removing the others.