Re: [PATCH] Remove automatic battery state change on attribute example

[Luiz Augusto von Dentz <luiz.dentz@gmail.com>]

Hi,

On Fri, Dec 3, 2010 at 4:56 PM, Claudio Takahasi
<claudio.takahasi@openbossa.org> wrote:
> On Fri, Dec 3, 2010 at 11:50 AM, Claudio Takahasi
> <claudio.takahasi@openbossa.org> wrote:
>> Hi Luiz,
>>
>> On Fri, Dec 3, 2010 at 11:37 AM, Luiz Augusto von Dentz
>> <luiz.dentz@gmail.com> wrote:
>>> Hi,
>>>
>>> On Fri, Dec 3, 2010 at 4:24 PM, Claudio Takahasi
>>> <claudio.takahasi@openbossa.org> wrote:
>>>> Legacy code implemented to test Indication and Notification
>>>> ---
>>>>  attrib/example.c |   19 -------------------
>>>>  1 files changed, 0 insertions(+), 19 deletions(-)
>>>>
>>>> diff --git a/attrib/example.c b/attrib/example.c
>>>> index c29e1e4..2d5b758 100644
>>>> --- a/attrib/example.c
>>>> +++ b/attrib/example.c
>>>> @@ -59,22 +59,6 @@
>>>>  #define FMT_KILOGRAM_UUID              0xA010
>>>>  #define FMT_HANGING_UUID               0xA011
>>>>
>>>> -static guint timeout_id = 0;
>>>> -
>>>> -static gboolean change_battery_state(gpointer user_data)
>>>> -{
>>>> -       static uint8_t state =  0x05;
>>>> -       uuid_t uuid;
>>>> -       uint8_t atval[1];
>>>> -
>>>> -       /* Battery state is being increased every 10 seconds. */
>>>> -       atval[0] = state++;
>>>> -       sdp_uuid16_create(&uuid, BATTERY_STATE_UUID);
>>>> -       attrib_db_update(0x0110, &uuid, atval, 1);
>>>> -
>>>> -       return TRUE;
>>>> -}
>>>> -
>>>>  static int register_attributes(void)
>>>>  {
>>>>        const char *devname = "Example Device";
>>>> @@ -170,8 +154,6 @@ static int register_attributes(void)
>>>>        atval[1] = 0x00;
>>>>        attrib_db_add(0x0111, &uuid, atval, 2);
>>>>
>>>> -       timeout_id = g_timeout_add_seconds(10, change_battery_state, NULL);
>>>> -
>>>>        /* Thermometer: primary service definition */
>>>>        sdp_uuid16_create(&uuid, GATT_PRIM_SVC_UUID);
>>>>        u16 = htons(THERM_HUMIDITY_SVC_UUID);
>>>> @@ -436,5 +418,4 @@ int server_example_init(void)
>>>>
>>>>  void server_example_exit(void)
>>>>  {
>>>> -       g_source_remove(timeout_id);
>>>>  }
>>>> --
>>>> 1.7.3.2
>>>>
>>>> --
>>>> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
>>>> the body of a message to majordomo@vger.kernel.org
>>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>>
>>>
>>> I wonder what is the relation of this code with attrib-server.c, does
>>> this work if we don't have the attrib server enabled?
>>>
>>> --
>>> Luiz Augusto von Dentz
>>> Computer Engineer
>>>
>>
>> Currently, we don't have compile option to disable the gatt server(in
>> the core), only configuration option: "EnableAttrib" in the main.conf.
>> The gatt server example "belongs" to the attribute plugin,
>> --disable-attrib disables the attrib client and the attribute server
>> example.
>>
>> Claudio.
>>
>
> The right main.conf option is:
> AttributeServer = false

Yep, but this still enables the server example code, even with
attribute server not enabled, which IMO sounds weird, anyway this
example code should probably be removed at some right? So why not do
it now?


-- 
Luiz Augusto von Dentz
Computer Engineer