Hi Andrei, > hdev is allocated with kzalloc so zero initialization is not needed. > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > include/net/bluetooth/hci_core.h | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h > index 36720f0..05bcea6 100644 > --- a/include/net/bluetooth/hci_core.h > +++ b/include/net/bluetooth/hci_core.h > @@ -443,10 +443,8 @@ static inline bool hci_conn_ssp_enabled(struct hci_conn *conn) > static inline void hci_conn_hash_init(struct hci_dev *hdev) > { > struct hci_conn_hash *h = &hdev->conn_hash; > + > INIT_LIST_HEAD(&h->list); > - h->acl_num = 0; > - h->sco_num = 0; > - h->le_num = 0; > } since there is only one user of hci_conn_hash_init, we should just replace that call with a direct INIT_LIST_HEAD. 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