[flashrom-gerrit] Change in flashrom[master]: usbblaster_spi.c: Refactor singleton states into reentrant pattern

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Anastasia Klimchuk.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/54044 )

Change subject: usbblaster_spi.c: Refactor singleton states into reentrant pattern
......................................................................

Patch Set 1:

(2 comments)

File usbblaster_spi.c:

https://review.coreboot.org/c/flashrom/+/54044/comment/23e52223_42e21f59 
PS1, Line 55: 	struct ftdi_context ftdic;
> Oh I had the same question in CB:52774 and I explained why I am doing this (in short, mostly so that […]

Ack

https://review.coreboot.org/c/flashrom/+/54044/comment/87f21f2b_1d35b965 
PS1, Line 185: 	struct ftdi_context ftdic;
> I usually try to allocate data as late as possible to simplify error paths (as soon as you allocate  […]

I believe this patch is fine, hence the +2. I don't have a strong opinion on this.