Agreed, this patch makes no sense at all.

If someone is building from my coreboot tree, then they can simply revert the patch there which was made to accommodate edk2 payload, and no change is needed to SeaBIOS.

On Tue, Feb 14, 2023 at 5:14 AM Peter Stuge <peter@stuge.se> wrote:
Hi Christopher,

christopherericlentocha@gmail.com wrote:
> tom Firmware
>
> while using the SeaBios payload with MrChromebox's SMM variable
> in coreboot. This is required even if we are booting off of a
> USB, since it will give MMC errors in Linux and Windows 10
> and 11 won't see it, though I didn't test 11, It would most
> likely do it since all Linux versions I tried did the same.

I'm merely a community member here but neither your commit message
nor the comments you add to source files explain to me what is being
done and why - only what the outcome is and that actually this
problem is much larger than your patch addresses and that your patch
is only applicable in certain circumstances (and harmful in others?).

That will likely not be accepted into the codebase.

It's completely okay to post the patch on this list anyway.

But if you want the problem you experience to actually be fixed
correctly/thoroughly then I think this issue needs much more work.

Is that your ambition?


Thanks and kind regards

//Peter
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org