Hi Marcel, On Sun, Sep 29, 2013 at 12:09 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > The controller type is limited to BR/EDR/LE and AMP controllers. This > can be easily encoded with just 2 bits and still leave enough room > for future controller types. > > Signed-off-by: Marcel Holtmann <marcel@xxxxxxxxxxxx> > --- > net/bluetooth/hci_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c > index 4549b5c..ebba5a6 100644 > --- a/net/bluetooth/hci_core.c > +++ b/net/bluetooth/hci_core.c > @@ -1610,7 +1610,7 @@ int hci_get_dev_info(void __user *arg) > > strcpy(di.name, hdev->name); > di.bdaddr = hdev->bdaddr; > - di.type = (hdev->bus & 0x0f) | (hdev->dev_type << 4); > + di.type = (hdev->bus & 0x0f) | ((hdev->dev_type & 0x30) << 4); Shouldn't it be (hdev->dev_type & 0x03) here? Best Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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