On Tue, Jul 04, 2017 at 10:35:06AM +0100, Ryan Harkin wrote:
On 3 July 2017 at 22:19, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
On 3 July 2017 at 19:49, Leif Lindholm leif.lindholm@linaro.org wrote:
On Mon, Jul 03, 2017 at 06:36:14PM +0100, Ard Biesheuvel wrote:
On 3 July 2017 at 17:17, Leif Lindholm leif.lindholm@linaro.org wrote:
For historic reasons, ArmVExpress.dsc.inc defaulted to the ArmVExpress MMIO-based implementation of *Efi*ResetSystemLib - invoked via EmbeddedPkg/ResetRuntimeDxe. Since the majority of our platforms are now 64-bit and PSCI-based, flip this logic to default to the ArmPkg ArmSmcPsciResetSystemLib implementation of ResetSystemLib - with ARM-targets retaining the old behaviour.
At the same time update FVP and Juno targets to use the new generic ResetSystemRuntimeDxe.
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Leif Lindholm leif.lindholm@linaro.org
OK, so you are intentionally keeping the 32-bit platforms on the reset runtime from EmbeddedPkg, and intend to address that later?
Correct. Doing that at the same time felt ... messy.
Fair enough
Reviewed-by: Ard Biesheuvel ard.biesheuvel@linaro.org
Fine with me too. I haven't tested it, but it looks safe enough.
Reviewed-by: Ryan Harkin ryan.harkin@linaro.org
Thanks guys. Pushed as bbdd9ce612.
/ Leif