Re: [PATCH v3 02/12] RISC-V: Filter ISA and MMU values in cpuinfo

From: Christoph Hellwig
Date: Mon Sep 10 2018 - 07:24:28 EST


On Thu, Sep 06, 2018 at 01:05:25AM -0700, Atish Patra wrote:
> +#elif defined(CONFIG_64BIT)
> + if ((strcmp(mmu_type, "riscv,sv39") != 0)
> + && (strcmp(mmu_type, "riscv,sv48") != 0))
> + return;

This should be:

if (strcmp(mmu_type, "riscv,sv39") != 0 &&
strcmp(mmu_type, "riscv,sv48") != 0)
return;

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>