Hi Denis:

On Mon, Apr 22, 2013 at 6:04 AM, Denis Kenzior <denk...@gmail.com> wrote:
> Hi Claudio,
>
>
> On 04/22/2013 07:58 AM, Claudio Takahasi wrote:
>>
>> This patch adds echo canceling and noise reduction property to Handsfree
>> interface, allowing the handsfree unit to enable or disable this feature
>> in the audio gateway.
>>
>> According to Bluetooth HFP 1.6 spec: By default, if the AG supports its
>> own embedded echo canceling and/or noise reduction functions, it shall
>> have them activated until the AT+NREC command is received.
>> ---
>>   doc/handsfree-api.txt | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/doc/handsfree-api.txt b/doc/handsfree-api.txt
>> index 0b5d421..6855d93 100644
>> --- a/doc/handsfree-api.txt
>> +++ b/doc/handsfree-api.txt
>> @@ -56,6 +56,13 @@ Properties   array{string} Features [readonly]
>>                         to activate or deactivate the function from the
>> HF, or
>>                         the AG could autonomously initiate it.
>>
>> +               boolean EchoCancelingNoiseReduction [readwrite]
>> +
>> +                       Boolean representing whether echo canceling and
>> noise
>> +                       reduction is enabled in the AG. This property may
>> be
>> +                       written to activate or deactivate this feature
>> from
>> +                       the HF unit.
>> +
>
>
> This feature can only be disabled, not enabled for a given SLC.  So setting
> this to 'True' should return an error or do something way fancier.
>
> I suspect that to implement this properly we need to store the ECNR setting
> on disk (e.g. /var/lib/ofono/<local bdaddr>/<remote bdaddr>/handsfree or
> something) and manage it appropriately inside the core.

It is true, the HFP spec defines the procedure to disable it only.
I implemented an option to disable it because I noticed different
behavior between the phones/modems. Anyway, I will remove it, we can
extend the API later if necessary.

Regarding the storage, the spec (page 53) is a little bit confusing to
me, I understood that the AG will enable it again when the connection
is dropped. So, we don't need to stored it.

Regards,
Claudio
_______________________________________________
ofono mailing list
ofono@ofono.org
https://lists.ofono.org/mailman/listinfo/ofono

Reply via email to