Hi Satish, On Thu, Jul 18, 2013, Sathish N wrote: > From: sathish N <sathish.n@xxxxxxxxxxxxxxxxxx> Since this seems to be your first submission let me confirm these git author details: is this really how you want your author name to be, i.e. with a lower case "s" and just "N" as your surname? At least in your email "From" header your name was with a capital "S". > --- > bluez/attrib/interactive.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) There seems to be something strange with your tree that generates this extra bluez/* path component. It makes "git am" fail to apply this patch: Applying: attrib: Fix interactive gatttool segfault error: bluez/attrib/interactive.c: does not exist in index Patch failed at 0001 attrib: Fix interactive gatttool segfault Besides the above issues the actual change to the code itself looks fine to me. 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