> From: Luiz Augusto von Dentz [mailto:luiz.dentz@xxxxxxxxx] > Sent: Friday, August 31, 2018 10:04 AM > To: Karabin, Josh <George.Karabin@xxxxxxxxxxxxx> > Cc: linux-bluetooth@xxxxxxxxxxxxxxx > Subject: Re: [External] [PATCH BlueZ v2 0/2] Change ConnectDevice to > connect a subset of services > > Hi Josh, > > On Fri, Aug 31, 2018 at 4:58 PM, Luiz Augusto von Dentz > <luiz.dentz@xxxxxxxxx> wrote: > > Hi Josh, > > > > On Fri, Aug 31, 2018 at 4:47 PM, Karabin, Josh > > <George.Karabin@xxxxxxxxxxxxx> wrote: > >> Hello all - I would like to point your attention to this > >> patch set again. I realize it's more complicated than v1, and likely > >> needs more time for review. > > > > Could you please add description to each patch, also is your email correct? Will do. My email Is correct - Josh is my middle name and the name that I commonly use, but our corporate address assignment defaults to my first name. > I forgot to mention that the documentation should come first and > perhaps it makes send that we use ConnectDevice in bluetoothctl, may > add an optional parameter for UUIDs list which in case it is set the > use Adapter.ConnectDevice instead of Device.Connect. Will do. Thanks. > >>> From: linux-bluetooth-owner@xxxxxxxxxxxxxxx [mailto:linux-bluetooth- > >>> owner@xxxxxxxxxxxxxxx] On Behalf Of Karabin, Josh > >>> Sent: Tuesday, August 21, 2018 11:37 AM > >>> To: linux-bluetooth@xxxxxxxxxxxxxxx > >>> Subject: [External] [PATCH BlueZ v2 0/2] Change ConnectDevice to > connect a > >>> subset of services > >>> > >>> This patch set makes it possible to use the experimental Adapter1 > >>> ConnectDevice DBUS API to connect to a device without also > >>> connecting each of the device's services. This enables the out-of-band > >>> discovery of an address and expected service set without a > >>> general discovery procedure. > >>> > >>> This patch builds on this previous commit: > >>> > >>> commit c3f7b6733c5f ("adapter: Add ConnectDevice method") > >>> > >>> and review feedback on an earlier patch here: > >>> > >>> https://marc.info/?l=linux-bluetooth&m=153451634605069&w=2 > >>> > >>> George Josh Karabin (2): > >>> Allow Adapter1 ConnectDevice method to connect a limited set of > >>> service UUIDs > >>> Update the Adapter1 API to match the change to ConnectDevice. > >>> > >>> George Josh Karabin (2): > >>> Allow Adapter1 ConnectDevice method to connect a limited set of > >>> service UUIDs > >>> Update the Adapter1 API to match the change to ConnectDevice. > >>> > >>> doc/adapter-api.txt | 16 +++++--- > >>> src/adapter.c | 91 > +++++++++++++++++++++++++++++++++++++++++-- > >>> -- > >>> 2 files changed, 95 insertions(+), 12 deletions(-) > >>> > >>> -- > >>> 2.17.1 > > > > > > > > -- > > Luiz Augusto von Dentz > > > > -- > Luiz Augusto von Dentz
