Hi Suraj, > Implements Atheros AR300x Serial HCI protocol. > This protocol extends H4 serial protocol to implement enhanced power management > features supported by Atheros AR300x serial Bluetooth chipsets. <snip> > +struct ath_struct { > + struct hci_uart *hu; > + unsigned int cur_sleep; > + > + spinlock_t hciath_lock; > + struct sk_buff *rx_skb; > + struct sk_buff_head txq; > + wait_queue_head_t wqevt; > + struct work_struct ctxtsw; > +}; I was going to merge this as it is, but then I did a usage check on your variables here and realized that you carry garbage around. I really would prefer if you guys start cleaning up your patches before blindly submitting them. I am not your janitor. So hciath_lock, rx_skb and wqevt are unused in the whole driver. Please remove them from the patch. 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