From 5a07cf5e7ae8cfe1266ef4fc445c947cb649a081 Mon Sep 17 00:00:00 2001 From: Lokesh Vutla Date: Thu, 16 Aug 2018 18:41:49 +0530 Subject: [PATCH] pinctrl: Fix build warning with ARM64 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Following build warning appears when pinctrl-single is built for ARM64: In file included from drivers/pinctrl/pinctrl-single.c:10:0: drivers/pinctrl/pinctrl-single.c: In function ‘single_configure_pins’: ./arch/arm/include/asm/io.h:43:28: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] #define __arch_getw(a) (*(volatile unsigned short *)(a)) Fix this by using phys_addr_t for variable reg instead of u32 Signed-off-by: Lokesh Vutla --- drivers/pinctrl/pinctrl-single.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c index 296eb63..d80c6ed 100644 --- a/drivers/pinctrl/pinctrl-single.c +++ b/drivers/pinctrl/pinctrl-single.c @@ -43,13 +43,13 @@ static int single_configure_pins(struct udevice *dev, { struct single_pdata *pdata = dev->platdata; int count = size / sizeof(struct single_fdt_pin_cfg); - int n, reg; + phys_addr_t n, reg; u32 val; for (n = 0; n < count; n++, pins++) { reg = fdt32_to_cpu(pins->reg); if ((reg < 0) || (reg > pdata->offset)) { - dev_dbg(dev, " invalid register offset 0x%08x\n", reg); + dev_dbg(dev, " invalid register offset 0x%pa\n", ®); continue; } reg += pdata->base; @@ -66,7 +66,7 @@ static int single_configure_pins(struct udevice *dev, pdata->width); continue; } - dev_dbg(dev, " reg/val 0x%08x/0x%08x\n",reg, val); + dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val); } return 0; }