[coreboot-gerrit] Change in coreboot[master]: src/device/oprom: Fix bootsplash display code for optionroms

[Johanna Schander (Code Review)]

Johanna Schander has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34537 )

Change subject: src/device/oprom: Fix bootsplash display code for optionroms
......................................................................

Patch Set 23:

(4 comments)

https://review.coreboot.org/c/coreboot/+/34537/22//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34537/22//COMMIT_MSG@10 
PS22, Line 10: set up as 1024x768@16.
> Sorry, can you please add what happens if a different resolution/depth is used?

Done

https://review.coreboot.org/c/coreboot/+/34537/22//COMMIT_MSG@17 
PS22, Line 17: This was testend on the wip razer blade stealth using the intel vgabios.
> What resolution did you test with?

Done

https://review.coreboot.org/c/coreboot/+/34537/22//COMMIT_MSG@17 
PS22, Line 17: testend
> tested

Done

https://review.coreboot.org/c/coreboot/+/34537/23/src/device/oprom/yabel/vbe... 
File src/device/oprom/yabel/vbe.c:

https://review.coreboot.org/c/coreboot/+/34537/23/src/device/oprom/yabel/vbe... 
PS23, Line 38: #include <boot/coreboot_tables.h>
> I think it's not needed anymore

Done