[coreboot-gerrit] Change in coreboot[master]: soc/amd/common/block/lpc/spi_dma: Yield after completing transaction

[Julius Werner (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Felix Held.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56321 )

Change subject: soc/amd/common/block/lpc/spi_dma: Yield after completing transaction
......................................................................

Patch Set 1:

(1 comment)

File src/soc/amd/common/block/lpc/spi_dma.c:

https://review.coreboot.org/c/coreboot/+/56321/comment/b42d9cce_1d8ab697 
PS1, Line 187: 	udelay(0);
I think you should use thread_yield_microseconds(0) instead of udelay(0) if you want to just explicitly yield to coop threads. That way the code itself explains what that's for and you don't need a comment to explain it. (We could also make a thread_yield() as an argument-less shortcut for thread_yield_microseconds(0) if that becomes a commonly needed thing.)