On Mon, May 21, 2012 at 06:10:07PM -0700, H Hartley Sweeten wrote: > @@ -210,7 +210,7 @@ static int pci20xxx_attach(struct comedi_device *dev, > if (ret < 0) > return ret; > > - devpriv->ioaddr = (void *)(unsigned long)it->options[0]; > + devpriv->ioaddr = (void __iomem *)(unsigned long)it->options[0]; Obviously, your patch is fine, but I was wondering if it was a problem that it->options was only a 32 bit? Seems like it would deserve a comment. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel