[flashrom-gerrit] Change in flashrom[master]: pickit2_spi.c: Separate shutdown from failed init cleanup

[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/+/52773 )

Change subject: pickit2_spi.c: Separate shutdown from failed init cleanup
......................................................................

Patch Set 1: Code-Review+2

(1 comment)

File pickit2_spi.c:

https://review.coreboot.org/c/flashrom/+/52773/comment/deb669b6_f2664fc6 
PS1, Line 482: 		goto init_err_cleanup_exit;
Just one comment to make this review not too boring: This actually changes
behavior. The original version failed to call pickit2_shutdown() if its
registration failed. It's a minor thing in this case (why would registration
fail?). Generally it's prefered though to focus on the functional changes
in the commit message (or just have them in a separate change).