[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Add Lewisburg defs for common/gpio driver

[Andrey Petrov (Code Review)]

Andrey Petrov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39425 )

Change subject: soc/intel/xeon_sp: Add Lewisburg defs for common/gpio driver
......................................................................

Patch Set 11:

hey Maxim, thanks for doing this! This is really useful. No wait. This is incredible.

I gave it a test on Cooperlake-SP CRB that has C62x. I used gpio_get() in loop to query all the pins.
Some came as 1 some are 0. I checked these 1s against schematics and they match pins that are pulled up. So it not a coincidence.

Unfortunately I do not have physical access to the board (that virus thing that everybody is talking about), so for real-world verification (e.g short a pin or blink a LED) we should try on something we have physical access to. I will try to arrange testing on TiogaPass