Change in coreboot[master]: soc/intel/common: add generic gpio lock mechanism

[Nick Vaccaro (Code Review)]

Attention is currently required from: Bora Guvendik, Caveh Jalali, Selma Bensaid, Maulik V Vaghela, Tim Wawrzynczak, Subrata Banik, Patrick Rudolph, Karthik Ramasubramanian.
Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58351 )

Change subject: soc/intel/common: add generic gpio lock mechanism
......................................................................

Patch Set 7:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/58351/comment/e93d4523_29c5ef3a 
PS7, Line 461: int gpio_lock_multiple_pads(const struct gpio_lock_config *pads_a, const size_t count_a,
             : 		const struct gpio_lock_config *pads_b, const size_t count_b)
> this is kind of an awkward API to use... […]

I had thought about that as well, seems much more straight-foward to have the routine support locking a single list.

However, I would suggest just calling it twice (once for each list, or if only one list exists, it will only get called once) instead of the malloc - catenate lists - free approach.

I *think* the overhead would be less and make for a simpler approach.  Thoughts?