Hi Luiz, On Thu, Feb 16, 2012 at 10:49 AM, Luiz Augusto von Dentz <luiz.dentz at gmail.com> wrote: > Hi Fr?d?ric, > > 2012/2/6 Fr?d?ric Dalleau <frederic.dalleau at linux.intel.com>: >> From: Pierre-Louis Bossart <pierre-louis.bossart at intel.com> >> >> Second version of patch to enable MP3 passthrough over A2DP, with >> the decoding taking place on the headset to reduce power consumption. >> This was only possible if PulseAudio was bypassed before with the >> a2dpsink element in gstreamer. > > I would not mess that much with device module, we clearly need to > separate the codec code from it so perhaps separating in endpoint > modules such as module-bluetooth-sbc and module-bluetooth-mpeg would > have been a better idea, that way is much easier to extend with other > codec support latter. Also lets avoid the use unix socket since it is > considered deprecated. The original series uses Unix sockets, i'm moving it to Media API. I agree some refactoring is needed and separating through endpoint modules is one way to achieve this. However, how would you handle the fact that several sinks and sources are connected to same remote device at same time, this could be confusing since only one could be "acquired". Regards, Fr?d?ric
