[coreboot-gerrit] Change in coreboot[master]: psp_verstage: initialize i2c in soc_init

[Furquan Shaikh (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Kangheui Won, Karthik Ramasubramanian, Felix Held.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55136 )

Change subject: psp_verstage: initialize i2c in soc_init
......................................................................

Patch Set 3: Code-Review+1

(2 comments)

Patchset:

PS3: 
LGTM after moving the function to psp_verstage/fch.c. Also, let's see what others think about the change.

File src/soc/amd/picasso/psp_verstage/chipset.c:

https://review.coreboot.org/c/coreboot/+/55136/comment/2ba42db7_2b487af5 
PS2, Line 31: 		espi_setup();
> Yeah let's make it consistent.

I think we can keep this function now in psp_verstage/fch.c in common/block, right? Sorry, I wrote chipset.c incorrectly before. I meant that both Picasso and Cezzane do the exact same thing and so the verstage_soc_init() can live in psp_verstage/fch.c.