Hi Jose, On Mon, Nov 22, 2010, Jose Antonio Santos Cadenas wrote: > Also a destroy function is added in order to avoid memory leaks > --- > health/mcap.c | 28 +++++++++++++++++++--------- > 1 files changed, 19 insertions(+), 9 deletions(-) The patch content itself is fine but you need to fix the commit message. Now you have half of the info in the summary and half in the message body (both halves being equaly relevant). The summary line should be a summary of the whole patch (in this case it could be e.g. "Fix MDL reference counting for IO watchers") and the body should contain the details. So please reformulate, resend, and pay more attention to this kind of things in the future. Thanks :) Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html