From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.73844.1584379052703714886 for ; Mon, 16 Mar 2020 10:17:33 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: thomas.abraham@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3E4031045 for ; Mon, 16 Mar 2020 10:17:32 -0700 (PDT) Received: from mail-il1-f180.google.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 25CB53F67D for ; Mon, 16 Mar 2020 10:17:32 -0700 (PDT) Received: by mail-il1-f180.google.com with SMTP id e8so17255291ilc.13 for ; Mon, 16 Mar 2020 10:17:32 -0700 (PDT) X-Gm-Message-State: ANhLgQ0ASfyKg2WG+L2C+hLnoNSIwKBuY0uKrbuTToA+cWEW3GjGrQUT eIXf4BLV7HSdjc4hHnaodqVB0goBlua/FCkVj0k= X-Google-Smtp-Source: ADFU+vtKpZkPZ4nYWknf6pbSJBVX3kWeUjUWPOq/n+AaUMKz71ixuqgXBOkFvJWCWqDVE3elGWn00hcvFA7kE7Req7s= X-Received: by 2002:a92:8dc3:: with SMTP id w64mr779511ill.68.1584379051178; Mon, 16 Mar 2020 10:17:31 -0700 (PDT) MIME-Version: 1.0 References: <20200312143520.16918-1-aditya.angadi@arm.com> <20200312143520.16918-3-aditya.angadi@arm.com> In-Reply-To: <20200312143520.16918-3-aditya.angadi@arm.com> From: "Thomas Abraham" Date: Mon, 16 Mar 2020 22:47:19 +0530 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [edk2-platforms][PATCH 2/8] Platform/ARM/SgiPkg: move the GIC related ACPI helper macros To: devel@edk2.groups.io, aditya.angadi@arm.com Cc: leif@nuviainc.com, michael.d.kinney@intel.com, vijayenthiran.subramaniam@arm.com Content-Type: text/plain; charset="UTF-8" On Thu, Mar 12, 2020 at 8:05 PM Aditya Angadi wrote: > > Move the ACPI helper macros defines related to GIC structure, > distributor, redistributor and ITS to SgiAcpiHeader.h as these are > common across ARM SGI/RD platforms. > > Signed-off-by: Aditya Angadi Reviewed-by: Thomas Abraham > --- > .../ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc | 68 +----------------- > .../ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc | 68 +----------------- > .../SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc | 57 +-------------- > Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h | 70 ++++++++++++++++++- > 4 files changed, 72 insertions(+), 191 deletions(-) > > diff --git a/Platform/ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc b/Platform/ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc > index 48e7a61478..9872549285 100644 > --- a/Platform/ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc > +++ b/Platform/ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc > @@ -1,7 +1,7 @@ > /** @file > * Multiple APIC Description Table (MADT) > * > -* Copyright (c) 2018, ARM Limited. All rights reserved. > +* Copyright (c) 2018-2020, ARM Limited. All rights reserved. > * > * SPDX-License-Identifier: BSD-2-Clause-Patent > * > @@ -14,72 +14,6 @@ > #include > #include > > -// EFI_ACPI_6_2_GIC_STRUCTURE > -#define EFI_ACPI_6_2_GICC_STRUCTURE_INIT(GicId, AcpiCpuUid, Mpidr, Flags, \ > - PmuIrq, GicBase, GicVBase, GicHBase, GsivId, GicRBase, Efficiency) \ > - { \ > - EFI_ACPI_6_2_GIC, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - GicId, /* CPUInterfaceNumber */ \ > - AcpiCpuUid, /* AcpiProcessorUid */ \ > - Flags, /* Flags */ \ > - 0, /* ParkingProtocolVersion */ \ > - PmuIrq, /* PerformanceInterruptGsiv */ \ > - 0, /* ParkedAddress */ \ > - GicBase, /* PhysicalBaseAddress */ \ > - GicVBase, /* GICV */ \ > - GicHBase, /* GICH */ \ > - GsivId, /* VGICMaintenanceInterrupt */ \ > - GicRBase, /* GICRBaseAddress */ \ > - Mpidr, /* MPIDR */ \ > - Efficiency, /* ProcessorPowerEfficiencyClass */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_DISTRIBUTOR_INIT(GicDistHwId, GicDistBase, \ > - GicDistVector, GicVersion) \ > - { \ > - EFI_ACPI_6_2_GICD, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE), \ > - EFI_ACPI_RESERVED_WORD, /* Reserved1 */ \ > - GicDistHwId, /* GicId */ \ > - GicDistBase, /* PhysicalBaseAddress */ \ > - GicDistVector, /* SystemVectorBase */ \ > - GicVersion, /* GicVersion */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GICR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_REDISTRIBUTOR_INIT(RedisRegionAddr, RedisDiscLength) \ > - { \ > - EFI_ACPI_6_2_GICR, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GICR_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - RedisRegionAddr, /* DiscoveryRangeBaseAddress */ \ > - RedisDiscLength /* DiscoveryRangeLength */ \ > - } > - > -// EFI_ACPI_6_2_GIC_ITS_STRUCTURE > -#define EFI_ACPI_6_2_GIC_ITS_INIT(GicItsId, GicItsBase) \ > - { \ > - EFI_ACPI_6_2_GIC_ITS, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_ITS_STRUCTURE), \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - GicItsId, /* GicItsId */ \ > - GicItsBase, /* PhysicalBaseAddress */ \ > - EFI_ACPI_RESERVED_DWORD /* DiscoveryRangeLength */ \ > - } > - > // Multiple APIC Description Table > #pragma pack (1) > > diff --git a/Platform/ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc b/Platform/ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc > index 6312743a47..05eb78c561 100644 > --- a/Platform/ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc > +++ b/Platform/ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc > @@ -1,7 +1,7 @@ > /** @file > * Multiple APIC Description Table (MADT) > * > -* Copyright (c) 2018, ARM Limited. All rights reserved. > +* Copyright (c) 2018-2020, ARM Limited. All rights reserved. > * > * SPDX-License-Identifier: BSD-2-Clause-Patent > * > @@ -17,72 +17,6 @@ > #define CORE_CNT (FixedPcdGet32 (PcdClusterCount) * \ > FixedPcdGet32 (PcdCoreCount)) > > -// EFI_ACPI_6_2_GIC_STRUCTURE > -#define EFI_ACPI_6_2_GICC_STRUCTURE_INIT(GicId, AcpiCpuUid, Mpidr, Flags, \ > - PmuIrq, GicBase, GicVBase, GicHBase, GsivId, GicRBase, Efficiency) \ > - { \ > - EFI_ACPI_6_2_GIC, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - GicId, /* CPUInterfaceNumber */ \ > - AcpiCpuUid, /* AcpiProcessorUid */ \ > - Flags, /* Flags */ \ > - 0, /* ParkingProtocolVersion */ \ > - PmuIrq, /* PerformanceInterruptGsiv */ \ > - 0, /* ParkedAddress */ \ > - GicBase, /* PhysicalBaseAddress */ \ > - GicVBase, /* GICV */ \ > - GicHBase, /* GICH */ \ > - GsivId, /* VGICMaintenanceInterrupt */ \ > - GicRBase, /* GICRBaseAddress */ \ > - Mpidr, /* MPIDR */ \ > - Efficiency, /* ProcessorPowerEfficiencyClass */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_DISTRIBUTOR_INIT(GicDistHwId, GicDistBase, \ > - GicDistVector, GicVersion) \ > - { \ > - EFI_ACPI_6_2_GICD, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE), \ > - EFI_ACPI_RESERVED_WORD, /* Reserved1 */ \ > - GicDistHwId, /* GicId */ \ > - GicDistBase, /* PhysicalBaseAddress */ \ > - GicDistVector, /* SystemVectorBase */ \ > - GicVersion, /* GicVersion */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GICR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_REDISTRIBUTOR_INIT(RedisRegionAddr, RedisDiscLength) \ > - { \ > - EFI_ACPI_6_2_GICR, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GICR_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - RedisRegionAddr, /* DiscoveryRangeBaseAddress */ \ > - RedisDiscLength /* DiscoveryRangeLength */ \ > - } > - > -// EFI_ACPI_6_2_GIC_ITS_STRUCTURE > -#define EFI_ACPI_6_2_GIC_ITS_INIT(GicItsId, GicItsBase) \ > - { \ > - EFI_ACPI_6_2_GIC_ITS, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_ITS_STRUCTURE), \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - GicItsId, /* GicItsId */ \ > - GicItsBase, /* PhysicalBaseAddress */ \ > - EFI_ACPI_RESERVED_DWORD /* DiscoveryRangeLength */ \ > - } > - > // Multiple APIC Description Table > #pragma pack (1) > > diff --git a/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc b/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc > index d4538233d7..47368931e3 100644 > --- a/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc > +++ b/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc > @@ -1,7 +1,7 @@ > /** @file > * Multiple APIC Description Table (MADT) > * > -* Copyright (c) 2019, ARM Limited. All rights reserved. > +* Copyright (c) 2019-2020, ARM Limited. All rights reserved. > * > * SPDX-License-Identifier: BSD-2-Clause-Patent > * > @@ -19,61 +19,6 @@ > > #define CHIP_CNT 2 > > -// EFI_ACPI_6_2_GIC_STRUCTURE > -#define EFI_ACPI_6_2_GICC_STRUCTURE_INIT(GicId, AcpiCpuUid, Mpidr, Flags, \ > - PmuIrq, GicBase, GicVBase, GicHBase, GsivId, GicRBase, Efficiency) \ > - { \ > - EFI_ACPI_6_2_GIC, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - GicId, /* CPUInterfaceNumber */ \ > - AcpiCpuUid, /* AcpiProcessorUid */ \ > - Flags, /* Flags */ \ > - 0, /* ParkingProtocolVersion */ \ > - PmuIrq, /* PerformanceInterruptGsiv */ \ > - 0, /* ParkedAddress */ \ > - GicBase, /* PhysicalBaseAddress */ \ > - GicVBase, /* GICV */ \ > - GicHBase, /* GICH */ \ > - GsivId, /* VGICMaintenanceInterrupt */ \ > - GicRBase, /* GICRBaseAddress */ \ > - Mpidr, /* MPIDR */ \ > - Efficiency, /* ProcessorPowerEfficiencyClass */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_DISTRIBUTOR_INIT(GicDistHwId, GicDistBase, \ > - GicDistVector, GicVersion) \ > - { \ > - EFI_ACPI_6_2_GICD, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE), \ > - EFI_ACPI_RESERVED_WORD, /* Reserved1 */ \ > - GicDistHwId, /* GicId */ \ > - GicDistBase, /* PhysicalBaseAddress */ \ > - GicDistVector, /* SystemVectorBase */ \ > - GicVersion, /* GicVersion */ \ > - { \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > - EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > - EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > - } \ > - } > - > -// EFI_ACPI_6_2_GICR_STRUCTURE > -#define EFI_ACPI_6_2_GIC_REDISTRIBUTOR_INIT(RedisRegionAddr, RedisDiscLength) \ > - { \ > - EFI_ACPI_6_2_GICR, /* Type */ \ > - sizeof (EFI_ACPI_6_2_GICR_STRUCTURE), /* Length */ \ > - EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > - RedisRegionAddr, /* DiscoveryRangeBaseAddress */ \ > - RedisDiscLength /* DiscoveryRangeLength */ \ > - } > - > // Multiple APIC Description Table > #pragma pack (1) > > diff --git a/Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h b/Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h > index 5083dde15d..ecb0d4eccf 100644 > --- a/Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h > +++ b/Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h > @@ -1,6 +1,6 @@ > /** @file > * > -* Copyright (c) 2018, ARM Limited. All rights reserved. > +* Copyright (c) 2018-2020, ARM Limited. All rights reserved. > * > * SPDX-License-Identifier: BSD-2-Clause-Patent > * > @@ -9,6 +9,8 @@ > #ifndef __SGI_ACPI_HEADER__ > #define __SGI_ACPI_HEADER__ > > +#include > + > // > // ACPI table information used to initialize tables. > // > @@ -32,4 +34,70 @@ > EFI_ACPI_ARM_CREATOR_REVISION /* UINT32 CreatorRevision */ \ > } > > +// EFI_ACPI_6_2_GIC_STRUCTURE > +#define EFI_ACPI_6_2_GICC_STRUCTURE_INIT(GicId, AcpiCpuUid, Mpidr, Flags, \ > + PmuIrq, GicBase, GicVBase, GicHBase, GsivId, GicRBase, Efficiency) \ > + { \ > + EFI_ACPI_6_2_GIC, /* Type */ \ > + sizeof (EFI_ACPI_6_2_GIC_STRUCTURE), /* Length */ \ > + EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > + GicId, /* CPUInterfaceNumber */ \ > + AcpiCpuUid, /* AcpiProcessorUid */ \ > + Flags, /* Flags */ \ > + 0, /* ParkingProtocolVersion */ \ > + PmuIrq, /* PerformanceInterruptGsiv */ \ > + 0, /* ParkedAddress */ \ > + GicBase, /* PhysicalBaseAddress */ \ > + GicVBase, /* GICV */ \ > + GicHBase, /* GICH */ \ > + GsivId, /* VGICMaintenanceInterrupt */ \ > + GicRBase, /* GICRBaseAddress */ \ > + Mpidr, /* MPIDR */ \ > + Efficiency, /* ProcessorPowerEfficiencyClass */ \ > + { \ > + EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > + EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > + EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > + } \ > + } > + > +// EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE > +#define EFI_ACPI_6_2_GIC_DISTRIBUTOR_INIT(GicDistHwId, GicDistBase, \ > + GicDistVector, GicVersion) \ > + { \ > + EFI_ACPI_6_2_GICD, /* Type */ \ > + sizeof (EFI_ACPI_6_2_GIC_DISTRIBUTOR_STRUCTURE), \ > + EFI_ACPI_RESERVED_WORD, /* Reserved1 */ \ > + GicDistHwId, /* GicId */ \ > + GicDistBase, /* PhysicalBaseAddress */ \ > + GicDistVector, /* SystemVectorBase */ \ > + GicVersion, /* GicVersion */ \ > + { \ > + EFI_ACPI_RESERVED_BYTE, /* Reserved2[0] */ \ > + EFI_ACPI_RESERVED_BYTE, /* Reserved2[1] */ \ > + EFI_ACPI_RESERVED_BYTE /* Reserved2[2] */ \ > + } \ > + } > + > +// EFI_ACPI_6_2_GICR_STRUCTURE > +#define EFI_ACPI_6_2_GIC_REDISTRIBUTOR_INIT(RedisRegionAddr, RedisDiscLength) \ > + { \ > + EFI_ACPI_6_2_GICR, /* Type */ \ > + sizeof (EFI_ACPI_6_2_GICR_STRUCTURE), /* Length */ \ > + EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > + RedisRegionAddr, /* DiscoveryRangeBaseAddress */ \ > + RedisDiscLength /* DiscoveryRangeLength */ \ > + } > + > +// EFI_ACPI_6_2_GIC_ITS_STRUCTURE > +#define EFI_ACPI_6_2_GIC_ITS_INIT(GicItsId, GicItsBase) \ > + { \ > + EFI_ACPI_6_2_GIC_ITS, /* Type */ \ > + sizeof (EFI_ACPI_6_2_GIC_ITS_STRUCTURE), \ > + EFI_ACPI_RESERVED_WORD, /* Reserved */ \ > + GicItsId, /* GicItsId */ \ > + GicItsBase, /* PhysicalBaseAddress */ \ > + EFI_ACPI_RESERVED_DWORD /* DiscoveryRangeLength */ \ > + } > + > #endif /* __SGI_ACPI_HEADER__ */ > -- > 2.17.1 > > > >