Re: [PATCH] irqchip/tegra: fix fix sparse warnings on __iomem

From: Jason Cooper
Date: Sun Jun 12 2016 - 21:10:13 EST


Hi Ben,

On Tue, Jun 07, 2016 at 01:24:52PM +0100, Ben Dooks wrote:
> Fix the following warnings from sparse due to casting to/from __iomem
> annotated variables:
>
> drivers/irqchip/irq-tegra.c:93:31: warning: incorrect type in initializer (different address spaces)
> drivers/irqchip/irq-tegra.c:93:31: expected void [noderef] <asn:2>*base
> drivers/irqchip/irq-tegra.c:93:31: got void *chip_data
> drivers/irqchip/irq-tegra.c:93:31: warning: incorrect type in initializer (different address spaces)
> drivers/irqchip/irq-tegra.c:93:31: expected void [noderef] <asn:2>*base
> drivers/irqchip/irq-tegra.c:93:31: got void *chip_data
> drivers/irqchip/irq-tegra.c:93:31: warning: incorrect type in initializer (different address spaces)
> drivers/irqchip/irq-tegra.c:93:31: expected void [noderef] <asn:2>*base
> drivers/irqchip/irq-tegra.c:93:31: got void *chip_data
> drivers/irqchip/irq-tegra.c:93:31: warning: incorrect type in initializer (different address spaces)
> drivers/irqchip/irq-tegra.c:93:31: expected void [noderef] <asn:2>*base
> drivers/irqchip/irq-tegra.c:93:31: got void *chip_data
> drivers/irqchip/irq-tegra.c:269:57: warning: incorrect type in argument 5 (different address spaces)
> drivers/irqchip/irq-tegra.c:269:57: expected void *chip_data
> drivers/irqchip/irq-tegra.c:269:57: got void [noderef] <asn:2>*<noident>
>
> Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
> ---
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Jason Cooper <jason@xxxxxxxxxxxxxx>
> Cc: Marc Zyngier <marc.zyngier@xxxxxxx>
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: Stephen Warren <swarren@xxxxxxxxxxxxx>
> Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> Cc: Alexandre Courbot <gnurou@xxxxxxxxx>
> Cc: linux-tegra@xxxxxxxxxxxxxxx
> ---
> drivers/irqchip/irq-tegra.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

Applied to irqchip/misc with Thierry's Ack.

thx,

Jason.