On Tue, 2009-09-22 at 02:56 +0400, Anton Vorontsov wrote: > On Tue, Sep 22, 2009 at 06:49:40AM +1000, Benjamin Herrenschmidt wrote: > > On Mon, 2009-09-21 at 13:38 -0600, Julie Zhu wrote: > > > > > + iounmap(hcd->regs); > > > +err_ioremap: > > > + irq_dispose_mapping(irq); > > > > You missed that one too :-) > > > > I'm actually considering making irq_dispose_mapping() a private > > API and rename it to something like __irq_dispose_mapping() :-) > > I.e. turning irq_dispose_mapping() into nop, and __-prefixed > (private) variant would actually dispose irq mapping? > > The nop part is important, because otherwise having no cleanup > variant of an allocation/mapping routine feels odd and might raise > questions by non-ppc people (e.g. subsystem maintainers when they > see no cleanup path for such a call). > > And if we'll want to refcount irq mappings in the future, some > drivers will be ready without modifications, i.e. the ones that > got irq_dispose_mapping() wrong: Yup. I agree. I'll look into it Ben. -- 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
