On Mon, Apr 11, 2011 at 11:19:13PM +0200, RafaÅ MiÅecki wrote:
> 2011/4/11 Greg KH <greg@xxxxxxxxx>:
> > 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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
