Sunil, I am glad that you don't have concern about the direction. "RiscV" is good to me.
Thanks, Ray > -----Original Message----- > From: Sunil V L <[email protected]> > Sent: Friday, January 12, 2024 6:19 PM > To: Ni, Ray <[email protected]> > Cc: [email protected]; [email protected]; Kinney, Michael D > <[email protected]>; Laszlo Ersek <[email protected]>; Gerd > Hoffmann <[email protected]> > Subject: Re: RFC: Folder layout change in UefiCpuPkg > > Hi Ray, > > On Fri, Jan 12, 2024 at 09:12:34AM +0000, Ni, Ray wrote: > > Sunil, > > I would like to hear your feedback regarding locations of following RiscV64 > components in UefiCpuPkg: > > * UefiCpuPkg/Library/BaseRiscV64CpuExceptionHandlerLib/ > > * UefiCpuPkg/Library/BaseRiscV64CpuTimerLib/ > > * UefiCpuPkg/CpuDxeRiscV64/ > > * UefiCpuPkg/CpuTimerDxeRiscV64/ > > > > I would like to move them to the following new locations accordingly: > > * UefiCpuPkg/Library/CpuExceptionHandlerLib/RiscV64/ > > * UefiCpuPkg/Library/CpuTimerLib/RiscV64/ > > * UefiCpuPkg/CpuDxe/RiscV64/ > > * UefiCpuPkg/CpuTimerDxe/RiscV64/ > > > > > > I want to avoid too many similar drivers in root folder, and too many > > libraries > in Library folder. > > > > Movement of the first 3 ones put the RiscV components under existing > folders. > > Movement of the last one creates the UefiCpuPkg/CpuTimerDxe folder, that > could be potentially shared by other archs as well. > > > > I raised similar comments to Chao Li who is working on LoongArch upstream. > > > > The location movement follows the 2nd pattern defined by edk2 coding > standard: > > Driver's location could be: > > <Feature><Phase>[<CpuArch>[<Vendor>]] > > or > > <Feature><Phase>[/<CpuArch>[/<Vendor>]] > > > > Library's location could be: > > > > <Phase>[<CpuArch>[<Vendor>]]<LibraryClassName>[<Dependency>] > > > > or > > > > <Phase><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]] > > > > > Your proposal looks good to me except better to keep directory name as > RiscV as in other packages. > > Thanks, > Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113783): https://edk2.groups.io/g/devel/message/113783 Mute This Topic: https://groups.io/mt/103679850/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
