Hi, On Thu, Oct 13, 2011 at 4:35 AM, Keng-Yu Lin <kengyu@xxxxxxxxxx> wrote: > init_adapter() is not called due to missing HCI commands of > read_bd_addr, read_local_version, read_local_features, > and read_local_name on some machines. > --- > plugins/hciops.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > Please see the links of the bug in different distros: > https://bugzilla.redhat.com/show_bug.cgi?id=727106 > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=614929 > https://launchpad.net/bugs/824144 > https://launchpad.net/bugs/824633 > https://launchpad.net/bugs/823467 > https://launchpad.net/bugs/822705 > > It is confirmed that this patch fixes for Dell Latutude 2120 > and EeePC 1001PXD. > > diff --git a/plugins/hciops.c b/plugins/hciops.c > index 9fb28c8..5f339c1 100644 > --- a/plugins/hciops.c > +++ b/plugins/hciops.c > @@ -2502,8 +2502,8 @@ static void device_devup_setup(int index) > hci_send_cmd(dev->sk, OGF_HOST_CTL, OCF_READ_STORED_LINK_KEY, > READ_STORED_LINK_KEY_CP_SIZE, &cp); > > - if (!dev->pending) > - init_adapter(index); > + DBG("dev->pending=%d", dev->pending); > + init_adapter(index); > } Thanks for the patch, but I suspect this may cause some problems if we didn't receive all the pending information, do you have bluetoothd logs showing the sequence of the events leading to this? -- 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
