Hi Andrei, > Patches looks OK (taking that typedef is a standard way in bluez). > > The only note: I think that we need to differentiate AMP and A2MP. When we > are dealing with A2MP protocol we could name structures / functions as > a2mp_* I tried to follow the convention of the spec, which uses A2MP only when referring to the protocol as a whole and when referring to the feature mask and the fixed channel. It uses AMP or 'AMP Protocol' in all other places, including individual protocol elements. Consider this, and if you still think it should be a2mp_ I'll do it. Peter. --Peter Krystad Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum -- 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
