[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Clean up stepping logic

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48408 )

Change subject: nb/intel/sandybridge: Clean up stepping logic
......................................................................

Patch Set 4: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/48408/3/src/cpu/intel/model_206ax/m... 
File src/cpu/intel/model_206ax/model_206ax.h:

https://review.coreboot.org/c/coreboot/+/48408/3/src/cpu/intel/model_206ax/m... 
PS3, Line 95: }
> Yeah, where can I place this instead?

There are already some functions around, e.g. one that decodes the
CPUID signature into family, modding, stepping, IIRC. It's not handy
if one just needs the stepping, though. In `arch/x86/include/arch/cpu.h`.