Re: [PATHCv10 1/2] usb: USB Type-C connector class

From: Greg KH
Date: Mon Nov 14 2016 - 09:11:19 EST


On Mon, Nov 14, 2016 at 02:32:35PM +0200, Heikki Krogerus wrote:
> > > +static void __exit typec_exit(void)
> > > +{
> > > + class_unregister(&typec_class);
> >
> > You forgot to clean up your idr :(
>
> Sorry, what idr? The port ids get removed in typec_release().

You have a static idr structure in the driver, right? You have to clean
it up when your code is going away so that it will free any memory it
had allocated with a call to idr_destroy() on module exit.

thanks,

greg k-h