From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <3ad703600712210117o284990d5mbd3928923845abe7@mail.gmail.com> Date: Fri, 21 Dec 2007 10:17:22 +0100 From: "Geoffrey Doret" To: "BlueZ development" In-Reply-To: <1198163860.8050.257.camel@aeonflux> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_397_8340647.1198228643067" References: <3ad703600712110300j7a3e628cj1e4d55dd9d4a0d5b@mail.gmail.com> <1197562554.5644.95.camel@aeonflux> <3ad703600712200615i1c039fbep5076ac119bddef61@mail.gmail.com> <1198163860.8050.257.camel@aeonflux> Subject: Re: [Bluez-devel] Patch proposal for serial manager Reply-To: BlueZ development List-Id: BlueZ development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: bluez-devel-bounces@lists.sourceforge.net Errors-To: bluez-devel-bounces@lists.sourceforge.net ------=_Part_397_8340647.1198228643067 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Hi Marcel, > > it has to take "hciX" strings or the local BD_ADDR of the adapter. Both > must work. This is how other methods handle this. The reason behind it > is that as an application developer using this method, you should not > worry about the details. The service should do all the heavy lifting. If I've understood, that is how hci_devid handle its parameter? If that is correct, the new adapter parameter is correctly handled. > > Don't forget to document this new method in serial-api.txt. Documentation is included in the last patch. Regards, Geoffrey Doret ------=_Part_397_8340647.1198228643067 Content-Type: text/x-patch; name=connect_service.patch Content-Transfer-Encoding: base64 X-Attachment-Id: f_fagmoyv20 Content-Disposition: attachment; filename=connect_service.patch SW5kZXg6IHV0aWxzL3NlcmlhbC9tYW5hZ2VyLmMKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2c3Jv b3QvYmx1ZXovdXRpbHMvc2VyaWFsL21hbmFnZXIuYyx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS4x MDAKZGlmZiAtdSAtcjEuMTAwIG1hbmFnZXIuYwotLS0gdXRpbHMvc2VyaWFsL21hbmFnZXIuYwkx MyBEZWMgMjAwNyAxOTowNzozNyAtMDAwMAkxLjEwMAorKysgdXRpbHMvc2VyaWFsL21hbmFnZXIu YwkyMCBEZWMgMjAwNyAxNTo1ODowNSAtMDAwMApAQCAtMTk5MSwzMiArMTk5MSwyMCBAQAogCQkJ ZGJ1c19tZXNzYWdlX25ld19tZXRob2RfcmV0dXJuKG1zZykpOwogfQogCi1zdGF0aWMgREJ1c0hh bmRsZXJSZXN1bHQgY29ubmVjdF9zZXJ2aWNlKERCdXNDb25uZWN0aW9uICpjb25uLAotCQkJCQlE QnVzTWVzc2FnZSAqbXNnLCB2b2lkICpkYXRhKQorc3RhdGljIERCdXNIYW5kbGVyUmVzdWx0IGNv bm5lY3Rfc2VydmljZV9ieV9kZXZpZChEQnVzQ29ubmVjdGlvbiAqY29ubiwKKwkJCQlEQnVzTWVz c2FnZSAqbXNnLCB2b2lkICpkYXRhLCBpbnQgZGV2X2lkLAorCQkJCWNvbnN0IGNoYXIgKmJkYSwg Y29uc3QgY2hhciAqcGF0dGVybikKIHsKIAlzdHJ1Y3QgcGVuZGluZ19jb25uZWN0ICpwZW5kaW5n LCAqcGM7Ci0JREJ1c0Vycm9yIGRlcnI7CiAJYmRhZGRyX3Qgc3JjOwotCWNvbnN0IGNoYXIgKmJk YSwgKnBhdHRlcm47CiAJbG9uZyB2YWw7Ci0JaW50IGRldl9pZCwgZXJyOworCWludCBlcnI7CiAJ Y2hhciB1dWlkW01BWF9MRU5fVVVJRF9TVFJdOwogCi0JZGJ1c19lcnJvcl9pbml0KCZkZXJyKTsK LQlpZiAoIWRidXNfbWVzc2FnZV9nZXRfYXJncyhtc2csICZkZXJyLAotCQkJCURCVVNfVFlQRV9T VFJJTkcsICZiZGEsCi0JCQkJREJVU19UWVBFX1NUUklORywgJnBhdHRlcm4sCi0JCQkJREJVU19U WVBFX0lOVkFMSUQpKSB7Ci0JCWVycm9yX2ludmFsaWRfYXJndW1lbnRzKGNvbm4sIG1zZywgZGVy ci5tZXNzYWdlKTsKLQkJZGJ1c19lcnJvcl9mcmVlKCZkZXJyKTsKLQkJcmV0dXJuIERCVVNfSEFO RExFUl9SRVNVTFRfSEFORExFRDsKLQl9Ci0KIAlwZW5kaW5nID0gZmluZF9wZW5kaW5nX2Nvbm5l Y3RfYnlfcGF0dGVybihiZGEsIHBhdHRlcm4pOwogCWlmIChwZW5kaW5nKQogCQlyZXR1cm4gZXJy b3JfaW5fcHJvZ3Jlc3MoY29ubiwgbXNnLCAiQ29ubmVjdGlvbiBpbiBwcm9ncmVzcyIpOwogCi0J ZGV2X2lkID0gaGNpX2dldF9yb3V0ZShOVUxMKTsKIAlpZiAoKGRldl9pZCA8IDApIHx8IChoY2lf ZGV2YmEoZGV2X2lkLCAmc3JjKSA8IDApKQogCQlyZXR1cm4gZXJyb3JfZmFpbGVkKGNvbm4sIG1z ZywgIkFkYXB0ZXIgbm90IGF2YWlsYWJsZSIpOwogCkBAIC0yMDkxLDYgKzIwNzksNDkgQEAKIAly ZXR1cm4gREJVU19IQU5ETEVSX1JFU1VMVF9IQU5ETEVEOwogfQogCitzdGF0aWMgREJ1c0hhbmRs ZXJSZXN1bHQgY29ubmVjdF9zZXJ2aWNlKERCdXNDb25uZWN0aW9uICpjb25uLAorCQkJCQlEQnVz TWVzc2FnZSAqbXNnLCB2b2lkICpkYXRhKQoreworCURCdXNFcnJvciBkZXJyOworCWNvbnN0IGNo YXIgKmJkYSwgKnBhdHRlcm47CisJaW50IGRldmlkOworCisJZGJ1c19lcnJvcl9pbml0KCZkZXJy KTsKKwlpZiAoIWRidXNfbWVzc2FnZV9nZXRfYXJncyhtc2csICZkZXJyLAorCQkJCURCVVNfVFlQ RV9TVFJJTkcsICZiZGEsCisJCQkJREJVU19UWVBFX1NUUklORywgJnBhdHRlcm4sCisJCQkJREJV U19UWVBFX0lOVkFMSUQpKSB7CisJCWVycm9yX2ludmFsaWRfYXJndW1lbnRzKGNvbm4sIG1zZywg ZGVyci5tZXNzYWdlKTsKKwkJZGJ1c19lcnJvcl9mcmVlKCZkZXJyKTsKKwkJcmV0dXJuIERCVVNf SEFORExFUl9SRVNVTFRfSEFORExFRDsKKwl9CisJZGV2aWQgPSBoY2lfZ2V0X3JvdXRlKE5VTEwp OworCisJcmV0dXJuIGNvbm5lY3Rfc2VydmljZV9ieV9kZXZpZChjb25uLCBtc2csIGRhdGEsIGRl dmlkLCBiZGEsIHBhdHRlcm4pOworfQorCitzdGF0aWMgREJ1c0hhbmRsZXJSZXN1bHQgY29ubmVj dF9zZXJ2aWNlX2J5X2FkYXB0ZXIoREJ1c0Nvbm5lY3Rpb24gKmNvbm4sCisJCQkJCURCdXNNZXNz YWdlICptc2csIHZvaWQgKmRhdGEpCit7CisJREJ1c0Vycm9yIGRlcnI7CisJY29uc3QgY2hhciAq YWRhcHRlciwgKmJkYSwgKnBhdHRlcm47CisJaW50IGRldmlkOworCisJZGJ1c19lcnJvcl9pbml0 KCZkZXJyKTsKKwlpZiAoIWRidXNfbWVzc2FnZV9nZXRfYXJncyhtc2csICZkZXJyLAorCQkJCURC VVNfVFlQRV9TVFJJTkcsICZhZGFwdGVyLAorCQkJCURCVVNfVFlQRV9TVFJJTkcsICZiZGEsCisJ CQkJREJVU19UWVBFX1NUUklORywgJnBhdHRlcm4sCisJCQkJREJVU19UWVBFX0lOVkFMSUQpKSB7 CisJCWVycm9yX2ludmFsaWRfYXJndW1lbnRzKGNvbm4sIG1zZywgZGVyci5tZXNzYWdlKTsKKwkJ ZGJ1c19lcnJvcl9mcmVlKCZkZXJyKTsKKwkJcmV0dXJuIERCVVNfSEFORExFUl9SRVNVTFRfSEFO RExFRDsKKwl9CisJZGV2aWQgPSBoY2lfZGV2aWQoYWRhcHRlcik7CisKKwlyZXR1cm4gY29ubmVj dF9zZXJ2aWNlX2J5X2RldmlkKGNvbm4sIG1zZywgZGF0YSwgZGV2aWQsIGJkYSwgcGF0dGVybik7 Cit9CisKIHN0YXRpYyBEQnVzSGFuZGxlclJlc3VsdCBkaXNjb25uZWN0X3NlcnZpY2UoREJ1c0Nv bm5lY3Rpb24gKmNvbm4sCiAJCQkJCURCdXNNZXNzYWdlICptc2csIHZvaWQgKmRhdGEpCiB7CkBA IC0yMjE5LDE1ICsyMjUwLDE2IEBACiB9CiAKIHN0YXRpYyBEQnVzTWV0aG9kVlRhYmxlIG1hbmFn ZXJfbWV0aG9kc1tdID0gewotCXsgIkNyZWF0ZVBvcnQiLAkJCWNyZWF0ZV9wb3J0LAkJInNzIiwJ InMiCX0sCi0JeyAiTGlzdFBvcnRzIiwJCQlsaXN0X3BvcnRzLAkJIiIsCSJhcyIJfSwKLQl7ICJS ZW1vdmVQb3J0IiwJCQlyZW1vdmVfcG9ydCwJCSJzIiwJIiIJfSwKLQl7ICJDcmVhdGVQcm94eSIs CQljcmVhdGVfcHJveHksCQkic3MiLAkicyIJfSwKLQl7ICJMaXN0UHJveGllcyIsCQlsaXN0X3By b3hpZXMsCQkiIiwJImFzIgl9LAotCXsgIlJlbW92ZVByb3h5IiwJCXJlbW92ZV9wcm94eSwJCSJz IiwJIiIJfSwKLQl7ICJDb25uZWN0U2VydmljZSIsCQljb25uZWN0X3NlcnZpY2UsCSJzcyIsCSJz Igl9LAotCXsgIkRpc2Nvbm5lY3RTZXJ2aWNlIiwJCWRpc2Nvbm5lY3Rfc2VydmljZSwJInMiLAki Igl9LAotCXsgIkNhbmNlbENvbm5lY3RTZXJ2aWNlIiwJY2FuY2VsX2Nvbm5lY3Rfc2VydmljZSwJ InNzIiwJIiIJfSwKKwl7ICJDcmVhdGVQb3J0IiwJCQljcmVhdGVfcG9ydCwJCQkic3MiLAkicyIJ fSwKKwl7ICJMaXN0UG9ydHMiLAkJCWxpc3RfcG9ydHMsCQkJIiIsCSJhcyIJfSwKKwl7ICJSZW1v dmVQb3J0IiwJCQlyZW1vdmVfcG9ydCwJCQkicyIsCSIiCX0sCisJeyAiQ3JlYXRlUHJveHkiLAkJ Y3JlYXRlX3Byb3h5LAkJCSJzcyIsCSJzIgl9LAorCXsgIkxpc3RQcm94aWVzIiwJCWxpc3RfcHJv eGllcywJCQkiIiwJImFzIgl9LAorCXsgIlJlbW92ZVByb3h5IiwJCXJlbW92ZV9wcm94eSwJCQki cyIsCSIiCX0sCisJeyAiQ29ubmVjdFNlcnZpY2UiLAkJY29ubmVjdF9zZXJ2aWNlLAkJInNzIiwJ InMiCX0sCisJeyAiQ29ubmVjdFNlcnZpY2VCeUFkYXB0ZXIiLAljb25uZWN0X3NlcnZpY2VfYnlf YWRhcHRlciwJInNzcyIsCSJzIgl9LAorCXsgIkRpc2Nvbm5lY3RTZXJ2aWNlIiwJCWRpc2Nvbm5l Y3Rfc2VydmljZSwJCSJzIiwJIiIJfSwKKwl7ICJDYW5jZWxDb25uZWN0U2VydmljZSIsCWNhbmNl bF9jb25uZWN0X3NlcnZpY2UsCQkic3MiLAkiIgl9LAogCXsgTlVMTCwgTlVMTCwgTlVMTCwgTlVM TCB9LAogfTsKIApJbmRleDogdXRpbHMvc2VyaWFsL3NlcmlhbC1hcGkudHh0Cj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K UkNTIGZpbGU6IC9jdnNyb290L2JsdWV6L3V0aWxzL3NlcmlhbC9zZXJpYWwtYXBpLnR4dCx2CnJl dHJpZXZpbmcgcmV2aXNpb24gMS4yMQpkaWZmIC11IC1yMS4yMSBzZXJpYWwtYXBpLnR4dAotLS0g dXRpbHMvc2VyaWFsL3NlcmlhbC1hcGkudHh0CTI2IE5vdiAyMDA3IDEzOjQxOjM5IC0wMDAwCTEu MjEKKysrIHV0aWxzL3NlcmlhbC9zZXJpYWwtYXBpLnR4dAkyMCBEZWMgMjAwNyAxNTo1ODowNSAt MDAwMApAQCAtNjgsNiArNjgsMjEgQEAKIAkJCQkJb3JnLmJsdWV6LkVycm9yLk5vdFN1cHBvcnRl ZAogCQkJCQlvcmcuYmx1ZXouRXJyb3IuRmFpbGVkCiAKKwkJc3RyaW5nIENvbm5lY3RTZXJ2aWNl QnlBZGFwdGVyKHN0cmluZyBhZGFwdGVyLCBzdHJpbmcgYWRkcmVzcywKKwkJCQlzdHJpbmcgcGF0 dGVybikKKworCQkJVXNlIHRoZSBnaXZlbiBhZGFwdGVyIHRvIGNvbm5lY3RzIHRvIGEgc3BlY2lm aWMgUkZDT01NCisJCQliYXNlZCBzZXJ2aWNlIG9uIGEgcmVtb3RlIGRldmljZSBhbmQgdGhlbiBj cmVhdGVzIGEKKwkJCVJGQ09NTSBUVFkgZGV2aWNlIGZvciBpdC4gVGhlIFJGQ09NTSBUVFkgZGV2 aWNlIGlzCisJCQlyZXR1cm5lZC4KKworCQkJUG9zc2libGUgZXJyb3JzOm9yZy5ibHVlei5FcnJv ci5JbnZhbGlkQXJndW1lbnRzCisJCQkJCW9yZy5ibHVlei5FcnJvci5DYW5jZWxlZAorCQkJCQlv cmcuYmx1ZXouRXJyb3IuSW5Qcm9ncmVzcworCQkJCQlvcmcuYmx1ZXouRXJyb3IuQ29ubmVjdGlv bkF0dGVtcHRGYWlsZWQKKwkJCQkJb3JnLmJsdWV6LkVycm9yLk5vdFN1cHBvcnRlZAorCQkJCQlv cmcuYmx1ZXouRXJyb3IuRmFpbGVkCisKIAkJdm9pZCBDYW5jZWxDb25uZWN0U2VydmljZShzdHJp bmcgYWRkcmVzcywgc3RyaW5nIHBhdHRlcm4pCiAKIAkJCUNhbmNlbCBhIHByZXZpb3VzIENvbm5l Y3RTZXJ2aWNlIG1ldGhvZCBjYWxsLgo= ------=_Part_397_8340647.1198228643067 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ ------=_Part_397_8340647.1198228643067 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Bluez-devel mailing list Bluez-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bluez-devel ------=_Part_397_8340647.1198228643067--