Re: [PATCH] firmware: qcom_scm: fix error for incompatible pointer

2019-07-23 Thread Minwoo Im
> > > > We just can cast phys_addr_t to dma_addr_t here. > > > > > > IME, casting is rarely a proper solution. > > > > *nod* > > > > ptr_phys probably should be a dma_addr_t. Unless this driver is so > > magic that it really wants a physical and not a dma address, in which > > case it needs to

Re: [PATCH] firmware: qcom_scm: fix error for incompatible pointer

2019-07-22 Thread Bjorn Andersson
On Mon 22 Jul 02:30 PDT 2019, Christoph Hellwig wrote: > On Mon, Jul 22, 2019 at 10:38:55AM +0200, Marc Gonzalez wrote: > > > In file included from drivers/firmware/qcom_scm.c:12:0: > > > ./include/linux/dma-mapping.h:636:21: note: expected ‘dma_addr_t * {aka > > > long long unsigned int *}’ but

Re: [PATCH] firmware: qcom_scm: fix error for incompatible pointer

2019-07-22 Thread Christoph Hellwig
On Mon, Jul 22, 2019 at 10:38:55AM +0200, Marc Gonzalez wrote: > > In file included from drivers/firmware/qcom_scm.c:12:0: > > ./include/linux/dma-mapping.h:636:21: note: expected ‘dma_addr_t * {aka > > long long unsigned int *}’ but argument is of type ‘phys_addr_t * {aka > > unsigned int *}’ >

Re: [PATCH] firmware: qcom_scm: fix error for incompatible pointer

2019-07-22 Thread Marc Gonzalez
Adding people who have worked on drivers/firmware/qcom_scm.c or DMA On 19/07/2019 15:43, Minwoo Im wrote: > The following error can happen when trying to build it: > > ``` > drivers/firmware/qcom_scm.c: In function ‘qcom_scm_assign_mem’: > drivers/firmware/qcom_scm.c:460:47: error: passing argum

[PATCH] firmware: qcom_scm: fix error for incompatible pointer

2019-07-19 Thread Minwoo Im
The following error can happen when trying to build it: ``` drivers/firmware/qcom_scm.c: In function ‘qcom_scm_assign_mem’: drivers/firmware/qcom_scm.c:460:47: error: passing argument 3 of ‘dma_alloc_coherent’ from incompatible pointer type [-Werror=incompatible-pointer-types] ptr = dma_alloc_