<div dir="ltr">Hi,<div><br></div><div>I had submitted the patch version 2 . please help to review the same</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 16, 2018 at 12:58 AM Georg Chini <<a href="mailto:georg@chini.tk">georg@chini.tk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 15.08.2018 09:24, Pali Rohár wrote:<br>
> On Wednesday 15 August 2018 09:24:04 Sathish Narasimman wrote:<br>
>> Hi Pali,<br>
>><br>
>> On Wed, Aug 15, 2018 at 1:31 AM, Pali Rohár <<a href="mailto:pali.rohar@gmail.com" target="_blank">pali.rohar@gmail.com</a>> wrote:<br>
>><br>
>>> On Tuesday 14 August 2018 22:49:27 Luiz Augusto von Dentz wrote:<br>
>>>> Hi Pali,<br>
>>>><br>
>>>> On Tue, Aug 14, 2018 at 10:28 PM Pali Rohár <<a href="mailto:pali.rohar@gmail.com" target="_blank">pali.rohar@gmail.com</a>><br>
>>> wrote:<br>
>>>>> On Tuesday 14 August 2018 19:44:49 Sathish Narasimman wrote:<br>
>>>>>> mSBC-encoded streams for HFP. The wideband speec encoding and<br>
>>> decoding<br>
>>>>>> is implemeted with this patch. This patch was refered from original<br>
>>>>>> patch of Joao Paula Rechi Vita and was verified with the supported<br>
>>>>>> bluetooth controller.<br>
>>>>>><br>
>>>>>> Signed-off-by: Sathish Narasimman <<a href="mailto:sathish.narasimman@intel.com" target="_blank">sathish.narasimman@intel.com</a>><br>
>>>>>> ---<br>
>>>>>>   src/modules/bluetooth/backend-ofono.c        |  20 +-<br>
>>>>>>   src/modules/bluetooth/module-bluez5-device.c | 371<br>
>>> ++++++++++++++++++++++++++-<br>
>>>>>>   2 files changed, 377 insertions(+), 14 deletions(-)<br>
>>>>> Hi! This looks great. What about support also for other backends, not<br>
>>>>> only ofono?<br>
>>>> Afaik only oFono supports WBS negotiation.<br>
>>> IIRC codec negotiation is done via AT commands. Therefore native backend<br>
>>> and its AT parser could be extended to support it.<br>
>>><br>
>> AFAIK i was not able to find Handsfree profile in native backend. I see<br>
>> only the headset profile in which codec negotiation is not available.<br>
>> please point me the right place where AT commands are implented in PA for<br>
>> HFP profile in native backend. i can try the best.<br>
> rfcomm_io_callback function in backend-native.c. There is implemented AT<br>
> parser. For codec selection is needed to issue some AT command,<br>
> therefore that code is needed to extend...<br>
><br>
Currently there is no HFP support in the native backend. There have<br>
been patches by James Bottomley to implement HFP, but they have<br>
never been merged due to remaining issues. See here:<br>
<a href="https://patchwork.freedesktop.org/series/30716/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/series/30716/</a><br>
<br>
</blockquote></div>