[PATCH] serial: imx: Fix build warning when CONFIG_LPAE=y

[PATCH] serial: imx: Fix build warning when CONFIG_LPAE=y

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

Building multi_v7_defconfig with CONFIG_LPAE=y leads to the following build 
warning:

drivers/tty/serial/imx.c: In function 'imx_verify_port':
drivers/tty/serial/imx.c:1542:6: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

Fix the casting to get rid of the warning.

Reported-by: Kevin Hilman <khilman@linaro.org>
Reported-by: Olof Johansson <olof@lixom.net>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 drivers/tty/serial/imx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c
index a0ebbc9..dd5e583 100644
--- a/drivers/tty/serial/imx.c
+++ b/drivers/tty/serial/imx.c
@@ -1539,7 +1539,7 @@ imx_verify_port(struct uart_port *port, struct serial_struct *ser)
 		ret = -EINVAL;
 	if (sport->port.uartclk / 16 != ser->baud_base)
 		ret = -EINVAL;
-	if ((void *)sport->port.mapbase != ser->iomem_base)
+	if (sport->port.mapbase != (unsigned long)ser->iomem_base)
 		ret = -EINVAL;
 	if (sport->port.iobase != ser->port)
 		ret = -EINVAL;
-- 
1.8.1.2

Re: [PATCH] serial: imx: Fix build warning when CONFIG_LPAE=y

[Olof Johansson]

On Fri, Sep 20, 2013 at 4:47 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Building multi_v7_defconfig with CONFIG_LPAE=y leads to the following build
> warning:
>
> drivers/tty/serial/imx.c: In function 'imx_verify_port':
> drivers/tty/serial/imx.c:1542:6: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
>
> Fix the casting to get rid of the warning.
>
> Reported-by: Kevin Hilman <khilman@linaro.org>
> Reported-by: Olof Johansson <olof@lixom.net>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

Already fixed by:

http://marc.info/?l=linux-kernel&m=137896012701204&w=2

(But sitting in Greg's queue due to the merge window).


-Olof