Hi Sheldon, On Wed, Feb 23, 2011, Sheldon Demario wrote: > + if (conn_state == STATE_CONNECTED) { I think it'd be cleaner to have: if (conn_state != STATE_CONNECTED) return; > + > + return; > +} Never have empty return statements at the end of void-functions. 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