Hi Andrei, > > > Make sure that code match exactly what comment says about pre 1.2 > > > LMP version. > > > > > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > > --- > > > net/bluetooth/hci_event.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > Acked-by: Marcel Holtmann <marcel@xxxxxxxxxxxx> > > > > > > > > diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c > > > index 60651fa..27b1ede 100644 > > > --- a/net/bluetooth/hci_event.c > > > +++ b/net/bluetooth/hci_event.c > > > @@ -498,7 +498,7 @@ static void hci_setup_event_mask(struct hci_dev *hdev) > > > > > > /* CSR 1.1 dongles does not accept any bitfield so don't try to set > > > * any event mask for pre 1.2 devices */ > > > - if (hdev->lmp_ver <= BLUETOOTH_VER_1_1) > > > + if (hdev->lmp_ver < BLUETOOTH_VER_1_2) > > > return; > > > > However, even here you need to use hci_ver since it is a HCI detail and > > has nothing to do with the LMP version. > > Do you think can I amend this patch and change lmp/hci ? your choice actually. I do not care. Treat both possibilities as acked by me. Regards Marcel -- 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
