On Mon, Apr 11, 2011 at 11:19:13PM +0200, Rafał Miłecki wrote: > 2011/4/11 Greg KH <[email protected]>: > > On Mon, Apr 11, 2011 at 11:25:14PM +0200, Rafał Miłecki wrote: > >> +static void axi_release_core_dev(struct device *dev) > >> +{ > >> + /* Silent lack-of-release warning */ > >> +} > > > > Ok, WTF!!!! > > Thank you for your kindly words. It's much more enjoyable to work on > kernel that way. I definitely made that on purpose.
I can't tell if you are kidding or not. Please read the documentation for how to do this properly. I find it really hard to believe that you wrote that comment instead of putting in the 2 lines of code required for this function. Especially as-it-is, your code does not work properly and leaks memory badly. Why would you do that on purpose? thanks, greg k-h _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
