Re: [PATCH v2 3/8] RISC-V: Remove NR_CPUs check during hartid search from DT

From: Christoph Hellwig
Date: Tue Jan 15 2019 - 08:52:58 EST


On Tue, Jan 08, 2019 at 01:38:33AM -0800, Atish Patra wrote:
> In non-smp configuration, hartid can be higher that NR_CPUS.
> riscv_of_processor_hartid should not be compared to hartid to
> NR_CPUS in that case. Moreover, this function checks all the
> DT properties of a hart node. NR_CPUS comparison seems out of
> place.
>
> Signed-off-by: Atish Patra <atish.patra@xxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>