Change in coreboot[master]: soc/amd/common/apob: Add support for asynchronously reading APOB_NV

[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/+/56232 )

Change subject: soc/amd/common/apob: Add support for asynchronously reading APOB_NV
......................................................................

Patch Set 6:

(2 comments)

File src/soc/amd/common/block/apob/apob_cache.c:

https://review.coreboot.org/c/coreboot/+/56232/comment/0156924a_b094ee6c 
PS6, Line 71: 	if  (fmap_locate_area(DEFAULT_MRC_CACHE, r) < 0) {
off-topic, but I'd really advise against juggling raw regions like this. We have fmap_locate_area_as_rdev() to do this all in one go.

https://review.coreboot.org/c/coreboot/+/56232/comment/acc7bf42_2f788fff 
PS6, Line 135: 	if (wait_for_thread(&global_apob_thread.handle) == CB_SUCCESS) {
I think this should be guarded by COOP_MULTITASKING, it looks weird if you just rely on this returning an error if you never started the thread.