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

[caveh jalali (Code Review)]

caveh jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37871 )

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

Patch Set 13:

(4 comments)

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

https://review.coreboot.org/c/coreboot/+/37871/13/src/soc/intel/common/block... 
PS13, Line 50: 16
sizeof(pmc_ipc_buffer)?

https://review.coreboot.org/c/coreboot/+/37871/13/src/soc/intel/common/block... 
PS13, Line 634: struct
const...
better yet, does cmd need to be passed by reference at all?

https://review.coreboot.org/c/coreboot/+/37871/13/src/soc/intel/common/block... 
PS13, Line 634: uint32_t
const...

https://review.coreboot.org/c/coreboot/+/37871/13/src/soc/intel/common/block... 
PS13, Line 636: uintptr_t
looks like pmcbase is being cast to (void *) everywhere
it is used, might as well define it as such.