[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block: Enable PMC IPC driver

[Brandon Breitenstein (Code Review)]

Brandon Breitenstein has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42077 )

Change subject: soc/intel/common/block: Enable PMC IPC driver
......................................................................

Patch Set 6:

(3 comments)

https://review.coreboot.org/c/coreboot/+/42077/6/src/soc/intel/common/block/... 
File src/soc/intel/common/block/pmc/pmclib.c:

https://review.coreboot.org/c/coreboot/+/42077/6/src/soc/intel/common/block/... 
PS6, Line 44: */
> nit: space before the comment closing

Done

https://review.coreboot.org/c/coreboot/+/42077/6/src/soc/intel/common/block/... 
PS6, Line 47:
> one more tab to line up with line above

Done

https://review.coreboot.org/c/coreboot/+/42077/6/src/soc/intel/common/block/... 
PS6, Line 624: enum cb_err
> nit: no need to use cb_err for static functions, it's more for exported APIs

Done