Ping. > -----Original Message----- > From: linux-bluetooth-owner@xxxxxxxxxxxxxxx [mailto:linux-bluetooth- > owner@xxxxxxxxxxxxxxx] On Behalf Of Gowtham Anandha Babu > Sent: Monday, February 02, 2015 12:21 PM > To: 'Luiz Augusto von Dentz' > Cc: linux-bluetooth@xxxxxxxxxxxxxxx; 'Bharat Panda'; cpgs@xxxxxxxxxxx > Subject: RE: [PATCH] test/pbap-client: Add support to fetch spd,fav PB > > Hi Luiz, > > > -----Original Message----- > > From: linux-bluetooth-owner@xxxxxxxxxxxxxxx [mailto:linux-bluetooth- > > owner@xxxxxxxxxxxxxxx] On Behalf Of Luiz Augusto von Dentz > > Sent: Friday, January 30, 2015 9:03 PM > > To: Gowtham Anandha Babu > > Cc: linux-bluetooth@xxxxxxxxxxxxxxx; Bharat Panda; cpgs@xxxxxxxxxxx > > Subject: Re: [PATCH] test/pbap-client: Add support to fetch spd,fav PB > > > > Hi Gowtham, > > > > On Fri, Jan 30, 2015 at 4:07 PM, Gowtham Anandha Babu > > <gowtham.ab@xxxxxxxxxxx> wrote: > > > Ping. > > > > > >> -----Original Message----- > > >> From: linux-bluetooth-owner@xxxxxxxxxxxxxxx > > >> [mailto:linux-bluetooth- owner@xxxxxxxxxxxxxxx] On Behalf Of > > >> Gowtham Anandha Babu > > >> Sent: Wednesday, January 28, 2015 7:12 PM > > >> To: linux-bluetooth@xxxxxxxxxxxxxxx > > >> Cc: bharat.panda@xxxxxxxxxxx; cpgs@xxxxxxxxxxx; Gowtham > > Anandha Babu > > >> Subject: [PATCH] test/pbap-client: Add support to fetch spd,fav PB > > >> > > >> Adds support to fetch spd,fav phonebook. > > >> Tested using PTS test case TC_PCE_PBB_BV_01_C. > > >> --- > > >> test/pbap-client | 2 +- > > >> 1 file changed, 1 insertion(+), 1 deletion(-) > > >> > > >> diff --git a/test/pbap-client b/test/pbap-client index > > >> 51e26eb..16a786b > > >> 100755 > > >> --- a/test/pbap-client > > >> +++ b/test/pbap-client > > >> @@ -170,6 +170,6 @@ if __name__ == '__main__': > > >> > > >> pbap_client.flush_transfers(lambda: > > >> test_paths(paths[1:])) > > >> > > >> - test_paths(["PB", "ICH", "OCH", "MCH", "CCH"]) > > >> + test_paths(["PB", "ICH", "OCH", "MCH", "CCH", "SPD", "FAV"]) > > >> > > >> mainloop.run() > > >> -- > > >> 1.9.1 > > > > We should be doing this with obexctl, is there something missing there? > > > > > > In obexctl, we are selecting the phonebook based on user input. We are > passing that as a dbus args simply without checking. > All this phonebook objects ["PB", "ICH", "OCH", "MCH", "CCH", "SPD", "FAV"] > are getting selected using obexctl. > If we give other than the above mentioned phonebook, its showing ' > org.bluez.obex.Error.InvalidArguments '. > Below is the log for the same. > > [00:1B:DC:07:33:4E]# cd pb > Attempting to Select to pb > Select successful > [00:1B:DC:07:33:4E]# cd och > Attempting to Select to och > Select successful > [00:1B:DC:07:33:4E]# cd ich > Attempting to Select to ich > Select successful > [00:1B:DC:07:33:4E]# cd mch > Attempting to Select to mch > Select successful > [00:1B:DC:07:33:4E]# cd spd > Attempting to Select to spd > Select successful > [00:1B:DC:07:33:4E]# cd fav > Attempting to Select to fav > Select successful > [00:1B:DC:07:33:4E]# cd sdfa > Attempting to Select to sdfa > Failed to Select: org.bluez.obex.Error.InvalidArguments > > Do we need to add a check before passing this args? Because dbus itself is > handling this issue by sending 'org.bluez.obex.Error.InvalidArguments'. > I don’t think extra check is not required here. > What do you think? > > Regards, > Gowtham Anandha Babu > > > -- > > Luiz Augusto von Dentz > > -- > > To unsubscribe from this list: send the line "unsubscribe > > linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in > the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo > info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
