>> /* Clear flags */ >> - hdev->flags = 0; >> + hdev->flags &= BIT(HCI_RAW); Is it for clearing the HCI_RAW flag? If so then it should be as below am I right? /* Clear flags */ - hdev->flags = 0; + hdev->flags &= ~BIT(HCI_RAW); Thanks, UjjaL Roy -- 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