Hi Anderson, >> 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? yes, this it should be 0x03 here. My mistake. 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
