Hi Vishal, On Thu, Apr 26, 2012, Johan Hedberg wrote: > > +static inline size_t eir_get_padding_offset(u8 *eir, size_t eir_len) > > I'm not completely sure about this name. Would eir_significant_len() > sound better? The core spec uses the terminology "significant part" and > non-significant part" to refer to this so I thought reusing that might > make it clearer what we're dealing with. I changed my mind: let's just call it eir_get_length(). And to make it more clear call the input parameters data and data_len. 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