[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 3:

(2 comments)

Will issue a new patch set once I get the smm_segs.h issue resolved.

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:
> No blank line.

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;
> Do both lines fit into 96 character line?

No. The first line of code is 101 characters and the second is 92 characters.

So far during the early part of the review the BOT has been complaining when any line of code is over 80 characters.  So, I made a big effort to ensure that everything is under 80 lines.

Is this a change?