Change in flashrom[master]: ichspi.c: Check SPI Cycle In-Progress prior start HW Seq

[Edward O'Callaghan (Code Review)]

Attention is currently required from: Subrata Banik.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/61854 )

Change subject: ichspi.c: Check SPI Cycle In-Progress prior start HW Seq
......................................................................

Patch Set 17:

(1 comment)

Patchset:

PS17:
> I know I can react much too harsh sometimes (it's just frustrating to see […]

Thanks Nico,

I just wanted to follow up here quickly to ensure that messaging does not get mixed up and this is going to get followed up on! Tomorrow is some company-wide day off.

The short answer just for the moment is that Subrata and I were discussing today if a possible unit-test using HECI command set that could perhaps reliably exploit the issue and how it is remediated by this patch can be contrived to make this more concrete.

I also followed up with Anastasia to help assist in making these changes smoother and she has provided some very useful insight to.

More to come but I just wanted to follow up with something here ASAP. Leaving as unresolved.