Hi! > +static void serdev_ctrl_release(struct device *dev) > +{ > + struct serdev_controller *ctrl = to_serdev_controller(dev); > + ida_simple_remove(&ctrl_ida, ctrl->nr); > + kfree(ctrl); > +} Would it make sense to do something like to_serdev_controller(dev) = NULL; there? > +MODULE_AUTHOR("Rob Herring <robh@xxxxxxxxxx"); Missing >. Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Attachment:
signature.asc
Description: Digital signature