* [BlueZ PATCH V2] btp: Update connect event structure
@ 2020-06-26 0:02 Tedd Ho-Jeong An
2020-06-26 16:58 ` Luiz Augusto von Dentz
0 siblings, 1 reply; 3+ messages in thread
From: Tedd Ho-Jeong An @ 2020-06-26 0:02 UTC (permalink / raw)
To: Linux Bluetooth Mailing List
From: Tedd Ho-Jeong An <tedd.an@intel.com>
This patch updates the connect event struct to align withe the btp spec.
Opcode 0x82 - Device Connected event
Controller Index: <controller id>
Event parameters: Address_Type (1 octet)
Address (6 octets)
Connection Interval (2 octets)
Connection Latency (2 octets)
Supervision Timeout (2 octets)
---
src/shared/btp.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/shared/btp.h b/src/shared/btp.h
index f0ac3a1ee..cc71a71df 100644
--- a/src/shared/btp.h
+++ b/src/shared/btp.h
@@ -259,6 +259,9 @@ struct btp_device_found_ev {
struct btp_gap_device_connected_ev {
uint8_t address_type;
bdaddr_t address;
+ uint16_t connection_interval;
+ uint16_t connection_latency;
+ uint16_t supervision_timeout;
} __packed;
#define BTP_EV_GAP_DEVICE_DISCONNECTED 0x83
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [BlueZ PATCH V2] btp: Update connect event structure
2020-06-26 0:02 [BlueZ PATCH V2] btp: Update connect event structure Tedd Ho-Jeong An
@ 2020-06-26 16:58 ` Luiz Augusto von Dentz
2020-06-26 17:33 ` Tedd Ho-Jeong An
0 siblings, 1 reply; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2020-06-26 16:58 UTC (permalink / raw)
To: Tedd Ho-Jeong An; +Cc: Linux Bluetooth Mailing List
Hi Tedd,
On Thu, Jun 25, 2020 at 5:10 PM Tedd Ho-Jeong An
<tedd.an@linux.intel.com> wrote:
>
> From: Tedd Ho-Jeong An <tedd.an@intel.com>
>
> This patch updates the connect event struct to align withe the btp spec.
>
> Opcode 0x82 - Device Connected event
> Controller Index: <controller id>
> Event parameters: Address_Type (1 octet)
> Address (6 octets)
> Connection Interval (2 octets)
> Connection Latency (2 octets)
> Supervision Timeout (2 octets)
> ---
> src/shared/btp.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/shared/btp.h b/src/shared/btp.h
> index f0ac3a1ee..cc71a71df 100644
> --- a/src/shared/btp.h
> +++ b/src/shared/btp.h
> @@ -259,6 +259,9 @@ struct btp_device_found_ev {
> struct btp_gap_device_connected_ev {
> uint8_t address_type;
> bdaddr_t address;
> + uint16_t connection_interval;
> + uint16_t connection_latency;
> + uint16_t supervision_timeout;
> } __packed;
This appears to be an API break, or we don't for BTP to be stable?
> #define BTP_EV_GAP_DEVICE_DISCONNECTED 0x83
> --
> 2.17.1
>
>
--
Luiz Augusto von Dentz
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [BlueZ PATCH V2] btp: Update connect event structure
2020-06-26 16:58 ` Luiz Augusto von Dentz
@ 2020-06-26 17:33 ` Tedd Ho-Jeong An
0 siblings, 0 replies; 3+ messages in thread
From: Tedd Ho-Jeong An @ 2020-06-26 17:33 UTC (permalink / raw)
To: Luiz Augusto von Dentz; +Cc: Linux Bluetooth Mailing List
Hi Luiz
On Fri, 2020-06-26 at 09:58 -0700, Luiz Augusto von Dentz wrote:
> Hi Tedd,
>
> On Thu, Jun 25, 2020 at 5:10 PM Tedd Ho-Jeong An
> <tedd.an@linux.intel.com> wrote:
> > From: Tedd Ho-Jeong An <tedd.an@intel.com>
> >
> > This patch updates the connect event struct to align withe the btp spec.
> >
> > Opcode 0x82 - Device Connected event
> > Controller Index: <controller id>
> > Event parameters: Address_Type (1 octet)
> > Address (6 octets)
> > Connection Interval (2 octets)
> > Connection Latency (2 octets)
> > Supervision Timeout (2 octets)
> > ---
> > src/shared/btp.h | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/src/shared/btp.h b/src/shared/btp.h
> > index f0ac3a1ee..cc71a71df 100644
> > --- a/src/shared/btp.h
> > +++ b/src/shared/btp.h
> > @@ -259,6 +259,9 @@ struct btp_device_found_ev {
> > struct btp_gap_device_connected_ev {
> > uint8_t address_type;
> > bdaddr_t address;
> > + uint16_t connection_interval;
> > + uint16_t connection_latency;
> > + uint16_t supervision_timeout;
> > } __packed;
>
> This appears to be an API break, or we don't for BTP to be stable?
It was old change and this patch was to align the change for BlueZ.
All other projects using BTP such as Zephyr and Mynewt are already using new
change.
>
> > #define BTP_EV_GAP_DEVICE_DISCONNECTED 0x83
> > --
> > 2.17.1
> >
> >
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-26 17:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-26 0:02 [BlueZ PATCH V2] btp: Update connect event structure Tedd Ho-Jeong An
2020-06-26 16:58 ` Luiz Augusto von Dentz
2020-06-26 17:33 ` Tedd Ho-Jeong An
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).