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

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/47655 )

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

Patch Set 5:

(3 comments)

https://review.coreboot.org/c/flashrom/+/47655/4/it85spi.c 
File it85spi.c:

https://review.coreboot.org/c/flashrom/+/47655/4/it85spi.c@250 
PS4, Line 250:         unsigned int i;
             : 
             :         struct it85spi_data *data;
> keep the declarations together. So […]

Done

https://review.coreboot.org/c/flashrom/+/47655/4/it85spi.c@318 
PS4, Line 318:         data->ce_high = 0;
             :         data->ce_low = 0;
>

Done

https://review.coreboot.org/c/flashrom/+/47655/4/it85spi.c@326 
PS4, Line 326: {
             : 		return 1;
> `free(data);` before the return, it was correct before.

Done