> -----Original Message----- > From: Schaefer, Daniel > Sent: Wednesday, September 29, 2021 7:36 AM > To: Chang, Abner (HPS SW/FW Technologist) <[email protected]>; > [email protected] > Cc: Michael D Kinney <[email protected]>; Liming Gao > <[email protected]>; Zhiguang Liu <[email protected]>; Ard > Biesheuvel <[email protected]>; Leif Lindholm > <[email protected]>; Sami Mujawar <[email protected]>; Gerd > Hoffmann <[email protected]>; Sunil V L <[email protected]> > Subject: Re: [PATCH V2 7/9] MdePkg: Add PcdPciMmio32(64)Translation > PCDs > > Also here. I think this should be combined into patch 8. Same reason as the previous one.
Because there are different maintainers for MdePkg and Arm*Pkg, thus we have to separate the patches for them based on Maintainers.txt. MdePkg owners just give their reviewed-by for the MdePkg changes but not Arm*Pkg changes. So those changes can't be in the same patch. The cover letter gives each module maintainers the whole picture of this patch set. So they can understand the reason having the changes on their module. Abner > > On 9/28/21 16:31, Abner Chang wrote: > > PcdPciMmio32Translation and PcdPciMmio64Translation PCDs are added > > to MdePkg as the common PCDs for ARM and RSIC-V archs. > > > > The one under ArmPkg is removed in the next patch. > > > > Signed-off-by: Abner Chang <[email protected]> > > Cc: Michael D Kinney <[email protected]> > > Cc: Liming Gao <[email protected]> > > Cc: Zhiguang Liu <[email protected]> > > Cc: Ard Biesheuvel <[email protected]> > > Cc: Leif Lindholm <[email protected]> > > Cc: Sami Mujawar <[email protected]> > > Cc: Gerd Hoffmann <[email protected]> > > Cc: Daniel Schaefer <[email protected]> > > Cc: Sunil V L <[email protected]> > > --- > > MdePkg/MdePkg.dec | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > > index 08d259764a..9df95abc50 100644 > > --- a/MdePkg/MdePkg.dec > > +++ b/MdePkg/MdePkg.dec > > @@ -2306,6 +2306,14 @@ > > # @Prompt PCI I/O Memory Map Window Base Address. > > > gEfiMdePkgTokenSpaceGuid.PcdPciIoTranslation|0x0|UINT64|0x00000040 > > > > + ## This value is used for the 32-bit PCI memory map I/O base address > translation. > > + # @Prompt 32-bit PCI Memory Map I/O Base Address translation. > > + > gEfiMdePkgTokenSpaceGuid.PcdPciMmio32Translation|0x0|UINT64|0x0000 > 0041 > > + > > + ## This value is used for the 64-bit PCI memory map I/O base address > translation. > > + # @Prompt 64-bit PCI Memory Map I/O Base Address translation. > > + > gEfiMdePkgTokenSpaceGuid.PcdPciMmio64Translation|0x0|UINT64|0x0000 > 0042 > > + > > ## This value is used to set the size of PCI express hierarchy. The > > default > is 256 MB. > > # @Prompt PCI Express Base Size. > > > gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseSize|0x10000000|UINT64|0x > 0000000f > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81249): https://edk2.groups.io/g/devel/message/81249 Mute This Topic: https://groups.io/mt/85920592/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
