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

[Michał Kopeć (Code Review)]

Attention is currently required from: Krystian Hebel, Michał Żygowski, Patrick Rudolph, Paul Menzel, Tim Crawford.

Michał Kopeć has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73981?usp=email )

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

Patch Set 2:

(2 comments)

Patchset:

PS2: 
Thanks for the review guys, I am finally getting around to addressing review under my CLs.

Rebased and switched to calling `call_smm` instead of open-coding the SMM call. Removed the delay as per Tim's suggestion.

File src/drivers/smmstore/ramstage.c:

https://review.coreboot.org/c/coreboot/+/73981/comment/7aa2fa0b_32a7c686 :
PS1, Line 79: 	} while (retry--);
> The assembly code and the loop as implemented is fine. […]

@Paul I added info on how many retries were done to the error message.