Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34518 )
Change subject: src/device/oprom src/lib: Remove broken bootsplash display code ......................................................................
Patch Set 6:
Patch Set 6:
If the resulting fix is a flickering 100ms of the (coreboot rendered) bootsplash being displayed until payload takes over and possibly changes to different videomode, will you call that a 'fix'.
It is more than 100ms tbh - but it has a visible "appearing" effect on the screen, which should be accounted for, if we want to keep and fix this feature..
Also: SeaBios drawing over it is resulting in ugly flicker..
I agree that if you're using SeaBIOS as your payload, this probably isn't the best option, but not everyone uses SeaBIOS, and not every payload resets the screen immediately.
I also agree that it should be fixed if it's broken and there's someone willing to maintain it.
We can add a note saying that if you're using SeaBIOS, not to use that option, to use the SeaBIOS splash screen. Or we can add a configurable delay so that people can see the coreboot splash screen for a sufficient amount of time if your complaint is that it goes by too fast. Or we can run a payload that doesn't clear the screen.
What I don't want is to just delete it without discussion. If nobody's willing to step up for this feature, then I'll stop fussing.