Hi Ujjal, >>> /* 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); this is for clearing all flags except the HCI_RAW bit. 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