At least fam14 has issues with cache coherency for CAR. You can try enabling CBMEM console for APs (SQUELCH_EARLY_SMP=n) and you will not get complete console log there. Depends of alignment AFAICS. But if we have an approach that avoids pushing AGESA quirks to common code, we should follow that path.

We should take suitable parts from amd/stoneyridge instead of making slighty modified copies, these should really merge into one AGESAv5 implementation eventually. Those PCI scratchpads were chosen because CAR shared across APs did not initially work there either and biosram_XX was not invented yet, either.

I think the correct thing to do would be to take biosram_XX implementation from amd/stoneyridge and implement both backup_top_of_low_cacheable() and ap_entry_ptr() on top of that. For all AGESA and binaryPI.

At the moment, urgent thing to do is have POSTCAR_STAGE=y in shape that can be merged, aka drop BINARYPI_LEGACY_WRAPPER entirely.

View Change

To view, visit change 37167. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I177cd6f84c3f36776ec7bc8eeacb5b2a1d9142d2
Gerrit-Change-Number: 37167
Gerrit-PatchSet: 2
Gerrit-Owner: Michał Żygowski <michal.zygowski@3mdeb.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski@3mdeb.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Sat, 23 Nov 2019 17:04:35 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment