diff mbox series

slimbus: qcom: fix incompatible pointer warning

Message ID 20171231142311.7952-1-srinivas.kandagatla@linaro.org
State Accepted
Commit db809859c8cee415293b830e67178f526d1eb2be
Headers show
Series slimbus: qcom: fix incompatible pointer warning | expand

Commit Message

Srinivas Kandagatla Dec. 31, 2017, 2:23 p.m. UTC 
From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>


One of the pointer passed to dmam_alloc_coherent seems to be
phys_addr_t * instead of dma_addr_t *. This address will be
used by dma apis, so change this to proper type.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

---
 drivers/slimbus/qcom-ctrl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.15.0

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Greg Kroah-Hartman Dec. 31, 2017, 4:52 p.m. UTC | #1 
On Sun, Dec 31, 2017 at 02:23:11PM +0000, srinivas.kandagatla@linaro.org wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

> 

> One of the pointer passed to dmam_alloc_coherent seems to be

> phys_addr_t * instead of dma_addr_t *. This address will be

> used by dma apis, so change this to proper type.

> 

> Reported-by: kbuild test robot <fengguang.wu@intel.com>

> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>


Also reported-by Arnd :(

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/drivers/slimbus/qcom-ctrl.c b/drivers/slimbus/qcom-ctrl.c
index 35ad70dbfe3a..f51de1277912 100644
--- a/drivers/slimbus/qcom-ctrl.c
+++ b/drivers/slimbus/qcom-ctrl.c
@@ -93,7 +93,7 @@ 
 
 struct slim_ctrl_buf {
 	void		*base;
-	phys_addr_t	phy;
+	dma_addr_t	phy;
 	spinlock_t	lock;
 	int		head;
 	int		tail;