[coreboot-gerrit] Change in coreboot[master]: nb/via/vx900: Assert 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: Assert framebuffer size is within limits
......................................................................

Patch Set 1:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34355/1/src/northbridge/via/vx900/m... 
PS1, Line 82: int
> If size_mb is somehow zero, then fb_pow will underflow in the subtraction to 255. […]

I'd just catch the zero separately, since it's rather significant (how do you end up with size_mb = 0 other than with some serious issue in the previous code?)