Hi Andrei, On Thu, Nov 03, 2011, Andrei Emeltchenko wrote: > > @@ -1956,6 +1964,10 @@ int mgmt_index_added(u16 index) > > > > int mgmt_index_removed(u16 index) > > { > > + uint8_t status = ENODEV; > > Does it make sense to mix type naming here and below? (u8, u16, uint8_t, uint16_t) No, it doesn't make sense. It should be u8, u16, etc. This is just my strong user-space background shining through. Will send a v2 in a minute. 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