Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37424 )
Change subject: Remove the non-working bayou payload; clean up payloads ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/37424/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37424/1//COMMIT_MSG@10 PS1, Line 10: payloads/Kconfig references not existing Kconfig files Which ones?
https://review.coreboot.org/c/coreboot/+/37424/1//COMMIT_MSG@9 PS1, Line 9: There are coding issues, there are things that don't : build correctly, a little more detail would be useful (or patches that fix those issues ;)
https://review.coreboot.org/c/coreboot/+/37424/1//COMMIT_MSG@18 PS1, Line 18: If a payload is so unloved nobody has looked at it : in years, much less used it, it is time for it to go. We should drop everything but edk2, seabios, depthcharge and grub2. Possibly drop seabios, too.
https://review.coreboot.org/c/coreboot/+/37424/1//COMMIT_MSG@21 PS1, Line 21: Bayou was a wonderful idea, and I'm sad to see it go, but ... oh well I believe Simon Glass reimplemented something like this, too.
There's a difference between payloads maintained in the repo (coreinfo and bayou) and outside (everything else). Maybe we want to give a little more love to the ones we maintain?
https://review.coreboot.org/c/coreboot/+/37424/1/3rdparty/fsp File 3rdparty/fsp:
https://review.coreboot.org/c/coreboot/+/37424/1/3rdparty/fsp@a1 PS1, Line 1: Subproject commit 59964173e18950debcc6b8856c5c928935ce0b4f This should not be part of this commit.