Patrick Georgi 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:
We should also keep in mind, that the current code structure requires Oprom execution. To my understanding it is currently impossbile to get a bootsplash together with native graphics init.
That's because this code predates native graphics init by 8 years or so and the NGI devs didn't bother to update it ;-)
So if we were to keep it, maybe it should be moved to src/lib or something alike and call it from all the framebuffer initialization codes.
Sounds good. do you want to do it, or shall I take over?
How are usually decisions like this worked out?
Discussion on the mailing list, or a patch to review on Gerrit. In either case it helps to avoid terms like "broken", "most likely never worked", "no more use" though unless you're _really_, _really_ sure about that.
As an aside, the whole coreboot ecosystem isn't SeaBIOS and Tianocore. In fact, those are the newcomers.