From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id E0D2FAC0EA1 for ; Wed, 3 Jul 2024 09:55:30 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=SjDiU1QL9zl+1bGd6UT/A44VHd4SqTr1eQGPclK5U6U=; c=relaxed/simple; d=groups.io; h=Message-ID:Date:MIME-Version:User-Agent:Subject:To:Cc:References:From:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20240206; t=1720000530; v=1; b=bUWFJ6pklPW1UO8TeCrcs0wREjCo+MHgNU3l2iOCgRogitvNqUWOkgl5WhDfYPI/h8yflkqy TDpmVFfE+ZmSXx7rIYNkTqj2owpnRnI3Fko0iqCddRWVPdhqfe9yDI2gO2/WvwfaeA4fDbBDeN9 ZuyDyLrNHBH+fuH2x/NIgcCPcUrZ7d4nkzLyAyxAWTGeGA/PW3xBnRG6MQgKEY3unEcMELZRSFC JwYSH6c3zgnq0wOC+pcQbINm1ZncgJmK52tIFPdzl9dS08E2BlZZ7lzGiR6cJBm92UKVRQKyvzT UQA2l5LjGhxjsapAzK/kOOseJVk2GAnQ5g7AxViqpIp/Q== X-Received: by 127.0.0.2 with SMTP id 9VidYY7687511xTIV5hcrzz9; Wed, 03 Jul 2024 02:55:29 -0700 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.47448.1720000528654372824 for ; Wed, 03 Jul 2024 02:55:28 -0700 X-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 43D50367; Wed, 3 Jul 2024 02:55:53 -0700 (PDT) X-Received: from [192.168.1.13] (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A48143F766; Wed, 3 Jul 2024 02:55:25 -0700 (PDT) Message-ID: Date: Wed, 3 Jul 2024 11:55:16 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs To: Sami Mujawar , "devel@edk2.groups.io" Cc: AbdulLateef Attar , Girish Mahadevan , "Jeff Brasen (jbrasen@nvidia.com)" , Jeshua Smith , Leif Lindholm , "Meenakshi Aggarwal (meenakshi.aggarwal@nxp.com)" , Sunil V L , Yeo Reum Yun , nd References: <20240619220629.1994362-1-pierre.gondois@arm.com> <555B1F5F-E761-4495-A690-6FDF298125A1@arm.com> From: "PierreGondois" In-Reply-To: <555B1F5F-E761-4495-A690-6FDF298125A1@arm.com> Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Wed, 03 Jul 2024 02:55:28 -0700 Resent-From: pierre.gondois@arm.com Reply-To: devel@edk2.groups.io,pierre.gondois@arm.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: D4MrXVgC5l3MMrXkFzkMO9VWx7686176AA= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=bUWFJ6pk; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=arm.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io Hello Sami, On 7/3/24 11:08, Sami Mujawar wrote: > Hi Pierre, >=20 > Overall, this patch series looks good to me. >=20 > I have some minor comments regarding the return value for the Arch hook f= unctions their placement in Common folder. > e.g. in Patch "DynamicTablesPkg: AcpiFadtLib: Prepare to support other ar= chs" > The file DynamicTablesPkg/Library/Acpi/Common/AcpiFadtLib/Common/CommonFa= dtGenerator.c > provides an empty stub for the arch specific implementation for the FadtA= rchUpdate () and returns > success. I think this function should return EFI_UNSUPPORTED to indicate = that this function is not > implemented and that the architecture needs to provide an implementation. >=20 > Also, the file name AcpiFadtLib/Common/CommonFadtGenerator.c should be ch= anged to > AcpiFadtLib/FadtGeneratorNull.c to clarify that the implementation does n= ot exist. >=20 > Similar changes are required for other patches as well. >=20 > Apart from the above, in patch " DynamicTablesPkg: FdtHwInfoParserLib: Mo= ve IRQ map to arch folder" > I think the file DynamicTablesPkg/Library/FdtHwInfoParserLib/Arm/ArmFdtUt= ility.c should be renamed to > DynamicTablesPkg/Library/FdtHwInfoParserLib/Arm/ArmFdtInterrupt.c >=20 > If you agree with the above, I will make the necessary changes before mer= ging. Yes sure, thanks a lot. You might also need the following 2 patches to pass the CI tests: https://edk2.groups.io/g/devel/message/119777 Regards, Pierre >=20 > With that, >=20 > Reviewed-by: Sami Mujawar >=20 > Regards, >=20 > Sami Mujawar >=20 >=20 > =EF=BB=BFOn 19/06/2024, 23:06, "Pierre Gondois" > wrote: >=20 >=20 > The DynamicTables framework has mainly been developed/tested against Arm > architecture. While still trying to have re-usable libraries, opening the > framework to other architectures implies some re-organization. >=20 >=20 > The libraries that are generic enough to be directly re-used are moved > to a Common/ directory. For some libraries, additional arch-specific hook= s > have been added to allow architectures specific modifications. >=20 >=20 > --- >=20 >=20 > Changes can be seen at: > https://github.com/pierregondois/edk2/tree/pg/dyntables_libraries_reorg <= https://github.com/pierregondois/edk2/tree/pg/dyntables_libraries_reorg> >=20 >=20 > --- >=20 >=20 > References: > 1. Staging branch creation: > URL: https://edk2.groups.io/g/devel/message/114790 >=20 >=20 > 2. edk2-staging Repo > URL: https://github.com/tianocore/edk2-staging.git > Branch Name: dynamictables-reorg >=20 >=20 > 3. edk2-platforms Repo > URL: https://github.com/tianocore/edk2-platforms.git > Branch Name: devel-dynamictables-reorg >=20 >=20 > --- >=20 >=20 > Cc: AbdulLateef Attar > > Cc: Girish Mahadevan > > Cc: Jeff Brasen > > Cc: Jeshua Smith > > Cc: Leif Lindholm > > Cc: Meenakshi Aggarwal > > Cc: Pierre Gondois > > Cc: Sami Mujawar > > Cc: Sunil V L = > > Cc: Yeo Reum Yun > >=20 >=20 > Pierre Gondois (15): > DynamicTablesPkg: Acpi: Move generic libraries to common folder > DynamicTablesPkg: Acpi: Prepare common libraries to support other > archs > DynamicTablesPkg: AcpiFadtLib: Prepare to support other archs > DynamicTablesPkg: AcpiDbg2Lib: Prepare to support other archs > DynamicTablesPkg: AcpiSpcrLib: Prepare to support other archs > DynamicTablesPkg: AcpiSratLib: Prepare to support other archs > DynamicTablesPkg: AcpiSsdtCpuTopologyLib: Avoid dependency on GICC > DynamicTablesPkg: DynamicTableManagerDxe: Refactor PresenceArray > DynamicTablesPkg: FdtHwInfoParserLib: Move ARM parsers to Arm > directory > DynamicTablesPkg: FdtHwInfoParserLib: Refactor to prepare for other > archs > DynamicTablesPkg: FdtHwInfoParserLib: Make Pci parser arch neutral > DynamicTablesPkg: FdtHwInfoParserLib: Make Serial Port parser arch > neutral > DynamicTablesPkg: FdtHwInfoParserLib: Move ArmLib.h to ArmGicCParser.c > DynamicTablesPkg: FdtHwInfoParserLib: Move IRQ map to arch folder > DynamicTablesPkg: FdtHwInfoParserLib: Create wrapper to get INTC addr > cells >=20 >=20 > .../Arm/ArmDynamicTableManagerDxe.c | 63 +++ > .../Common/CommonDynamicTableManagerDxe.c | 58 +++ > .../DynamicTableManagerDxe.c | 70 +-- > .../DynamicTableManagerDxe.h | 63 +++ > .../DynamicTableManagerDxe.inf | 7 + > DynamicTablesPkg/DynamicTables.dsc.inc | 64 +-- > .../SsdtCpuTopologyGenerator.h | 147 ------- > .../AcpiDbg2Lib/AcpiDbg2Lib.inf} | 22 +- > .../Common/AcpiDbg2Lib/Arm/ArmDbg2Generator.c | 67 +++ > .../AcpiDbg2Lib/Common/CommonDbg2Generator.c | 59 +++ > .../AcpiDbg2Lib}/Dbg2Generator.c | 24 +- > .../Acpi/Common/AcpiDbg2Lib/Dbg2Generator.h | 56 +++ > .../AcpiFadtLib/AcpiFadtLib.inf} | 16 +- > .../Common/AcpiFadtLib/Arm/ArmFadtGenerator.c | 126 ++++++ > .../AcpiFadtLib/Common/CommonFadtGenerator.c | 46 ++ > .../AcpiFadtLib}/FadtGenerator.c | 86 +--- > .../Acpi/Common/AcpiFadtLib/FadtGenerator.h | 35 ++ > .../AcpiMcfgLib/AcpiMcfgLib.inf} | 9 +- > .../AcpiMcfgLib}/McfgGenerator.c | 0 > .../AcpiPcctLib/AcpiPcctLib.inf} | 2 +- > .../AcpiPcctLib}/PcctGenerator.c | 0 > .../AcpiPcctLib}/PcctGenerator.h | 0 > .../AcpiPpttLib/AcpiPpttLib.inf} | 2 +- > .../AcpiPpttLib}/PpttGenerator.c | 0 > .../AcpiPpttLib}/PpttGenerator.h | 0 > .../AcpiRawLib/AcpiRawLib.inf} | 9 +- > .../AcpiRawLib}/RawGenerator.c | 0 > .../AcpiSpcrLib/AcpiSpcrLib.inf} | 9 +- > .../AcpiSpcrLib}/SpcrGenerator.c | 2 +- > .../AcpiSratLib/AcpiSratLib.inf} | 9 +- > .../Common/AcpiSratLib/Arm/ArmSratGenerator.c | 262 +++++++++++ > .../AcpiSratLib/Common/CommonSratGenerator.c | 77 ++++ > .../AcpiSratLib}/SratGenerator.c | 214 +-------- > .../Acpi/Common/AcpiSratLib/SratGenerator.h | 59 +++ > .../Arm/ArmSsdtCpuTopologyGenerator.c | 408 ++++++++++++++++++ > .../SsdtCpuTopologyGenerator.c | 341 ++------------- > .../SsdtCpuTopologyGenerator.h | 343 +++++++++++++++ > .../SsdtCpuTopologyLib.inf} | 9 +- > .../AcpiSsdtPcieLib}/SsdtPcieGenerator.c | 2 +- > .../AcpiSsdtPcieLib}/SsdtPcieGenerator.h | 0 > .../AcpiSsdtPcieLib/SsdtPcieLib.inf} | 2 +- > .../SsdtSerialPortGenerator.c | 0 > .../SsdtSerialPortLib.inf} | 6 +- > .../Arm/ArmFdtHwInfoParser.c | 83 ++++ > .../FdtHwInfoParserLib/Arm/ArmFdtUtility.c | 118 +++++ > .../{ =3D> Arm}/BootArch/ArmBootArchParser.c | 2 +- > .../{ =3D> Arm}/BootArch/ArmBootArchParser.h | 0 > .../GenericTimer/ArmGenericTimerParser.c | 4 +- > .../GenericTimer/ArmGenericTimerParser.h | 0 > .../{ =3D> Arm}/Gic/ArmGicCParser.c | 5 +- > .../{ =3D> Arm}/Gic/ArmGicCParser.h | 0 > .../{ =3D> Arm}/Gic/ArmGicDParser.c | 4 +- > .../{ =3D> Arm}/Gic/ArmGicDParser.h | 0 > .../{ =3D> Arm}/Gic/ArmGicDispatcher.c | 12 +- > .../{ =3D> Arm}/Gic/ArmGicDispatcher.h | 0 > .../{ =3D> Arm}/Gic/ArmGicItsParser.c | 4 +- > .../{ =3D> Arm}/Gic/ArmGicItsParser.h | 0 > .../{ =3D> Arm}/Gic/ArmGicMsiFrameParser.c | 4 +- > .../{ =3D> Arm}/Gic/ArmGicMsiFrameParser.h | 0 > .../{ =3D> Arm}/Gic/ArmGicRParser.c | 4 +- > .../{ =3D> Arm}/Gic/ArmGicRParser.h | 0 > .../FdtHwInfoParserLib/FdtHwInfoParser.c | 78 +--- > .../FdtHwInfoParserLib/FdtHwInfoParser.h | 27 ++ > .../FdtHwInfoParserInclude.h | 1 - > .../FdtHwInfoParserLib/FdtHwInfoParserLib.inf | 48 ++- > .../Library/FdtHwInfoParserLib/FdtUtility.c | 71 --- > .../Library/FdtHwInfoParserLib/FdtUtility.h | 30 ++ > ...igSpaceParser.c =3D> PciConfigSpaceParser.c} | 19 +- > ...igSpaceParser.h =3D> PciConfigSpaceParser.h} | 10 +- > ...mSerialPortParser.c =3D> SerialPortParser.c} | 16 +- > ...mSerialPortParser.h =3D> SerialPortParser.h} | 8 +- > 71 files changed, 2248 insertions(+), 1074 deletions(-) > create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/Arm/Ar= mDynamicTableManagerDxe.c > create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/Common= /CommonDynamicTableManagerDxe.c > create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/Dynami= cTableManagerDxe.h > delete mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCpuTopologyL= ibArm/SsdtCpuTopologyGenerator.h > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiDbg2LibArm/AcpiDbg2LibArm.i= nf =3D> Common/AcpiDbg2Lib/AcpiDbg2Lib.inf} (75%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiDbg2Lib/Arm/A= rmDbg2Generator.c > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiDbg2Lib/Commo= n/CommonDbg2Generator.c > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiDbg2LibArm =3D> Common/Acpi= Dbg2Lib}/Dbg2Generator.c (93%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiDbg2Lib/Dbg2G= enerator.h > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiFadtLibArm/AcpiFadtLibArm.i= nf =3D> Common/AcpiFadtLib/AcpiFadtLib.inf} (75%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiFadtLib/Arm/A= rmFadtGenerator.c > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiFadtLib/Commo= n/CommonFadtGenerator.c > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiFadtLibArm =3D> Common/Acpi= FadtLib}/FadtGenerator.c (87%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiFadtLib/FadtG= enerator.h > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiMcfgLibArm/AcpiMcfgLibArm.i= nf =3D> Common/AcpiMcfgLib/AcpiMcfgLib.inf} (85%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiMcfgLibArm =3D> Common/Acpi= McfgLib}/McfgGenerator.c (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPcctLibArm/AcpiPcctLibArm.i= nf =3D> Common/AcpiPcctLib/AcpiPcctLib.inf} (90%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPcctLibArm =3D> Common/Acpi= PcctLib}/PcctGenerator.c (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPcctLibArm =3D> Common/Acpi= PcctLib}/PcctGenerator.h (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPpttLibArm/AcpiPpttLibArm.i= nf =3D> Common/AcpiPpttLib/AcpiPpttLib.inf} (90%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPpttLibArm =3D> Common/Acpi= PpttLib}/PpttGenerator.c (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiPpttLibArm =3D> Common/Acpi= PpttLib}/PpttGenerator.h (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiRawLibArm/AcpiRawLibArm.inf= =3D> Common/AcpiRawLib/AcpiRawLib.inf} (85%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiRawLibArm =3D> Common/AcpiR= awLib}/RawGenerator.c (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSpcrLibArm/AcpiSpcrLibArm.i= nf =3D> Common/AcpiSpcrLib/AcpiSpcrLib.inf} (86%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSpcrLibArm =3D> Common/Acpi= SpcrLib}/SpcrGenerator.c (96%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSratLibArm/AcpiSratLibArm.i= nf =3D> Common/AcpiSratLib/AcpiSratLib.inf} (74%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiSratLib/Arm/A= rmSratGenerator.c > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiSratLib/Commo= n/CommonSratGenerator.c > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSratLibArm =3D> Common/Acpi= SratLib}/SratGenerator.c (75%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiSratLib/SratG= enerator.h > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiSsdtCpuTopolo= gyLib/Arm/ArmSsdtCpuTopologyGenerator.c > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtCpuTopologyLibArm =3D> = Common/AcpiSsdtCpuTopologyLib}/SsdtCpuTopologyGenerator.c (79%) > create mode 100644 DynamicTablesPkg/Library/Acpi/Common/AcpiSsdtCpuTopolo= gyLib/SsdtCpuTopologyGenerator.h > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtCpuTopologyLibArm/SsdtC= puTopologyLibArm.inf =3D> Common/AcpiSsdtCpuTopologyLib/SsdtCpuTopologyLib.= inf} (80%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtPcieLibArm =3D> Common/= AcpiSsdtPcieLib}/SsdtPcieGenerator.c (96%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtPcieLibArm =3D> Common/= AcpiSsdtPcieLib}/SsdtPcieGenerator.h (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtPcieLibArm/SsdtPcieLibA= rm.inf =3D> Common/AcpiSsdtPcieLib/SsdtPcieLib.inf} (91%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtSerialPortLibArm =3D> C= ommon/AcpiSsdtSerialPortLib}/SsdtSerialPortGenerator.c (100%) > rename DynamicTablesPkg/Library/Acpi/{Arm/AcpiSsdtSerialPortLibArm/SsdtSe= rialPortLibArm.inf =3D> Common/AcpiSsdtSerialPortLib/SsdtSerialPortLib.inf}= (87%) > create mode 100644 DynamicTablesPkg/Library/FdtHwInfoParserLib/Arm/ArmFdt= HwInfoParser.c > create mode 100644 DynamicTablesPkg/Library/FdtHwInfoParserLib/Arm/ArmFdt= Utility.c > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/BootArch/A= rmBootArchParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/BootArch/A= rmBootArchParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/GenericTim= er/ArmGenericTimerParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/GenericTim= er/ArmGenericTimerParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= CParser.c (96%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= CParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= DParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= DParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= Dispatcher.c (92%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= Dispatcher.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= ItsParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= ItsParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= MsiFrameParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= MsiFrameParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= RParser.c (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/{ =3D> Arm}/Gic/ArmGic= RParser.h (100%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/Pci/{ArmPciConfigSpace= Parser.c =3D> PciConfigSpaceParser.c} (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/Pci/{ArmPciConfigSpace= Parser.h =3D> PciConfigSpaceParser.h} (93%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/{ArmSerialPortP= arser.c =3D> SerialPortParser.c} (95%) > rename DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/{ArmSerialPortP= arser.h =3D> SerialPortParser.h} (89%) >=20 >=20 > -- > 2.25.1 >=20 >=20 >=20 >=20 >=20 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119780): https://edk2.groups.io/g/devel/message/119780 Mute This Topic: https://groups.io/mt/106770151/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-