[coreboot-gerrit] Change in coreboot[master]: nb/via/vx900: Ensure framebuffer size is within limits

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34355 )

Change subject: nb/via/vx900: Ensure framebuffer size is within limits
......................................................................

Patch Set 4: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/34355/3/src/northbridge/via/vx900/m... 
File src/northbridge/via/vx900/memmap.c:

https://review.coreboot.org/c/coreboot/+/34355/3/src/northbridge/via/vx900/m... 
PS3, Line 83: 	assert(size_mb != 0);
> The rest of the code technically rounds down to the nearest power of 2, so I suppose that comment is […]

I have never bothered to check or test, but I think asserts don't get a print in console the way die() does.