[coreboot-gerrit] Change in coreboot[master]: lib: Rewrite qemu-armv7 ramdetect

[Angel Pons (Code Review)]

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

Change subject: lib: Rewrite qemu-armv7 ramdetect
......................................................................

Patch Set 6: Code-Review+2

(2 comments)

Other than two small things, LGTM

https://review.coreboot.org/c/coreboot/+/33934/6/src/lib/ramdetect.c 
File src/lib/ramdetect.c:

https://review.coreboot.org/c/coreboot/+/33934/6/src/lib/ramdetect.c@22 
PS6, Line 22: mib
Minor: I'd suggest using a different name to avoid confusion with the 'MiB' constant

https://review.coreboot.org/c/coreboot/+/33934/6/src/lib/ramdetect.c@30 
PS6, Line 30: int
This could be an unsigned type, e.g. size_t?

Avoids having several signed/unsigned comparisons, since ARRAY_SIZE returns size_t