[coreboot-gerrit] Change in coreboot[master]: inteltool: add Lewisburg C62x GPIOs support

[Maxim Polyakov (Code Review)]

Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34942 )

Change subject: inteltool: add Lewisburg C62x GPIOs support
......................................................................

Patch Set 15:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34942/7/util/inteltool/gpio.c 
File util/inteltool/gpio.c:

https://review.coreboot.org/c/coreboot/+/34942/7/util/inteltool/gpio.c@1050 
PS7, Line 1050: _PROD
> I would remove the "PROD" suffix, as there's no naming clash with other parts.

Done

https://review.coreboot.org/c/coreboot/+/34942/7/util/inteltool/pcr.c 
File util/inteltool/pcr.c:

PS7:
> I guess the Super SKU is an unfused chip, with all the silicon features enabled. […]

Yes, I remember these wonderful AMD processors with 3 cores in the AM3 socket. And one more core could be unlocked :)
Ok, Thanks