[coreboot-gerrit] [S] Change in coreboot[main]: cpu/intel/model_206ax: Print supported C-states

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Felix Held, Patrick Rudolph.

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78133?usp=email )

Change subject: cpu/intel/model_206ax: Print supported C-states
......................................................................

Patch Set 5:

(1 comment)

File src/arch/x86/cpu_common.c:

https://review.coreboot.org/c/coreboot/+/78133/comment/bf0caed9_282dd858 :
PS5, Line 196: int cpu_get_c_state_support(const int lvl)
> hmm, i'd make both return value and parameter unsigned integers, since those won't and shouldn't be  […]

Is the input parameter 'lvl' really a 'state'. And the return value is 'sub-state' while function name has 'state'?