[coreboot-gerrit] Change in ...coreboot[master]: smbios: Add and adapt fields for version 3.2

[Francois Toguo Fotso (Code Review)]

Francois Toguo Fotso has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32090 )

Change subject: smbios: Add and adapt fields for version 3.2
......................................................................

Patch Set 5:
> Patch Set 1:
>
> (1 comment)
>
> Sorry for the revert. Please, split out the functional
> changes (e.g. the cpuid stuff around core/thread count)
> into their own patch (it's not related to the SMBIOS
> update, is it?). This change does a lot of things, and
> I haven't spotted anything yet that is *required* for
> SMBIOS 3.2.

Nico,
From the 3.2 spec, core counts and thread counts don't seem to be optional.
The newly added procedure is populating them to comply to one of your previous comment:
" please don't add fields without giving them a reasonable value"
I can remove it from this patch if you require it and set the values to 0. Because the count generated by the code before this upgrade was not accurate, at least not on the GKL Sparky platform.

Regards,
Francois