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.web08.14103.1637687366519669604 for ; Tue, 23 Nov 2021 09:09:26 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@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 325821063; Tue, 23 Nov 2021 09:09:26 -0800 (PST) Received: from e126645.nice.arm.com (unknown [10.34.129.32]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 85A233F5A1; Tue, 23 Nov 2021 09:09:25 -0800 (PST) From: "PierreGondois" To: devel@edk2.groups.io Cc: Sami Mujawar , Alexei Fedorov Subject: [PATCH v3 09/15] DynamicTablesPkg: FdtHwInfoParser: Add MSI Frame parser Date: Tue, 23 Nov 2021 18:08:32 +0100 Message-Id: <20211123170838.143805-10-Pierre.Gondois@arm.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20211123170838.143805-1-Pierre.Gondois@arm.com> References: <20211123170838.143805-1-Pierre.Gondois@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Pierre Gondois Arm GIC version 2 systems that support Message Signalled Interrupts implement GICv2m MSI frame(s). Each GICv2m MSI frame consists of a 4k page which includes registers to generate message signalled interrupts to an associated GIC distributor. The frame also includes registers to discover the set of distributor lines which may be signalled by MSIs from that frame. A system may have multiple MSI frames, and separate frames may be defined for secure and non-secure access. A MSI Frame structure is part of the Multiple APIC Description Table (MADT) and must only be used to describe non-secure MSI frames. The MSI Frame information is described in the platform Device Tree, the bindings for which can be found at: - linux/Documentation/devicetree/bindings/interrupt-controller/ arm,gic.yaml - linux/Documentation/devicetree/bindings/interrupt-controller/ arm,gic-v3.yaml The FdtHwInfoParser implements a MSI Frame Parser that parses the platform Device Tree to create CM_ARM_GIC_MSI_FRAME_INFO objects which are encapsulated in a Configuration Manager descriptor object and added to the platform information repository. The platform Configuration Manager can then utilise this information when generating the MADT table. Signed-off-by: Pierre Gondois --- .../Gic/ArmGicMsiFrameParser.c | 214 ++++++++++++++++++ .../Gic/ArmGicMsiFrameParser.h | 50 ++++ 2 files changed, 264 insertions(+) create mode 100644 DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGi= cMsiFrameParser.c create mode 100644 DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGi= cMsiFrameParser.h diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFra= meParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFra= meParser.c new file mode 100644 index 000000000000..9e6715f69ce7 --- /dev/null +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFrameParse= r.c @@ -0,0 +1,214 @@ +/** @file + Arm Gic Msi frame Parser. + + Copyright (c) 2021, ARM Limited. All rights reserved.
+ SPDX-License-Identifier: BSD-2-Clause-Patent + + @par Reference(s): + - linux/Documentation/devicetree/bindings/interrupt-controller/arm,gic= .yaml + - linux/Documentation/devicetree/bindings/interrupt-controller/arm,gic= -v3.yaml +**/ + +#include "CmObjectDescUtility.h" +#include "FdtHwInfoParser.h" +#include "Gic/ArmGicDispatcher.h" +#include "Gic/ArmGicMsiFrameParser.h" + +/** List of "compatible" property values for Msi-frame nodes. + + Any other "compatible" value is not supported by this module. +*/ +STATIC CONST COMPATIBILITY_STR MsiFrameCompatibleStr[] =3D { + {"arm,gic-v2m-frame"} +}; + +/** COMPATIBILITY_INFO structure for the MSI frame. +*/ +STATIC CONST COMPATIBILITY_INFO MsiFrameCompatibleInfo =3D { + ARRAY_SIZE (MsiFrameCompatibleStr), + MsiFrameCompatibleStr +}; + +/** Parse a Msi frame node. + + @param [in] Fdt Pointer to a Flattened Device Tree (Fdt). + @param [in] MsiFrameNode Offset of a Msi frame node. + @param [in] MsiFrameId Frame ID. + @param [out] MsiFrameInfo The CM_ARM_GIC_MSI_FRAME_INFO to populate. + + @retval EFI_SUCCESS The function completed successfully. + @retval EFI_ABORTED An error occurred. + @retval EFI_INVALID_PARAMETER Invalid parameter. +**/ +STATIC +EFI_STATUS +EFIAPI +MsiFrameNodeParser ( + IN CONST VOID * Fdt, + IN INT32 MsiFrameNode, + IN UINT32 MsiFrameId, + OUT CM_ARM_GIC_MSI_FRAME_INFO * MsiFrameInfo + ) +{ + EFI_STATUS Status; + INT32 AddressCells; + CONST UINT8 * Data; + INT32 DataSize; + + if ((Fdt =3D=3D NULL) || + (MsiFrameInfo =3D=3D NULL)) { + ASSERT (0); + return EFI_INVALID_PARAMETER; + } + + Status =3D FdtGetParentAddressInfo (Fdt, MsiFrameNode, &AddressCells, = NULL); + if (EFI_ERROR (Status)) { + ASSERT (0); + return Status; + } + + // Don't support more than 64 bits and less than 32 bits addresses. + if ((AddressCells < 1) || + (AddressCells > 2)) { + ASSERT (0); + return EFI_ABORTED; + } + + Data =3D fdt_getprop (Fdt, MsiFrameNode, "reg", &DataSize); + if ((Data =3D=3D NULL) || (DataSize < (INT32)(AddressCells * sizeof (U= INT32)))) { + // If error or not enough space. + ASSERT (0); + return EFI_ABORTED; + } + + if (AddressCells =3D=3D 2) { + MsiFrameInfo->PhysicalBaseAddress =3D fdt64_to_cpu (*(UINT64*)Data); + } else { + MsiFrameInfo->PhysicalBaseAddress =3D fdt32_to_cpu (*(UINT32*)Data); + } + + MsiFrameInfo->GicMsiFrameId =3D MsiFrameId; + + return EFI_SUCCESS; +} + +/** CM_ARM_GIC_MSI_FRAME_INFO parser function. + + The following structure is populated: + typedef struct CmArmGicMsiFrameInfo { + UINT32 GicMsiFrameId; // {Populated} + UINT64 PhysicalBaseAddress; // {Populated} + UINT32 Flags; // {default =3D 0} + UINT16 SPICount; + UINT16 SPIBase; + } CM_ARM_GIC_MSI_FRAME_INFO; + + A parser parses a Device Tree to populate a specific CmObj type. None, + one or many CmObj can be created by the parser. + The created CmObj are then handed to the parser's caller through the + HW_INFO_ADD_OBJECT interface. + This can also be a dispatcher. I.e. a function that not parsing a + Device Tree but calling other parsers. + + @param [in] FdtParserHandle A handle to the parser instance. + @param [in] FdtBranch When searching for DT node name, restrict + the search to this Device Tree branch. + + @retval EFI_SUCCESS The function completed successfully. + @retval EFI_ABORTED An error occurred. + @retval EFI_INVALID_PARAMETER Invalid parameter. + @retval EFI_NOT_FOUND Not found. + @retval EFI_UNSUPPORTED Unsupported. +**/ +EFI_STATUS +EFIAPI +ArmGicMsiFrameInfoParser ( + IN CONST FDT_HW_INFO_PARSER_HANDLE FdtParserHandle, + IN INT32 FdtBranch + ) +{ + EFI_STATUS Status; + INT32 MsiFrameNode; + UINT32 MsiFrameNodeCount; + + UINT32 Index; + CM_ARM_GIC_MSI_FRAME_INFO MsiFrameInfo; + VOID * Fdt; + + if (FdtParserHandle =3D=3D NULL) { + ASSERT (0); + return EFI_INVALID_PARAMETER; + } + + Fdt =3D FdtParserHandle->Fdt; + + // Count the number of nodes having the "interrupt-controller" propert= y. + Status =3D FdtCountPropNodeInBranch ( + Fdt, + FdtBranch, + "msi-controller", + &MsiFrameNodeCount + ); + if (EFI_ERROR (Status)) { + ASSERT (0); + return Status; + } + + if (MsiFrameNodeCount =3D=3D 0) { + return EFI_NOT_FOUND; + } + + // Parse each node having the "msi-controller" property. + MsiFrameNode =3D FdtBranch; + for (Index =3D 0; Index < MsiFrameNodeCount; Index++) { + ZeroMem (&MsiFrameInfo, sizeof (CM_ARM_GIC_MSI_FRAME_INFO)); + + Status =3D FdtGetNextPropNodeInBranch ( + Fdt, + FdtBranch, + "msi-controller", + &MsiFrameNode + ); + if (EFI_ERROR (Status)) { + ASSERT (0); + if (Status =3D=3D EFI_NOT_FOUND) { + // Should have found the node. + Status =3D EFI_ABORTED; + } + return Status; + } + + if (!FdtNodeIsCompatible (Fdt, MsiFrameNode, &MsiFrameCompatibleInfo= )) { + ASSERT (0); + Status =3D EFI_UNSUPPORTED; + return Status; + } + + // Parse the Msi information. + Status =3D MsiFrameNodeParser ( + Fdt, + MsiFrameNode, + Index, + &MsiFrameInfo + ); + if (EFI_ERROR (Status)) { + ASSERT (0); + return Status; + } + + // Add the CmObj to the Configuration Manager. + Status =3D AddSingleCmObj ( + FdtParserHandle, + CREATE_CM_ARM_OBJECT_ID (EArmObjGicMsiFrameInfo), + &MsiFrameInfo, + sizeof (CM_ARM_GIC_MSI_FRAME_INFO), + NULL + ); + if (EFI_ERROR (Status)) { + ASSERT (0); + return Status; + } + } // for + + return Status; +} diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFra= meParser.h b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFra= meParser.h new file mode 100644 index 000000000000..40f83010a686 --- /dev/null +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicMsiFrameParse= r.h @@ -0,0 +1,50 @@ +/** @file + Arm Gic Msi frame Parser. + + Copyright (c) 2021, ARM Limited. All rights reserved.
+ SPDX-License-Identifier: BSD-2-Clause-Patent + + @par Reference(s): + - linux/Documentation/devicetree/bindings/interrupt-controller/arm,gic= .yaml + - linux/Documentation/devicetree/bindings/interrupt-controller/arm,gic= -v3.yaml +**/ + +#ifndef ARM_GIC_MSI_FRAME_PARSER_H_ +#define ARM_GIC_MSI_FRAME_PARSER_H_ + +/** CM_ARM_GIC_MSI_FRAME_INFO parser function. + + The following structure is populated: + typedef struct CmArmGicMsiFrameInfo { + UINT32 GicMsiFrameId; // {Populated} + UINT64 PhysicalBaseAddress; // {Populated} + UINT32 Flags; // {default =3D 0} + UINT16 SPICount; + UINT16 SPIBase; + } CM_ARM_GIC_MSI_FRAME_INFO; + + A parser parses a Device Tree to populate a specific CmObj type. None, + one or many CmObj can be created by the parser. + The created CmObj are then handed to the parser's caller through the + HW_INFO_ADD_OBJECT interface. + This can also be a dispatcher. I.e. a function that not parsing a + Device Tree but calling other parsers. + + @param [in] FdtParserHandle A handle to the parser instance. + @param [in] FdtBranch When searching for DT node name, restrict + the search to this Device Tree branch. + + @retval EFI_SUCCESS The function completed successfully. + @retval EFI_ABORTED An error occurred. + @retval EFI_INVALID_PARAMETER Invalid parameter. + @retval EFI_NOT_FOUND Not found. + @retval EFI_UNSUPPORTED Unsupported. +**/ +EFI_STATUS +EFIAPI +ArmGicMsiFrameInfoParser ( + IN CONST FDT_HW_INFO_PARSER_HANDLE FdtParserHandle, + IN INT32 FdtBranch + ); + +#endif // ARM_GIC_MSI_FRAME_PARSER_H_ --=20 2.25.1