On Wed, Jul 27, 2011 at 10:03:15AM +0200, Sascha Hauer wrote: > On Wed, Jul 27, 2011 at 01:29:33PM +0800, Tony Lin wrote: > > + if (!ppriv->phy_regs || !ppriv->ctrl_regs) { > > + iounmap(ppriv->phy_regs); > > + iounmap(ppriv->ctrl_regs); > > + return -ENOMEM; > > + } > > So you call iounmap(0) here. That is not a problem. iounmap() is like kfree(). It's designed to cope with NULL. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html