[M] Change in coreboot[main]: soc/intel: Use write{64,32,16,8}p and read{64,32,16,8}p

[Martin L Roth (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Dinesh Gehlot, Elyes Haouas, Eran Mitrani, Jeff Daly, Johnny Lin, Julius Werner, Kapil Porwal, Subrata Banik, Tarun, Tim Chu, Vanessa Eusebio.

Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80193?usp=email )

Change subject: soc/intel: Use write{64,32,16,8}p and read{64,32,16,8}p
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> Is this really a good idea? I think we intentionally made the `write32()` function take a `void *` ( […]

I'm with Julius on this. I think having just the write[8|16|32]() functions is preferable to having one that does the cast automatically. Let's get rid of the p versions.