Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37171 )
Change subject: drives/amd/agesa: remove legacy wrapper and separate CAR teardown ......................................................................
Patch Set 1:
Okay. So what is our starting point? I would like to avoid work duplication. I can go with binaryPI postcar and AGESA,binaryPI C bootblock on top of the change you will point me. I already got the postcar and C bootblock on apu2 working earlier. Please let me know about the course.
Work on the biosram and ap_entry, keeping in mind we should share the same implementation code with amd/stoneyridge on how APs behave in bootblock. Like done with FSP, I feel all AGESAv5 interface code should go under drivers/ because it gets obscure if cpu/amd/pi starts to build code from soc/amd/stoneyridge. Even more so, if cpu/amd/agesa does the same for C_ENV_BB feature.
I won't touch anything C_ENV_BB related until POSTCAR_STAGE for apu2 gets merged, so that's also orthogonal development so our works should not collide too bad.