Change in flashrom[master]: stlinkv3_spi.c: Refactor singleton states into reentrant pattern

[Nico Huber (Code Review)]

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

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

Patch Set 1:

(1 comment)

File stlinkv3_spi.c:

https://review.coreboot.org/c/flashrom/+/54042/comment/1e755dd2_49382243 
PS1, Line 533: stlinkv3_spi_open(sck_freq_kHz, stlinkv3_handle
> +1

How is this not actionable?

Before this commit, the handle was closed unless register_shutdown()
failed.

After this commit, the handle is closed unless calloc() failed or
register_shutdown() failed. A nit, but still a regression. And blaming
the existing code for it seems odd.