Re: [PATCH] PCI: mobiveil: Fix csr_read/write build issue

2019-09-27 Thread Christoph Hellwig
On Thu, Sep 26, 2019 at 10:29:34AM +0100, Andrew Murray wrote: > Though I'd be just as happy if the csr_[read,write][l,] functions were > renamed to mobiveil_csr_[read,write][l,]. Please do that instead, using such generic names as csr_* in a driver is a bad idea, with or without a __ prefix.

Re: [PATCH] PCI: mobiveil: Fix csr_read/write build issue

2019-09-26 Thread Andrew Murray
On Wed, Sep 25, 2019 at 10:21:21PM +0800, Kefeng Wang wrote: > The riscv has csr_read/write macro, see arch/riscv/include/asm/csr.h, > the same function naming will cause build error, rename them to > __csr_read/write to fix it. > > drivers/pci/controller/pcie-mobiveil.c:238:69: error: macro

[PATCH] PCI: mobiveil: Fix csr_read/write build issue

2019-09-25 Thread Kefeng Wang
The riscv has csr_read/write macro, see arch/riscv/include/asm/csr.h, the same function naming will cause build error, rename them to __csr_read/write to fix it. drivers/pci/controller/pcie-mobiveil.c:238:69: error: macro "csr_read" passed 3 arguments, but takes just 1 static u32