On Fri, 22 Dec 2023 at 03:37, Ni, Ray <ray...@intel.com> wrote: > > Good to me. > > > > Thanks, > > Ray > > From: Chao Li <lic...@loongson.cn> > Sent: Friday, December 22, 2023 9:15 AM > To: devel@edk2.groups.io; a...@kernel.org; Ni, Ray <ray...@intel.com> > Cc: Kumar, Rahul R <rahul.r.ku...@intel.com>; Gerd Hoffmann > <kra...@redhat.com>; Leif Lindholm <quic_llind...@quicinc.com>; Ard > Biesheuvel <ardb+tianoc...@kernel.org>; Sami Mujawar <sami.muja...@arm.com> > Subject: Re: [edk2-devel] [PATCH v4 19/37] UefiCpuPkg: Add MMIO method in > CpuIo2Dxe > > > > > > > > Thanks, > Chao > > On 2023/12/21 21:59, Ard Biesheuvel wrote: > > On Thu, 21 Dec 2023 at 13:41, Chao Li <lic...@loongson.cn> wrote: > > > > ... > > A given platform will either be able to use port IO, or it will need > > to use MMIO translation. So a driver that can do both based on a PCD > > is not very useful, it makes more sense to have two different drivers, > > and the platform incorporates the one it can use. > > > > Ok, I get it, I think you are more inclined to move the ArmCpuIo2Dxe to a new > place and rename it, maybe: CpuMmio2Dxe, while some platforms choose the > different CpuIo2 drvier by themselves when building FW, right? > >
OK. Note that there are 15 references to ArmPkg/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf in edk2-platforms, so if you move the existing driver, please - create the new one by making a copy - update all edk2 and edk2-platforms users to refer to the new version - delete the old version -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112843): https://edk2.groups.io/g/devel/message/112843 Mute This Topic: https://groups.io/mt/103261693/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
