On Tue, Dec 21, 2021 at 2:32 PM Alistair Francis <alistai...@gmail.com> wrote: > > On Mon, Dec 20, 2021 at 5:52 PM Bin Meng <bmeng...@gmail.com> wrote: > > > > On Thu, Dec 16, 2021 at 12:55 PM Alistair Francis > > <alistair.fran...@opensource.wdc.com> wrote: > > > > > > From: Alistair Francis <alistair.fran...@wdc.com> > > > > > > As per the device tree specification let's set the clock-frequency for > > > the virt CPUs. > > > > > > QEMU doesn't really have an exact clock, so let's just 1000000 as it's a > > > nice round number and matches the sifive_u CLINT_TIMEBASE_FREQ. > > > > > > Resolves: https://gitlab.com/qemu-project/qemu/-/issues/472 (RISC-V virt) > > > > I tend to think this issue is not valid, as the comment in the issue > > says the value should reflect the running frequency. > > > > Note the "clock-frequency" was once in the codes but was intentionally > > removed before because it is not needed: > > > > See commit 7ae05377b85f (" riscv: hw: Drop "clock-frequency" property > > of cpu nodes") > > Hmmm... Good point. > > But if the device tree spec says it's required then I think we should > include it. Even if it isn't super relevant for us
The Linux kernel upstream RISC-V DTS files also don't have "clock-frequency" properties. I doubt there is benefit to provide one. Regards, Bin
