Hi Andrei, On Thu, May 10, 2012, Andrei Emeltchenko wrote: > + printf("Country code: %c%c%c\n", chan_list->country_code[0], > + chan_list->country_code[1], chan_list->country_code[2]); We still use tabs-only indentation in user space. The above line is violating this. > + printf("Reg ext id %d reg class %d coverage class %d\n", > + triplet->ext.reg_extension_id, > + triplet->ext.reg_class, > + triplet->ext.coverage_class); Same here. > + if (triplet->chans.num_channels == 1) > + printf("MAC: %2.2X:%2.2X:%2.2X:%2.2X:%2.2X:%2.2X\n", > + tlv->val[0], tlv->val[1], tlv->val[2], > + tlv->val[3], tlv->val[4], tlv->val[5]); And here. Btw, it also looks like the printf is incorrectly indented (same indentation as the if). > + printf("PAL CAP: %2.2x %2.2x %2.2x %2.2x\n", > + tlv->val[0], tlv->val[1], tlv->val[2], > + tlv->val[3]); Tabs + spaces here again. > + printf("PAL VER: %2.2x Comp ID: %4.4x SubVer: %4.4x\n", > + ver->ver, btohs(ver->company_id), > + btohs(ver->sub_ver)); 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