Hi Marcel, On Monday 24 February 2014 11:34:26 Marcel Holtmann wrote: > Hi Szymon, > > >>> Added PICS and PIXIT target Android 4.4 > >>> --- > >>> android/Makefile.am | 2 + > >>> android/pics-hsp.txt | 103 > >>> ++++++++++++++++++++++++++++++++++++++++++++++++++ android/pixit-hsp.txt > >>> > >>> | 30 +++++++++++++++ > >>> > >>> 3 files changed, 135 insertions(+) > >>> create mode 100644 android/pics-hsp.txt > >>> create mode 100644 android/pixit-hsp.txt > >> > >> so we are doing Headset profile (HSP) and also Handsfree profile (HFP). I > >> am fine with also doing HSP since it seems that is what Android uses on > >> tablets. > > > > Both HSP and HFP are handled by handsfree HAL sp we would probably need > > some extra argument in IPC to allow disabling HFP if not required. And > > some configuration (runtime/compile time?) flag for libhal. > > I do not want a compile time option. Is that how Bluedroid handles it. For > examples Nexus 4 vs Nexus 7. Yes, Bluedroid is compile-time configured, device that wants to use non-default options provides those in bdroid_buildcfg.h header eg. this is defined for Nexus7 (flo), but not for Nexus4 (mako) #define BTIF_HF_SERVICES (BTA_HSP_SERVICE_MASK) > > The easiest would be to have some IPC option to set HSP and HFP. Meaning > that we could have a property that the HAL sets and then just uses that to > tell the daemon what to enabled. > > I am thinking we might just need a flags parameter to the Register module > command in the core service. Or alternatively, have a Set Option command. > > Right now I would tend to do the Set Option command. So the HAL can > configure certain options. I agree, for now we can provide such option command, if it turns out that more profiles HALs need configuration we can move to flags in register commands. -- Szymon K. Janc szymon.janc@xxxxxxxxx -- 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