Re: [PATCH 3/4] Bearer documentation

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1460 bytes --]

Hi Rémi,

On 01/10/2011 06:41 AM, Rémi Denis-Courmont wrote:
> On Friday 07 January 2011 18:23:25 ext Denis Kenzior, you wrote:
>> Hi Rémi,
>>
>> On 01/07/2011 10:02 AM, Rémi Denis-Courmont wrote:
>>> ---
>>>
>>>  doc/connman-api.txt |   10 ++++++++++
>>>  1 files changed, 10 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/doc/connman-api.txt b/doc/connman-api.txt
>>> index b13efd1..18185d7 100644
>>> --- a/doc/connman-api.txt
>>> +++ b/doc/connman-api.txt
>>> @@ -150,6 +150,16 @@ Properties	boolean Active [readwrite]
>>>
>>>  			Holds whether the context is activated.  This value
>>>  			can be set to activate / deactivate the context.
>>>
>>> +		string Bearer [readonly, optional]
>>> +
>>> +			Contains the current packet switched bearer of the
>>> +			connection context.
>>> +
>>> +			Possible values are the same as for the Technology
>>> +			property the NetworkRegistration object, plus
>>> +			"none" if no data activity is ongoing, or "unknown"
>>> +			if not known.
>>> +
>>
>> Please remove this "unknown" bit.  The property is already optional, so
>> if it is never reported by the driver we shouldn't report it up, even if
>> the context is active.
> 
> That would not work. We need some default value otherwise we cannot signal the 
> change from known to unknown, at least not with the oFono property change 
> interface.
> 

When do you think that will ever happen?

Regards,
-Denis