Re: [edk2-devel] [PATCH v2] MdePkg: Include Acpi header file

["Wasim Khan" <wasim.khan@nxp.com>]

> -----Original Message-----
> From: Philippe Mathieu-Daudé <philmd@redhat.com>
> Sent: Monday, June 1, 2020 2:17 PM
> To: devel@edk2.groups.io; Wasim Khan <wasim.khan@nxp.com>
> Cc: michael.d.kinney@intel.com; liming.gao@intel.com; Varun Sethi
> <V.Sethi@nxp.com>
> Subject: Re: [edk2-devel] [PATCH v2] MdePkg: Include Acpi header file
> 
> On 4/20/20 9:48 AM, Wasim Khan wrote:
> > ACPI memory mapped configuration space access (MCFG) table requires
> > defination of EFI_ACPI_DESCRIPTION_HEADER.
> 
> What about:
> 
> - DynamicTablesPkg/Include/Library/TableHelperLib.h
> - DynamicTablesPkg/Include/StandardNameSpaceObjects.h
> - EmbeddedPkg/Include/Library/AcpiLib.h

EmbeddedPkg/Include/Library/AcpiLib.h  => Acpi10.h is included in this file.
For other two, Acpi.h /Acpi10.h has been included in the files where above header file are used. 


> 
> >
> > Signed-off-by: Wasim Khan <wasim.khan@nxp.com>
> > ---
> > Changes in v2: Corrected typo in subject
> >
> >  .../IndustryStandard/MemoryMappedConfigurationSpaceAccessTable.h        |
> 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git
> >
> a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccess
> > Table.h
> >
> b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccess
> > Table.h
> > index 3356ec9..8a4933d 100644
> > ---
> >
> a/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAccess
> > Table.h
> > +++
> b/MdePkg/Include/IndustryStandard/MemoryMappedConfigurationSpaceAc
> > +++ cessTable.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
> >  //
> >