[coreboot-gerrit] Change in ...coreboot[master]: src/mainboard/lenovo/g505s: Disable SeaBIOS options not supported by ...

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31357 )

Change subject: src/mainboard/lenovo/g505s: Disable SeaBIOS options not supported by hardware
......................................................................

Patch Set 6:
> Patch Set 6:
>
> Sadly there is no progress in the review of G505S patches, despite their great significance:

You do reviews; you get reviews. And don't get offended when your particular solution is not merged as-is.

I thought there was a conclusion AMD / AtomBIOS only has to be loaded (from OS driver perspective), and the memory location of said blob filled in ACPI. If your proposal always forced running them, that's probably the reason for no-further-review and no merge, if that's what happened.

Did I miss work that only loads the discrete AtomBIOS tables?