On 02/17/15 18:14, Jordan Justen wrote:
> This name better aligns with InstallXenTables and InstallOvmfFvTables.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jordan Justen <[email protected]>
> ---
>  OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c  | 2 +-
>  OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h  | 3 ++-
>  OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c | 4 ++--
>  3 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c 
> b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> index 3632d84..8bfb189 100644
> --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> +++ b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> @@ -261,7 +261,7 @@ AcpiPlatformEntryPoint (
>    if (XenDetected ()) {
>      Status = InstallXenTables (AcpiTable);
>    } else {
> -    Status = InstallAllQemuLinkedTables (AcpiTable);
> +    Status = InstallQemuFwCfgTables (AcpiTable);
>    }
>  
>    if (EFI_ERROR (Status)) {
> diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h 
> b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
> index e757233..ae11e79 100644
> --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
> +++ b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
> @@ -63,8 +63,9 @@ InstallXenTables (
>  
>  EFI_STATUS
>  EFIAPI
> -InstallAllQemuLinkedTables (
> +InstallQemuFwCfgTables (
>    IN   EFI_ACPI_TABLE_PROTOCOL       *AcpiProtocol
>    );
> +
>  #endif
>  
> diff --git a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c 
> b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
> index a4a71f9..806446e 100644
> --- a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
> +++ b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
> @@ -552,7 +552,7 @@ Process2ndPassCmdAddPointer (
>  **/
>  EFI_STATUS
>  EFIAPI
> -InstallAllQemuLinkedTables (
> +InstallQemuFwCfgTables (
>    IN   EFI_ACPI_TABLE_PROTOCOL       *AcpiProtocol
>    )
>  {
> @@ -716,6 +716,6 @@ QemuFwCfgAcpiPlatformEntryPoint (
>                    );
>    ASSERT_EFI_ERROR (Status);
>  
> -  Status = InstallAllQemuLinkedTables (AcpiTable);
> +  Status = InstallQemuFwCfgTables (AcpiTable);
>    return Status;
>  }
> 

Ah, I see, you applied some of the patches from your other series too,
like moving the entry point into the same file. Seems fine.

Reviewed-by: Laszlo Ersek <[email protected]>


------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to