[coreboot-gerrit] Change in coreboot[master]: cpu/intel/common: Create get_cache_info() function

[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: cpu/intel/common: Create get_cache_info() function
......................................................................

Patch Set 1:

(1 comment)

File src/cpu/intel/common/common_init.c:

https://review.coreboot.org/c/coreboot/+/55965/comment/16158e71_76d841aa 
PS1, Line 332: 
             : 	const size_t assoc = CPUID_CACHE_WAYS_OF_ASSOC(res) + 1;
             : 	const size_t partitions = CPUID_CACHE_PHYS_LINE(res) + 1;
             : 	const size_t cache_line_size = CPUID_CACHE_COHER_LINE(res) + 1;
             : 	const size_t number_of_sets = CPUID_CACHE_NO_OF_SETS(res) + 1;
             : 	const size_t cache_size = assoc * partitions * cache_line_size * number_of_sets;
             : 	printk(BIOS_INFO, "Cache Information : ");
             : 	printk(BIOS_INFO, "assoc=%zd par=%zd line_size=%zd sets=%zd\n",
             : 		assoc, partitions, cache_line_size, number_of_sets);
             : 	printk(BIOS_INFO, "@@ cache_size 0x%lx bytes\n", cache_size);
> > also remarkably similar to https://review.coreboot. […]

looking a little closer, I'm not sure if we'd SMBIOS (generic x86 code) to depend on soc/intel/common code, so maybe this function should go somewhere in arch/x86, maybe arch/x86/cpu_common.c ?