[S] Change in coreboot[master]: drivers/smmstore/ramstage.c: retry smmstore init up 5 times

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Sean Rhodes, Michał Żygowski, Patrick Rudolph, Paul Menzel, Michał Kopeć, Krystian Hebel.

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73981 )

Change subject: drivers/smmstore/ramstage.c: retry smmstore init up 5 times
......................................................................

Patch Set 1:

(1 comment)

File src/drivers/smmstore/ramstage.c:

https://review.coreboot.org/c/coreboot/+/73981/comment/96e0dbb7_86f2b538 
PS1, Line 79: 	} while (retry--);
> I'm afraid this can't be done, even `mdelay()` should be removed from the code. […]

Maybe CB:74741 raise_io_smi() would work here. Sounds like the polling should not be hidden under drivers/smmstore. There's no retry of outb() yet, just the noop/pause part. I wonder what the argumentation was to repeat the outb() ?