[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Don't run dcache_clean_all() in psp_verstage

[Eric Peers (Code Review)]

Eric Peers has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43678 )

Change subject: soc/amd/picasso: Don't run dcache_clean_all() in psp_verstage
......................................................................

Patch Set 4: Code-Review+1

I considered another option over chat with Martin. Why not make the dcache_clean_all the weak function?

The benefit: One less layer
The cost: PSP has it permanently defined to do nothing. We might actually want such a function to invalidate a PSP cache. Name of function also doesn't follow actual implementation which is confusing.