Changes since v2:
- drop common declaration of EfiResetSystemLib since its only user is a
  DXE_RUNTIME_DRIVER
- adhere to EDK2 coding style in patch #2

Changes since v1:
Instead of disallowing the use of ArmVExpressSysConfigLib by runtime drivers,
just let it return UNSUPPORTED when invoked after switching to virtual mode.
The PL031 driver already deals with that case anyway, so the patch against
that has been dropped. This change alone prevents the crash on Foundation model,
but since now, it won't actually reset or power off the machine, the PSCI change
is retained as well.

----- v1 blurb -----
As it turns out, the poweroff crash on the Foundation model was caused
by the fact that the default VExpress EfiResetSystemLib expects to find
the syscfg MMIO register block at its boot time ID mapped offset. However,
at runtime, it is only accessible if it is runtime remapped, which is
cumbersome since the same registers control all kinds of system features
that are under control of the OS.

So this series removes the ArmVExpressSysConfigLib dependencies from
drivers that are used at runtime. For the RTC driver, the dependency
is simply dropped. For the reset driver, we use PSCI calls into EL3
instead. Note that this requires PSCI compliant firmware to be running
in EL3.

Ard Biesheuvel (2):
  ArmVExpressPkg: use PSCI for system reset at runtime
  ArmVExpressPkg: disallow the use of ArmVExpressSysConfigLib at runtime

 ArmPlatformPkg/ArmVExpressPkg/ArmVExpress.dsc.inc              | 10 +++++++++-
 .../Library/ArmVExpressSysConfigLib/ArmVExpressSysConfig.c     |  6 ++++++
 .../ArmVExpressSysConfigLib/ArmVExpressSysConfigLib.inf        |  1 +
 3 files changed, 16 insertions(+), 1 deletion(-)

-- 
1.9.1


------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to