On Wed, Feb 08, 2023 at 06:02:44PM +0000, Kinney, Michael D wrote: > Hi Sunil, > > One comment on the name of this module. CpuTimerDxe is a very generic name, > but the > Implementation is for RISCV only and depends on SBL services. Correct? > > If I look at other TimerDxe modules, they have in their name some indication > of > the timer hardware/software dependencies in their name, > > * LocalApicTimerDxe.inf > * HpetTimerDxe.inf > > How about RiscVTimerDxe or RiscVSbiTimerLib? > > Mike > Thanks!, Mike. Your suggestion makes sense. Since most of the drivers in UefiCpuPkg were named as Cpu*, I named it like this. Let me change it to RiscVTimerDxe.
Thanks! Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#99808): https://edk2.groups.io/g/devel/message/99808 Mute This Topic: https://groups.io/mt/96593498/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
