Hi Gowtham, On Fri, Nov 07, 2014, Gowtham Anandha Babu wrote: > Changes made to decode RFCOMM hdr and print the same. > > RFCOMM: Unnumbered Info with Header Check (UIH)(0xef) Could you add a space here between (UIH) and (0xef) > Address : (0x01) No space before ':' and no need to have parenthesis around the hex value. The reason why that exists elsewhere is to make it clear that the value is separate of the preceding string translation (which you don't have here). > Command/Response Bit: 0 > DLCI : (0x00) > Control : (0xef) > Poll/Final Bit : 0 > Length : 10 > FCS : (0xaa) > 81 11 20 e0 27 00 9a 02 00 07 aa .. .'...... I'd prefer if we could try to stick to the hcidump-style decoding and make this more compact. I.e. you could have something like: Address: 0x01 cr 0 dlci 0 Control: 0xef poll/final 0 Length: 10 ... 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
