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

[Angel Pons (Code Review)]

Angel Pons 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: Code-Review+1

(2 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

https://review.coreboot.org/c/coreboot/+/42077/6/src/soc/intel/common/block/... 
PS6, Line 610: 	const uintptr_t pmcbase = soc_read_pmc_base();
             : 	return (void *)(pmcbase + pmc_reg_offset);
> I think `return (soc_read_pmc_base() + pmc_reg_offset);` would be enough, the cast to `void *` is no […]

The cast is because readX / writeX functions take a pointer