[coreboot-gerrit] Change in coreboot[master]: intel/common/block: Add gpmr common driver

[Paul Menzel (Code Review)]

Attention is currently required from: Jamie Ryu, Wonkyu Kim, Subrata Banik, Ethan Tsao, Ravishankar Sarawadi, Angel Pons, Nick Vaccaro, Tim Wawrzynczak.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63170 )

Change subject: intel/common/block: Add gpmr common driver
......................................................................

Patch Set 14:

(3 comments)

File src/soc/intel/common/block/gpmr/gpmr.c:

https://review.coreboot.org/c/coreboot/+/63170/comment/d1594402_cbed09a7 
PS13, Line 39: 	printk(BIOS_ERR, "%s: No available free gpmr found\n", __func__);
Maybe also log MAX_GPMR_REGS.

https://review.coreboot.org/c/coreboot/+/63170/comment/428343a2_f2516b6e 
PS13, Line 44: uint32_t
> We want to have fixed size(32bit)

Why?

https://review.coreboot.org/c/coreboot/+/63170/comment/42ccfb49_a767d96d 
PS13, Line 46: int
> This should be int as get_available_gpmr can return -1 (error case)

Indeed, but maybe a better name can be found? `num` is misleading at list to me.