Re: [RFC 2/3] RISC-V:Support per-hart timebase-frequency

From: Palmer Dabbelt
Date: Fri Sep 28 2018 - 20:20:48 EST


On Mon, 17 Sep 2018 07:23:08 PDT (-0700), Christoph Hellwig wrote:
On Fri, Sep 14, 2018 at 02:54:55PM -0700, Atish Patra wrote:
Follow the updated DT specs and read the timebase-frequency
from the boot cpu. Keep the old DT reading as well for backward
compatibility. This patch is rework of old patch from Palmer.

Signed-off-by: Atish Patra <atish.patra@xxxxxxx>

This setup looks a bit odd because it keeps blindly overwriting
riscv_timebase for every cpu found. Shouldn't we at least check that
they all match for now as the rest of the port assumes that?

I agree. There should be at least a warning if they don't match, but ideally we'd just support per-CPU timebases. We have those systems already, they just don't boot Linux (at least, not yet :)).