[S] Change in coreboot[main]: arch/x86/cpu: Add helper function to compute cache

[Felix Held (Code Review)]

Attention is currently required from: Andy Ebrahiem, Angel Pons, Jincheng Li, Jérémy Compostella, Maximilian Brune, Naresh Solanki, Patrick Rudolph, Shuo Liu.

Felix Held has posted comments on this change by Naresh Solanki. ( https://review.coreboot.org/c/coreboot/+/85638?usp=email )

Change subject: arch/x86/cpu: Add helper function to compute cache
......................................................................

Patch Set 7: Code-Review+1

(1 comment)

File src/arch/x86/cpu_common.c:

https://review.coreboot.org/c/coreboot/+/85638/comment/6d6b0168_30e471ab?usp... :
PS7, Line 215: bool __weak soc_fill_cpu_cache_info(uint8_t level, struct cpu_cache_info *info)
hmm, i wonder if it would be better to have a kconfig option to replace the common implementation by a soc-specific one instead of having a weak function that gets overridden.
in soc/amd we tried to minimize the usage of weak functions, since in the kconfig symbol case, mismatches result in linker errors instead of unexpected behavior.