Bluetooth: Use sizeof(*pointer) instead of sizeof(type)

It is preferred to use sizeof(*pointer) instead of sizeof(type)
due to the type of the variable can change and one needs not
change the former (unlike the latter). This patch has no effect
on runtime behavior.

Signed-off-by: Erick Archer <erick.archer@outlook.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

diff --git a/drivers/bluetooth/btrtl.c b/drivers/bluetooth/btrtl.c
index 4f1e37b4f7802daceffc1ea46bd943215fcba4bd..f2f37143c454d5c18adeba6087d97d32bea4905e 100644 (file)
--- a/drivers/bluetooth/btrtl.c
+++ b/drivers/bluetooth/btrtl.c
@@ -811,7 +811,7 @@ static int rtl_download_firmware(struct hci_dev *hdev,
        struct sk_buff *skb;
        struct hci_rp_read_local_version *rp;
 
-       dl_cmd = kmalloc(sizeof(struct rtl_download_cmd), GFP_KERNEL);
+       dl_cmd = kmalloc(sizeof(*dl_cmd), GFP_KERNEL);
        if (!dl_cmd)
                return -ENOMEM;
 

diff --git a/drivers/bluetooth/hci_ldisc.c b/drivers/bluetooth/hci_ldisc.c
index 17a2f158a0dfab2b7832819cfe01f1d004508dc0..30192bb083549ee37bb94263c0c5bc24c64d7f86 100644 (file)
--- a/drivers/bluetooth/hci_ldisc.c
+++ b/drivers/bluetooth/hci_ldisc.c
@@ -488,7 +488,7 @@ static int hci_uart_tty_open(struct tty_struct *tty)
        if (tty->ops->write == NULL)
                return -EOPNOTSUPP;
 
-       hu = kzalloc(sizeof(struct hci_uart), GFP_KERNEL);
+       hu = kzalloc(sizeof(*hu), GFP_KERNEL);
        if (!hu) {
                BT_ERR("Can't allocate control structure");
                return -ENFILE;

diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index 9a0bc86f9aace53282742b381f08bf3ac93e230b..2f47267508d5466ceb527f69bbf2c7f1670f7259 100644 (file)
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -569,7 +569,7 @@ static int qca_open(struct hci_uart *hu)
        if (!hci_uart_has_flow_control(hu))
                return -EOPNOTSUPP;
 
-       qca = kzalloc(sizeof(struct qca_data), GFP_KERNEL);
+       qca = kzalloc(sizeof(*qca), GFP_KERNEL);
        if (!qca)
                return -ENOMEM;
 
@@ -1040,8 +1040,7 @@ static void qca_controller_memdump(struct work_struct *work)
                }
 
                if (!qca_memdump) {
-                       qca_memdump = kzalloc(sizeof(struct qca_memdump_info),
-                                             GFP_ATOMIC);
+                       qca_memdump = kzalloc(sizeof(*qca_memdump), GFP_ATOMIC);
                        if (!qca_memdump) {
                                mutex_unlock(&qca->hci_memdump_lock);
                                return;

diff --git a/drivers/bluetooth/hci_vhci.c b/drivers/bluetooth/hci_vhci.c
index 28750a40f0ed52aab23f519e47bdd886d46780bd..c4046f8f1985ab21ee7266818b3cda4e0a32f630 100644 (file)
--- a/drivers/bluetooth/hci_vhci.c
+++ b/drivers/bluetooth/hci_vhci.c
@@ -633,7 +633,7 @@ static int vhci_open(struct inode *inode, struct file *file)
 {
        struct vhci_data *data;
 
-       data = kzalloc(sizeof(struct vhci_data), GFP_KERNEL);
+       data = kzalloc(sizeof(*data), GFP_KERNEL);
        if (!data)
                return -ENOMEM;