Re: [RFC][PATCH V3] axi: add AXI bus driver

[Greg KH]

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/