Hi Ulisses, On Fri, Apr 20, 2012 at 11:51:53AM -0300, Ulisses Furquim wrote: > > +/* AMP Manager functions */ > > +void amp_mgr_get(struct amp_mgr *mgr) > > I'm probably late to say this but maybe amp_mgr_hold() instead of > _get() like we already have in the stack would be better? I actually like more get since this indicates that kref is used with kref_get as shown below. Best regards Andrei Emeltchenko > > Regards, > > -- Ulisses > > > +{ > > + BT_DBG("mgr %p", mgr); > > + > > + kref_get(&mgr->kref); > > +} > > + -- 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