Hi Bastien, On 5 September 2017 at 11:03, Bastien Nocera <hadess@xxxxxxxxxx> wrote: > On Mon, 2017-09-04 at 20:12 +0200, Bastien Nocera wrote: >> <snip> >> + >> +inline CablePairingType >> +get_pairing_type(uint16_t vid, >> + uint16_t pid, >> + char **name, >> + uint16_t *source, >> + uint16_t *version) > > I have a slightly updated version of that section. "static" is missing > in front of the inline function for use with RPM build's strict linker > settings, and I'm sure the indentation police would like to see that > function return type and name on the same line ;) > -- > 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 I'm not sure why this is in header in first place ? -- pozdrawiam Szymon K. Janc -- 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