[coreboot-gerrit] Change in coreboot[master]: arch/x86: Helper functions to get deterministic cache parameters

[Subrata Banik (Code Review)]

Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Rizwan Qureshi, Angel Pons, Patrick Rudolph.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55965 )

Change subject: arch/x86: Helper functions to get deterministic cache parameters
......................................................................

Patch Set 4:

(1 comment)

Patchset:

PS4:
> I wonder if having a struct that encompasses this information would lead to a little less code, e,g. […]

This is really bice suggestion, i was thinking to create smaller sets of function where things are independent enough and caller can call those routines based on need. But your point is also valid, that, there may be less chance that some one would like to know what is number of cache sets or its being fully associative or not, so its better to fill all those CPUID EAX=leaf, ECX={0/1/2/3..} output into a single structure as pass to the caller to use it wisely ?