Bluetooth: Remove unneeded initialization in hci_alloc_dev()
authorDavid Herrmann <dh.herrmann@googlemail.com>
Sun, 22 Apr 2012 12:39:59 +0000 (14:39 +0200)
committerGustavo Padovan <gustavo@padovan.org>
Wed, 9 May 2012 04:40:40 +0000 (01:40 -0300)
We allocate memory with kzalloc() so there is no need to call
memset(..., 0, ...) or similar.

Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/hci_core.c

index a362f01..aa45ea4 100644 (file)
@@ -1713,20 +1713,16 @@ int hci_le_scan(struct hci_dev *hdev, u8 type, u16 interval, u16 window,
 struct hci_dev *hci_alloc_dev(void)
 {
        struct hci_dev *hdev;
-       int i;
 
        hdev = kzalloc(sizeof(struct hci_dev), GFP_KERNEL);
        if (!hdev)
                return NULL;
 
-       hdev->flags = 0;
-       hdev->dev_flags = 0;
        hdev->pkt_type  = (HCI_DM1 | HCI_DH1 | HCI_HV1);
        hdev->esco_type = (ESCO_HV1);
        hdev->link_mode = (HCI_LM_ACCEPT);
        hdev->io_capability = 0x03; /* No Input No Output */
 
-       hdev->idle_timeout = 0;
        hdev->sniff_max_interval = 800;
        hdev->sniff_min_interval = 80;
 
@@ -1761,12 +1757,6 @@ struct hci_dev *hci_alloc_dev(void)
 
        setup_timer(&hdev->cmd_timer, hci_cmd_timer, (unsigned long) hdev);
 
-       memset(&hdev->stat, 0, sizeof(struct hci_dev_stats));
-       atomic_set(&hdev->promisc, 0);
-
-       for (i = 0; i < NUM_REASSEMBLY; i++)
-               hdev->reassembly[i] = NULL;
-
        hci_init_sysfs(hdev);
        discovery_init(hdev);
        hci_conn_hash_init(hdev);