[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Move GPIO PM configuration to soc level

[EricR Lai (Code Review)]

EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37685 )

Change subject: soc/intel/cannonlake: Move GPIO PM configuration to soc level
......................................................................

Patch Set 13:

(2 comments)

https://review.coreboot.org/c/coreboot/+/37685/13/src/soc/intel/cannonlake/a... 
File src/soc/intel/cannonlake/acpi/lpit.asl:

https://review.coreboot.org/c/coreboot/+/37685/13/src/soc/intel/cannonlake/a... 
PS13, Line 80: OGPM
> I don't understand why we need to have NVS now? Can't we just always follow the policy of saving and […]

@Furquan, this is Tim wanted. I think it okay to let mainboard can choice they want this or not.

https://review.coreboot.org/c/coreboot/+/37685/13/src/soc/intel/cannonlake/a... 
PS13, Line 100: OGPM
> Same question as above.

Ack