[edk2-devel] [PATCH] MdePkg: Include Acpi hearder file

Wasim Khan posted 1 patch 3 years, 11 months ago
Failed in applying to current master (apply log)
.../IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h        | 2 ++
1 file changed, 2 insertions(+)
[edk2-devel] [PATCH] MdePkg: Include Acpi hearder file
Posted by Wasim Khan 3 years, 11 months ago
ACPI memory mapped configuration space access (MCFG) table requires
defination of EFI_ACPI_DESCRIPTION_HEADER.

Signed-off-by: Wasim Khan <wasim.khan@nxp.com>
---
 .../IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h        | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
index 3356ec9..8a4933d 100644
--- a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
+++ b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
@@ -10,6 +10,8 @@
 #ifndef _MEMORY_MAPPED_CONFIGURATION_SPACE_ACCESS_TABLE_H_
 #define _MEMORY_MAPPED_CONFIGURATION_SPACE_ACCESS_TABLE_H_
 
+#include <IndustryStandard/Acpi.h>
+
 //
 // Ensure proper structure formats
 //
-- 
2.7.4


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#57422): https://edk2.groups.io/g/devel/message/57422
Mute This Topic: https://groups.io/mt/73037290/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH] MdePkg: Include Acpi hearder file
Posted by Laszlo Ersek 3 years, 11 months ago
On 04/15/20 13:40, Wasim Khan wrote:
> ACPI memory mapped configuration space access (MCFG) table requires
> defination of EFI_ACPI_DESCRIPTION_HEADER.
> 
> Signed-off-by: Wasim Khan <wasim.khan@nxp.com>
> ---
>  .../IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h        | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
> index 3356ec9..8a4933d 100644
> --- a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
> +++ b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h
> @@ -10,6 +10,8 @@
>  #ifndef _MEMORY_MAPPED_CONFIGURATION_SPACE_ACCESS_TABLE_H_
>  #define _MEMORY_MAPPED_CONFIGURATION_SPACE_ACCESS_TABLE_H_
>  
> +#include <IndustryStandard/Acpi.h>
> +
>  //
>  // Ensure proper structure formats
>  //
> 

There's a typo in the subject: s/hearder/header/.

(I'll let the MdePkg maintainers review this patch beyond this
superficial comment.)

Thanks
Laszlo


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#57474): https://edk2.groups.io/g/devel/message/57474
Mute This Topic: https://groups.io/mt/73037290/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-