From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by mx.groups.io with SMTP id smtpd.web12.21269.1658936742394382033 for ; Wed, 27 Jul 2022 08:45:42 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=jXLgK4YN; spf=pass (domain: kernel.org, ip: 145.40.68.75, mailfrom: ardb@kernel.org) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id C7765B821AA for ; Wed, 27 Jul 2022 15:45:40 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 92AF0C43140 for ; Wed, 27 Jul 2022 15:45:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1658936739; bh=SmWYQ+nUDu+v2pwWdLuX4UOZOr0mUU3efKoGOuLrQkM=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=jXLgK4YNFOwM57X2w8w+QaR6VfEfTMGV70VP+TrJ6J3Lq9X/zz9PSZus9zH1EzsEj J+XEAA1CeTZ/DNyHGYrOKS6Svm2n8crvrF38I6daugU1ss6Ks2uLNXY172zdn1ai3j BC8706A2W0cBpqkxg5paghbLPRCKrI1H1jg8aLaGEm7wrPtY7NL+F5VPB/C6SKihYY Qw2dZu43/QOKBTuR0wbhIB8JJb9eV7L9Wx2252u3o0qyzFZYKNVRN0FNzfP8ENdqZ7 O/2QaYt2FlV6DCr9ha03JpQOQYiUtul4oACs2wJF5D1hlBi/45XYeUlpVv1/QlgDdg o4ajFz6hJYcBw== Received: by mail-oi1-f181.google.com with SMTP id n133so11133588oib.0 for ; Wed, 27 Jul 2022 08:45:39 -0700 (PDT) X-Gm-Message-State: AJIora+6sXPgPrJH3hYUTIhXDSYP9C/1f7KuKmvPknwdkhu4D0aiZ6S+ L88IZSOqiSZC2/xQjxt9GttO60iYB2LtmkgTmCA= X-Google-Smtp-Source: AGRyM1vBkxuZJNxoA1gBkzR20W4lhN2+Mo8mxbw9SvvQy9RT09KV2L24QJzUHY9uv0XKwXKtutB5Zv/P53bo3xBjbP0= X-Received: by 2002:a05:6808:300b:b0:337:b697:b077 with SMTP id ay11-20020a056808300b00b00337b697b077mr2198291oib.126.1658936738617; Wed, 27 Jul 2022 08:45:38 -0700 (PDT) MIME-Version: 1.0 References: <20220714165536.31344-1-sami.mujawar@arm.com> <17034793F9AA5ACF.21682@groups.io> <7CD98743-53E6-4F15-BD0B-B8EAB33B756D@arm.com> In-Reply-To: <7CD98743-53E6-4F15-BD0B-B8EAB33B756D@arm.com> From: "Ard Biesheuvel" Date: Wed, 27 Jul 2022 08:45:27 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH edk2-platforms v4 00/17] Fix build breaks and update IORT revision macro To: edk2-devel-groups-io , Sami Mujawar Cc: "ardb+tianocore@kernel.org" , "quic_llindhol@quicinc.com" , "Meenakshi Aggarwal (meenakshi.aggarwal@nxp.com)" , "shuyiqi@phytium.com.cn" , "jialing@phytium.com.cn" , "xiepeng@phytium.com.cn" , Pierre Gondois , Matteo Carlini , Akanksha Jain , Ben Adderson , "gaoliming@byosoft.com.cn" , Jeremy Linton , Thomas Abraham , Chandni Cherukuri , "nhi@os.amperecomputing.com" , "vunguyen@os.amperecomputing.com" , "thang@os.amperecomputing.com" , "chuong@os.amperecomputing.com" , nd Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, 27 Jul 2022 at 02:21, Sami Mujawar wrote: > > Hi All, > > Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3458 > > The edk2 patch series for IORT Rev E.d updates has been submitted to the = mailing list and has been reviewed at > https://edk2.groups.io/g/devel/message/91361 > > The corresponding edk2-platforms series is posted on the mailing list at = https://edk2.groups.io/g/devel/message/91543 > > The edk2-platforms series fixes a number of build breaks as well as makes= the necessary updates to reflect the changes due to IORT Rev E.d updates. > > I would prefer this patch series to be merged in the edk2-stable202208 ta= g. Therefore, requesting review for the edk2-platform patches from the resp= ective maintainers for the following modules: > - Silicon/AMD/Styx > - Silicon/Socionext/SynQuacer > - Platform/LeMaker > - Platform/Socionext > - Silicon/NXP & Platform/NXP/* > For the associated fixes for all the above: Reviewed-by: Ard Biesheuvel Feel free to push all of these changes when you feel it is appropriate. > =EF=BB=BFOn 19/07/2022, 17:24, "devel@edk2.groups.io on behalf of Sami Mu= jawar via groups.io" wrote: > > Hi All, > > The IORT Rev E.d seriesfor edk2 has received the r-b from the respect= ive > maintainers. > > Can the following package maintainers review this series and provide > feedback, please? > > - Silicon/AMD/Styx > > - Silicon/Socionext/SynQuacer > > - Platform/LeMaker > > - Platform/Socionext > > - Silicon/NXP & Platform/NXP/* > > Regards, > > Sami Mujawar > > On 14/07/2022 05:55 pm, Sami Mujawar wrote: > > This v4 series includes 5 additional patches (13-17) that > > fix various build breaks caused by changes in edk2 repo. > > The patches 1-12 have not changed between v3 series, except > > updates to the r-b received. > > > > The v3 patch series renames the EFI_ACPI_IO_REMAPPING_TABLE_REV0 > > macro to EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00 based on the > > feedback received. It also includes an additional patch that fixes > > the build break in CelloBoard due to missing VariablePolicyHelperLi= b > > dependency. > > > > The v2 patch series fixes the build break in Ampere/Jade and > > Pythium/FT2000-4 due to missing VariableFlashInfoLib dependency > > and also updates the IORT revision macro. > > > > The IORT Rev E.d specification updates the IORT table revision > > to 5. Following this the IORT table revision macro > > EFI_ACPI_IO_REMAPPING_TABLE_REVISION which was at Rev 0 has > > been renamed to EFI_ACPI_IO_REMAPPING_TABLE_REV0. Therefore, > > this series updates the following platforms to reflect this > > renaming: > > - FVP > > - Morello FVP > > - SGI > > - AMD/Styx > > - Socionext/SynQuacer > > - RaspberryPi > > - N1SDP > > - Ampere/Jade > > - Pythium/FT2000-4 > > > > Note: This patch series is dependent on the edk2 patch series: > > "[PATCH v6 0/8] IORT Rev E.d specification updates" > > (https://edk2.groups.io/g/devel/message/91361), which must > > be merged in edk2 before this patch series can be integrated. > > > > The changes can be seen at: > > https://github.com/samimujawar/edk2-platforms/tree/1527_iort_rev_ed= _platforms_v4 > > > > Sami Mujawar (17): > > Platform/ARM: FVP: Update for IORT revision macro renaming > > Platform/ARM: Morello: Update for IORT revision macro renaming > > Platform/ARM: SGI: Update for IORT revision macro renaming > > Silicon/AMD/Styx: Update for IORT revision macro renaming > > Silicon/Socionext/SynQuacer: Update for IORT revision macro rena= ming > > Platform/RaspberryPi: Update for IORT revision macro renaming > > Platform/ARM: N1SDP: Update for IORT revision macro renaming > > Silicon/Ampere: Add VariableFlashInfoLib > > Platform/Ampere: JadePkg: Update for IORT revision macro renamin= g > > Silicon/Phytium: Add VariableFlashInfoLib > > Silicon/Phytium: FT2000-4Pkg: Update for IORT revision macro ren= aming > > Platform/LeMaker: Fix missing dependency on VariablePolicyHelper= Lib > > Platform/Socionext: Fix missing dependency on VariableFlashInfoL= ib > > Silicon/NXP: Fix missing dependency on VariableFlashInfoLib > > Platform/NXP/LS1043aRdbPk: Fix build break due to change in > > ARM_CORE_INFO > > Platform/NXP/LS1046aFrwyPkg: Fix build break due to change in > > ARM_CORE_INFO > > Platform/NXP/LX2160aRdbPkg: Fix build break due to change in > > ARM_CORE_INFO > > > > Platform/ARM/Morello/ConfigurationManager/ConfigurationManagerDxe= /ConfigurationManagerFvp.c | 2 +- > > Platform/ARM/N1Sdp/ConfigurationManager/ConfigurationManagerDxe/C= onfigurationManager.c | 2 +- > > Platform/ARM/SgiPkg/AcpiTables/Iort.aslc = | 2 +- > > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManage= rDxe/ConfigurationManager.c | 2 +- > > Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c = | 2 +- > > Platform/LeMaker/CelloBoard/CelloBoard.dsc = | 1 + > > Platform/NXP/LS1043aRdbPkg/Library/ArmPlatformLib/ArmPlatformLib.= c | 2 +- > > Platform/NXP/LS1046aFrwyPkg/Library/ArmPlatformLib/ArmPlatformLib= .c | 2 +- > > Platform/NXP/LX2160aRdbPkg/Library/ArmPlatformLib/ArmPlatformLib.= c | 2 +- > > Platform/RaspberryPi/AcpiTables/Iort.aslc = | 2 +- > > Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc = | 1 + > > Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc = | 2 +- > > Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc = | 1 + > > Silicon/NXP/NxpQoriqLs.dsc.inc = | 1 + > > Silicon/Phytium/FT2000-4Pkg/Drivers/AcpiTables/Iort.aslc = | 2 +- > > Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc = | 1 + > > Silicon/Socionext/SynQuacer/AcpiTables/Iort.aslc = | 2 +- > > 17 files changed, 17 insertions(+), 12 deletions(-) > > > > > > > > > > >=20 > >