Re: [PATCH v5 12/21] libnvdimm: namespace indices: read and validate

From: Dan Williams
Date: Wed Jun 10 2015 - 11:54:50 EST


On Mon, Jun 8, 2015 at 11:39 PM, Christoph Hellwig <hch@xxxxxx> wrote:
>> +#include <asm-generic/io-64-nonatomic-lo-hi.h>
>> +
>> +#ifndef __io_virt
>> +#define __io_virt(x) ((void __force *) (x))
>> +#endif
>
> NAK. driver code mus never include asm-generic headers directly,
> and __io_virt isn't something that's up to a driver to redefine either.
>
> I think we really need the memremap series in first before trying to
> do the things done in this patch.

I was abusing read[wlq] and write[wlq]. All this patch needs is a
conversion to use __leXXto_cpu and __cpu_to_leXX helpers.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/