On Monday, December 09, 2013 8:56 PM, Mark Brown wrote: > On Mon, Dec 09, 2013 at 12:21:11PM +0100, Michal Nazarewicz wrote: > > On Mon, Dec 09 2013, Jingoo Han <[email protected]> wrote: > > > > - spi100k->base = (void __iomem *)dev_get_platdata(&pdev->dev); > > > + spi100k->base = dev_get_platdata(&pdev->dev); > > > I might be wrong, but I would assume that __iomem is the important bit > > here and removing this cast will make linter unhappy. > > Yes, I'd expect so. Jingoo have you tested with sparse?
Oops, I didn't test sparse warning. This patch makes sparse warning as below: drivers/spi/spi-omap-100k.c:444:23: warning: incorrect type in assignment (different address spaces) drivers/spi/spi-omap-100k.c:444:23: expected void [noderef] <asn:2>*base drivers/spi/spi-omap-100k.c:444:23: got void * Please, ignore this patch. Sorry for annoying you. :-) Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
