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

[Nico Huber (Code Review)]

Nico Huber 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 5:

I don't understand the arguments against overriding
dcache_clean_all(). If a weak function isn't the way to
go (I also see it like that), one can still use other
mechanisms (weak functions mostly just spare us the
addition of a Kconfig symbol).

Alternatively, we could just guard the call to it. Either
way, we have `rules.h` do advertise properties of the
environment. Why not add an ENV_USER_SPACE and then
don't try to clear caches if it's set?