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

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Furquan Shaikh, Rizwan Qureshi, Subrata Banik, Angel Pons, Patrick Rudolph.
Tim Wawrzynczak 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.:

```
struct cpu_cache_info {
    unsigned int type;
    unsigned int level;
    unsigned int num_ways;
    unsigned int num_sets;
    unsigned int line_size;
    unsigned int size;
    unsigned int physical_partitions;
    unsigned int num_cores_shared;
    bool fully_associative;
};
```

or something similar? Then there can just be a function that maybe accepts a pointer to that type and all of the helpers and caching goes away, e.g.:

```
void fill_cpu_cache_info(struct cpu_cache_info *info)
{
     struct cpuid_result result;
     if (!info)
          return;

get_leaf();
     info->type = ...
     etc;
}
```

WDYT?