Change in flashrom[master]: pony_spi.c: Extract out get_params to simplify init

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan, Light, Anastasia Klimchuk.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63130 )

Change subject: pony_spi.c: Extract out get_params to simplify init
......................................................................

Patch Set 3:

(1 comment)

File pony_spi.c:

https://review.coreboot.org/c/flashrom/+/63130/comment/ac5dcf12_bccea0c8 
PS3, Line 167: bool have_device;
> A bit of nit isn't it? Is there a solid technical reason here or just personal preference?

From my point of view: Review doesn't scale well with the size of
a patch. Even if it's a very simple patch hunk, every additional
hunk can distract from the error-prone part of a patch. Doing fixes
and simple changes like this in advance works well, FWIW.