[coreboot-gerrit] Change in coreboot[master]: soc/intel: Add get_pmbase

[Name of user not set (Code Review)]

[e-mail] has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37990 )

Change subject: soc/intel: Add get_pmbase
......................................................................

Patch Set 4:

(3 comments)

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/apollolake/pm... 
File src/soc/intel/apollolake/pmutil.c:

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/apollolake/pm... 
PS3, Line 251:
> Done

Done

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/cannonlake/pm... 
File src/soc/intel/cannonlake/pmutil.c:

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/cannonlake/pm... 
PS3, Line 277:
> Please, no blank line here either.

Done

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/quark/acpi.c 
File src/soc/intel/quark/acpi.c:

https://review.coreboot.org/c/coreboot/+/37990/3/src/soc/intel/quark/acpi.c@... 
PS3, Line 113: 		& B_QNC_LPC_PM1BLK_MASK;
> It was changed to 96 characters at some point.

changed the second line to comply