Change in coreboot[master]: soc/intel/common: implement ioc driver

[Subrata Banik (Code Review)]

Attention is currently required from: Jamie Ryu, Wonkyu Kim, Ethan Tsao, Ravishankar Sarawadi, Paul Menzel, Tim Wawrzynczak, Angel Pons, Nick Vaccaro.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63198 )

Change subject: soc/intel/common: implement ioc driver
......................................................................

Patch Set 12:

(3 comments)

File src/soc/intel/common/block/gpmr/gpmr.c:

https://review.coreboot.org/c/coreboot/+/63198/comment/15ed7711_f8a62a1e 
PS12, Line 27: 	uint32_t data32;
             : 
             : 	data32 = gpmr_read32(offset);
             : 	data32 |= ordata;
             : 	gpmr_write32(offset, data32);
> you mean ioc_or32?

yeah something like that, because you would using gpmr_or32 from lockdown.c file as well in MTL. isn't it ?

File src/soc/intel/common/block/ioc/ioc.c:

https://review.coreboot.org/c/coreboot/+/63198/comment/19a84250_ee04c5b0 
PS12, Line 15: read32
> Ack

you marked ACK but I don't see any change. am I missing anything ?

File src/soc/intel/common/pch/lockdown/lockdown.c:

https://review.coreboot.org/c/coreboot/+/63198/comment/1c81c937_61fbe311 
PS12, Line 32: GPMR_DMICTL
> Yes, With IOC interface, there is no CTL register and it does not need to do it.

hmm, this is the problem with common code then you need #if/#endif