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

[Maxim Polyakov (Code Review)]

Attention is currently required from:  Felix Singer, Lance Zhao, Nico Huber, Johnny Lin, Angel Pons, Arthur Heymans, Michael Niewöhner, Patrick Rudolph.
Maxim Polyakov 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 19:

(1 comment)

File util/inteltool/inteltool.c:

https://review.coreboot.org/c/coreboot/+/35919/comment/f0d489cc_781ce618 
PS18, Line 521: <start-end>
> > Technically `<start-end>` already covers the single-core case. Because […]

Ok, I see. Thanks for your comments.

In this case, I would prefer the shorter option:
--cpu-range <start>[-<end>] (this fits perfectly into the current boundaries in
the usage help).

And yes, you are right, we are not immune from the case when the second
parameter is incorrect: 1-qwerty, or even the first - 1qwerty (both cases will
return 1 for us)...
If we want to avoid this, for example, we can calculate the number of characters
for cores_range_start and compare this with the length of the optarg or use
something better than sscanf. But I don't think it needs to be complicated.