* [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later
@ 2012-11-09 8:31 Syam Sidhardhan
2012-11-09 8:31 ` [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM Syam Sidhardhan
2012-11-09 10:44 ` [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Marcel Holtmann
0 siblings, 2 replies; 6+ messages in thread
From: Syam Sidhardhan @ 2012-11-09 8:31 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Syam Sidhardhan
The gdbus require D-Bus 1.5 version.
Log:
CC gdbus/object.o
gdbus/object.c: In function ‘properties_set’:
gdbus/object.c:876:7: error: ‘DBUS_ERROR_UNKNOWN_PROPERTY’ undeclared
(first use in this function)
gdbus/object.c:876:7: note: each undeclared identifier is reported
only once for each function it appears in
gdbus/object.c:881:6: error: ‘DBUS_ERROR_PROPERTY_READ_ONLY’
undeclared (first use in this function)
make[1]: *** [gdbus/object.o] Error 1
make: *** [all] Error 2
---
acinclude.m4 | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/acinclude.m4 b/acinclude.m4
index ddc8183..1ae402c 100644
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -92,8 +92,8 @@ AC_DEFUN([AC_INIT_BLUEZ], [
])
AC_DEFUN([AC_PATH_DBUS], [
- PKG_CHECK_MODULES(DBUS, dbus-1 >= 1.4, dummy=yes,
- AC_MSG_ERROR(D-Bus >= 1.4 is required))
+ PKG_CHECK_MODULES(DBUS, dbus-1 >= 1.5, dummy=yes,
+ AC_MSG_ERROR(D-Bus >= 1.5 is required))
AC_SUBST(DBUS_CFLAGS)
AC_SUBST(DBUS_LIBS)
])
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM
2012-11-09 8:31 [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Syam Sidhardhan
@ 2012-11-09 8:31 ` Syam Sidhardhan
2012-11-16 8:05 ` Johan Hedberg
2012-11-09 10:44 ` [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Marcel Holtmann
1 sibling, 1 reply; 6+ messages in thread
From: Syam Sidhardhan @ 2012-11-09 8:31 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Syam Sidhardhan
This patch enable us to test the auto select PSM by passing
PSM value as 0.
Ex: l2test -d -P 0
l2test[2585]: Waiting for connection on psm 4099 ...
---
test/l2test.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/test/l2test.c b/test/l2test.c
index 7645681..72ad4ba 100644
--- a/test/l2test.c
+++ b/test/l2test.c
@@ -87,7 +87,7 @@ static long buffer_size = 2048;
/* Default addr and psm and cid */
static bdaddr_t bdaddr;
-static unsigned short psm = 0x1011;
+static unsigned short psm = 0;
static unsigned short cid = 0;
/* Default number of frames to send (-1 = infinite) */
@@ -375,8 +375,6 @@ static int do_connect(char *svr)
addr.l2_cid = htobs(cid);
else if (psm)
addr.l2_psm = htobs(psm);
- else
- goto error;
if (connect(sk, (struct sockaddr *) &addr, sizeof(addr)) < 0 ) {
syslog(LOG_ERR, "Can't connect: %s (%d)",
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later
2012-11-09 8:31 [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Syam Sidhardhan
2012-11-09 8:31 ` [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM Syam Sidhardhan
@ 2012-11-09 10:44 ` Marcel Holtmann
[not found] ` <CAFBvHif+xS12ywGEUdpycTtU--TVZSTgAb4mJuN_O5NKJSz36w@mail.gmail.com>
1 sibling, 1 reply; 6+ messages in thread
From: Marcel Holtmann @ 2012-11-09 10:44 UTC (permalink / raw)
To: Syam Sidhardhan; +Cc: linux-bluetooth
Hi Syam,
> The gdbus require D-Bus 1.5 version.
>
> Log:
> CC gdbus/object.o
> gdbus/object.c: In function ‘properties_set’:
> gdbus/object.c:876:7: error: ‘DBUS_ERROR_UNKNOWN_PROPERTY’ undeclared
> (first use in this function)
> gdbus/object.c:876:7: note: each undeclared identifier is reported
> only once for each function it appears in
> gdbus/object.c:881:6: error: ‘DBUS_ERROR_PROPERTY_READ_ONLY’
> undeclared (first use in this function)
> make[1]: *** [gdbus/object.o] Error 1
> make: *** [all] Error 2
if they are still simple defines, then lets just #ifndef them and define
them in gdbus/object.c
Regards
Marcel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later
[not found] ` <CAFBvHif+xS12ywGEUdpycTtU--TVZSTgAb4mJuN_O5NKJSz36w@mail.gmail.com>
@ 2012-11-14 13:03 ` Syam Sidhardhan
0 siblings, 0 replies; 6+ messages in thread
From: Syam Sidhardhan @ 2012-11-14 13:03 UTC (permalink / raw)
To: Marcel Holtmann; +Cc: linux-bluetooth
Hi,
On Tue, Nov 13, 2012 at 10:01 PM, Syam Sidhardhan
<syamsidhardh@gmail.com> wrote:
> Hi Marcel,
>
> On Fri, Nov 9, 2012 at 4:14 PM, Marcel Holtmann <marcel@holtmann.org> wro=
te:
>> Hi Syam,
>>
>>> The gdbus require D-Bus 1.5 version.
>>>
>>> Log:
>>> CC gdbus/object.o
>>> gdbus/object.c: In function =E2=80=98properties_set=E2=80=99:
>>> gdbus/object.c:876:7: error: =E2=80=98DBUS_ERROR_UNKNOWN_PROPERTY=E2=80=
=99 undeclared
>>> (first use in this function)
>>> gdbus/object.c:876:7: note: each undeclared identifier is reported
>>> only once for each function it appears in
>>> gdbus/object.c:881:6: error: =E2=80=98DBUS_ERROR_PROPERTY_READ_ONLY=E2=
=80=99
>>> undeclared (first use in this function)
>>> make[1]: *** [gdbus/object.o] Error 1
>>> make: *** [all] Error 2
>>
>> if they are still simple defines, then lets just #ifndef them and define
>> them in gdbus/object.c
>>
>
> Yes, those are simple defines. I'll send a patch according to your sugges=
tion.
>
Missing the mailing list in my earlier reply. CC-ing here.
Regards,
Syam.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM
2012-11-09 8:31 ` [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM Syam Sidhardhan
@ 2012-11-16 8:05 ` Johan Hedberg
2012-11-19 14:09 ` Syam Sidhardhan
0 siblings, 1 reply; 6+ messages in thread
From: Johan Hedberg @ 2012-11-16 8:05 UTC (permalink / raw)
To: Syam Sidhardhan; +Cc: linux-bluetooth
Hi Syam,
On Fri, Nov 09, 2012, Syam Sidhardhan wrote:
> This patch enable us to test the auto select PSM by passing
> PSM value as 0.
>
> Ex: l2test -d -P 0
> l2test[2585]: Waiting for connection on psm 4099 ...
> ---
> test/l2test.c | 4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
>
> diff --git a/test/l2test.c b/test/l2test.c
> index 7645681..72ad4ba 100644
> --- a/test/l2test.c
> +++ b/test/l2test.c
> @@ -87,7 +87,7 @@ static long buffer_size = 2048;
>
> /* Default addr and psm and cid */
> static bdaddr_t bdaddr;
> -static unsigned short psm = 0x1011;
> +static unsigned short psm = 0;
> static unsigned short cid = 0;
>
> /* Default number of frames to send (-1 = infinite) */
> @@ -375,8 +375,6 @@ static int do_connect(char *svr)
> addr.l2_cid = htobs(cid);
> else if (psm)
> addr.l2_psm = htobs(psm);
> - else
> - goto error;
>
> if (connect(sk, (struct sockaddr *) &addr, sizeof(addr)) < 0 ) {
> syslog(LOG_ERR, "Can't connect: %s (%d)",
At least the second chunk is for the initiating (client) part, not the
server (which the commit message implies you're dealing with). There's
no "auto select" for the client.
Johan
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM
2012-11-16 8:05 ` Johan Hedberg
@ 2012-11-19 14:09 ` Syam Sidhardhan
0 siblings, 0 replies; 6+ messages in thread
From: Syam Sidhardhan @ 2012-11-19 14:09 UTC (permalink / raw)
To: Syam Sidhardhan, linux-bluetooth
Hi Johan,
On Fri, Nov 16, 2012 at 1:35 PM, Johan Hedberg <johan.hedberg@gmail.com> wrote:
> Hi Syam,
>
> On Fri, Nov 09, 2012, Syam Sidhardhan wrote:
>> This patch enable us to test the auto select PSM by passing
>> PSM value as 0.
>>
>> Ex: l2test -d -P 0
>> l2test[2585]: Waiting for connection on psm 4099 ...
>> ---
>> test/l2test.c | 4 +---
>> 1 files changed, 1 insertions(+), 3 deletions(-)
>>
>> diff --git a/test/l2test.c b/test/l2test.c
>> index 7645681..72ad4ba 100644
>> --- a/test/l2test.c
>> +++ b/test/l2test.c
>> @@ -87,7 +87,7 @@ static long buffer_size = 2048;
>>
>> /* Default addr and psm and cid */
>> static bdaddr_t bdaddr;
>> -static unsigned short psm = 0x1011;
>> +static unsigned short psm = 0;
>> static unsigned short cid = 0;
>>
>> /* Default number of frames to send (-1 = infinite) */
>> @@ -375,8 +375,6 @@ static int do_connect(char *svr)
>> addr.l2_cid = htobs(cid);
>> else if (psm)
>> addr.l2_psm = htobs(psm);
>> - else
>> - goto error;
>>
>> if (connect(sk, (struct sockaddr *) &addr, sizeof(addr)) < 0 ) {
>> syslog(LOG_ERR, "Can't connect: %s (%d)",
>
> At least the second chunk is for the initiating (client) part, not the
> server (which the commit message implies you're dealing with). There's
> no "auto select" for the client.
>
True. While preparing the patch it got misplaced.
I'll send an updated version.
Thanks,
Syam.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-11-19 14:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-09 8:31 [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Syam Sidhardhan
2012-11-09 8:31 ` [PATCH BlueZ 2/2] l2test: Add support to test auto select PSM Syam Sidhardhan
2012-11-16 8:05 ` Johan Hedberg
2012-11-19 14:09 ` Syam Sidhardhan
2012-11-09 10:44 ` [PATCH BlueZ 1/2] build: Require D-Bus 1.5 or later Marcel Holtmann
[not found] ` <CAFBvHif+xS12ywGEUdpycTtU--TVZSTgAb4mJuN_O5NKJSz36w@mail.gmail.com>
2012-11-14 13:03 ` Syam Sidhardhan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).