build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39430 )
Change subject: edid_fill_fb: Add row_alignment argument ......................................................................
Patch Set 5:
(4 comments)
https://review.coreboot.org/c/coreboot/+/39430/5/src/device/oprom/realmode/x... File src/device/oprom/realmode/x86.c:
https://review.coreboot.org/c/coreboot/+/39430/5/src/device/oprom/realmode/x... PS5, Line 370: uint32_t row_alignment = fb_helper_get_row_alignement( 'alignement' may be misspelled - perhaps 'alignment'?
https://review.coreboot.org/c/coreboot/+/39430/5/src/device/oprom/yabel/vbe.... File src/device/oprom/yabel/vbe.c:
https://review.coreboot.org/c/coreboot/+/39430/5/src/device/oprom/yabel/vbe.... PS5, Line 754: uint32_t row_alignment = fb_helper_get_row_alignement( 'alignement' may be misspelled - perhaps 'alignment'?
https://review.coreboot.org/c/coreboot/+/39430/5/src/include/framebuffer_inf... File src/include/framebuffer_info.h:
https://review.coreboot.org/c/coreboot/+/39430/5/src/include/framebuffer_inf... PS5, Line 45: * To be used by drivers that don't know the hardware dependend alignment, for example the 'dependend' may be misspelled - perhaps 'dependent'?
https://review.coreboot.org/c/coreboot/+/39430/5/src/include/framebuffer_inf... PS5, Line 48: static inline uint32_t fb_helper_get_row_alignement(uint32_t scanline_bytes, 'alignement' may be misspelled - perhaps 'alignment'?