]> git.proxmox.com Git - mirror_edk2.git/commitdiff
Readme.md: Add iASL note and QEMU sample link
authorMichael Kubacki <michael.kubacki@microsoft.com>
Wed, 9 Sep 2020 23:47:09 +0000 (16:47 -0700)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Tue, 5 Apr 2022 00:42:38 +0000 (00:42 +0000)
Adds a note that the build now depends on an ASL compiler supporting
PlatformRtMechanism by default. The minimum iASL version supported
is noted in addition to an alternative to remove the code that
requires the new OperationRegion.

Also, a link is provided to an example of how to integrate the
package into a platform using QEMU/OvmfPkg as the sample platform
and firmware.

Cc: Andrew Fish <afish@apple.com>
Cc: Kang Gao <kang.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Michael Kubacki <michael.kubacki@microsoft.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Benjamin You <benjamin.you@intel.com>
Cc: Liu Yun <yun.y.liu@intel.com>
Cc: Ankit Sinha <ankit.sinha@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
Acked-by: Michael D Kinney <michael.d.kinney@intel.com>
Acked-by: Liming Gao <gaoliming@byosoft.com.cn>
Acked-by: Leif Lindholm <quic_llindhol@quicinc.com>
Reviewed-by: Ankit Sinha <ankit.sinha@intel.com>
PrmPkg/Readme.md

index 40df8f00a0c826ff8be2f13985786456310cef83..f340eeb6425851e55cf695d6e028407f02651234 100644 (file)
@@ -13,6 +13,14 @@ to be leveraged by platform firmware with minimal overhead to integrate PRM func
 formal design and is not validated at product quality. The development of this feature is shared in the edk2-staging\r
 branch to simplify collaboration by allowing direct code contributions and early feedback throughout its development.\r
 \r
+> By default, the build makes use of a new ACPI OperationRegion type specifically introduced for PRM called\r
+`PlatformRtMechanism`. Support for this OperationRegion is planned for the next release of the ACPI specification.\r
+However, support for `PlatformRtMechanism` is already included in the iASL Compiler/Disassembler for early prototyping\r
+(i.e. this package). If you would like the default build to work and/or to use PRM handlers that are invoked\r
+through ACPI, iASL compiler [20200528](https://acpica.org/node/181) or greater must be used. If you are only\r
+interested in compiling the code and/or using direct call style PRM handlers, you can simply remove\r
+`PrmSsdtInstallDxe` from `PrmPkg.dsc`.\r
+\r
 ## How to Build PrmPkg\r
 As noted earlier, resources in `PrmPkg` are intended to be referenced by a platform firmware so it can adopt support\r
 for PRM. In that case, the platform firmware should add the `PrmConfigDxe` and `PrmLoaderDxe` drivers to its DSC and\r
@@ -60,6 +68,21 @@ To build `PrmPkg` as a standalone package:
    > __*Note*__: Due to the way PRM modules are compiled with exports, **only building on Visual Studio compiler tool\r
    chains is currently supported**.\r
 \r
+In the future, each new terminal session can start at step #4. Within a terminal session, start at step #8.\r
+\r
+> __*Note*__: \\r
+> This package has been used without modification in several environments including client, server,\r
+> and virtual systems.\r
+>\r
+> A functional example of how to integrate this code into a platform is available here:\r
+> https://github.com/makubacki/edk2/tree/sample_ovmfpkg_prmpkg_integration\r
+>\r
+> That build will load the drivers and PRM sample modules provided in this package in the open source emulator\r
+> [QEMU](https://www.qemu.org/) by including it in the [`OvmfPkg`](https://github.com/tianocore/edk2/tree/master/OvmfPkg) build.\r
+>\r
+> You can add your own PRM modules into the build and check them with the `PrmInfo` UEFI application described\r
+> later in this document and dump the PRMT table in the OS to check if your PRM module is represented as expected.\r
+\r
 ### Build Flags\r
 As PRM is a new feature at a proof-of-concept (POC) level of maturity, there's some changes to the normal build\r
 available as build flags. By default, if no flags are specified, the build is done with the currently expected plan of\r