[coreboot-gerrit] Change in coreboot[master]: inteltool: Allow to set cores range for MSRs dump

[Angel Pons (Code Review)]

Attention is currently required from:  Felix Singer, Lance Zhao, Nico Huber, Maxim Polyakov, Johnny Lin, Arthur Heymans, Michael Niewöhner, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35919 )

Change subject: inteltool: Allow to set cores range for MSRs dump
......................................................................

Patch Set 18:

(1 comment)

File util/inteltool/inteltool.c:

https://review.coreboot.org/c/coreboot/+/35919/comment/65d24f3b_93319bd0 
PS18, Line 521: <start-end>
> Technically `<start-end>` already covers the single-core case. Because
> `start-end` is just a symbol here. So we could keep just that. I just
> noticed that we could also make the dash explicit.
>
> I guess any of these two would be more accurate:
> --cpu-range [<core> | <start>-<end>]
>
> --cpu-range <start>[-<end>]
>
> (Only almost accurate, because I assume sscanf() also accepts `<start>-` ;)
> maybe even any trailing garbage. But that's an implementation detail and not
> the expected syntax.)

+1