Hi Radek, On Fri, Feb 11, 2011, Radoslaw Jablonski wrote: > + ret = phonebook_pull_read(irmc->request); > + > + if (err) { > + *err = ret; > + } No empty line before the if-statement and no {} for one-line sections. > + ret = phonebook_pull_read(irmc->request); > + > + if (ret < 0) { No empty line here, like above. > + ret = phonebook_pull_read(request); > + > + if (ret < 0) Same here. > + len = string_read(obj->buffer, buf, count); > + > + if (len == 0 && !obj->lastpart) { And here. 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