Yes, it is not I/O memory address. I was wrong on that. But it makes sparse happy. Is here any side effect of using ioread16() ? regards, ronit On Tue, Aug 18, 2015 at 2:27 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > On Tue, Aug 18, 2015 at 12:16:57PM +0530, Ronit Halder wrote: >> This patch fixes the warning generated by sparse >> "cast removes address space of expression" by using ioread16 >> function insted of directly dereferencing I/O memory. >> >> Signed-off-by: Ronit halder <ronit.crj@xxxxxxxxx> > > Are these really iomem pointers? A lot of drivers use a fake > screen_base which is just a buffer of normal memory. > > regards, > dan carpenter > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel