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

[Nico Huber (Code Review)]

Nico Huber 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 6: Code-Review+1

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34355/6/src/northbridge/via/vx900/m... 
PS6, Line 85: 	u8 fb_pow = 0;
            : 	do {
            : 		fb_pow++;
            : 	} while (size_mb >> fb_pow);
            : 	fb_pow--;
            : 
            : 	size_mb = (1U << fb_pow);
Nit, with the die() in place, it should be the same as

size_mb = 1u << log2(size_mb);