build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39003 )
Change subject: [WIP]drivers: Replace multiple fill_fb_framebuffer with single instance ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39003/1/src/device/oprom/realmode/x... File src/device/oprom/realmode/x86.c:
https://review.coreboot.org/c/coreboot/+/39003/1/src/device/oprom/realmode/x... PS1, Line 371: if (!fb_info) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/39003/1/src/device/oprom/yabel/vbe.... File src/device/oprom/yabel/vbe.c:
https://review.coreboot.org/c/coreboot/+/39003/1/src/device/oprom/yabel/vbe.... PS1, Line 755: if (!fb_info) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/39003/1/src/drivers/xgi/common/xgi_... File src/drivers/xgi/common/xgi_coreboot.c:
https://review.coreboot.org/c/coreboot/+/39003/1/src/drivers/xgi/common/xgi_... PS1, Line 363: if (!fb_info) { braces {} are not necessary for single statement blocks