[coreboot-gerrit] Change in coreboot[master]: drivers: Replace set_vbe_mode_info_valid

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39004 )

Change subject: drivers: Replace set_vbe_mode_info_valid
......................................................................

Patch Set 29:

(3 comments)

https://review.coreboot.org/c/coreboot/+/39004/28//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39004/28//COMMIT_MSG@24 
PS28, Line 24: The platforms doesn't read an EDID anyway.
> Some do, though.

Right, I squashed two commits, so this is no longer true.

https://review.coreboot.org/c/coreboot/+/39004/28//COMMIT_MSG@24 
PS28, Line 24: doesn't
> don't

Done

https://review.coreboot.org/c/coreboot/+/39004/28/src/drivers/aspeed/common/... 
File src/drivers/aspeed/common/ast_mode_corebootfb.c:

https://review.coreboot.org/c/coreboot/+/39004/28/src/drivers/aspeed/common/... 
PS28, Line 227: 				edid.framebuffer_bits_per_pixel);
> Why not use _from_edid() here?

Done