[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Split I2c implementation for psp_verstage

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42064 )

Change subject: soc/amd/picasso: Split I2c implementation for psp_verstage
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/42064/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42064/5//COMMIT_MSG@7 
PS5, Line 7: soc/amd/picasso: Split I2c implementation for psp_verstage
> i2c or I2C. First is more readable to my eyes.

Done

https://review.coreboot.org/c/coreboot/+/42064/9/src/soc/amd/picasso/Makefil... 
File src/soc/amd/picasso/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/42064/9/src/soc/amd/picasso/Makefil... 
PS9, Line 48: verstage-y
> Could make this verstage_arm-y, and verstage_x86-y

I'm going to look at recombining these into a single file in a follow-on patch.  If I can't, i'll update this to be cleaner then.