Hi Michal, On Mon, Dec 20, 2010, Michal Labedzki wrote: > --- a/lib/hci.c > +++ b/lib/hci.c > @@ -56,8 +56,8 @@ typedef struct { > > static int is_number(const char *c) > { > - while(*c) { > - if (! isdigit(*c)) > + while (*c) { > + if (!isdigit(*c)) > return 0; > ++c; Could you please merge this part to your first patch that introduced the issue. Besides that the cleanups seem fine to me (after a very quick look). 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