staging: rtl8712: hal_init.c: Remove leading p from variable names
authorNishka Dasgupta <nishkadg.linux@gmail.com>
Fri, 7 Jun 2019 05:42:36 +0000 (11:12 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 9 Jun 2019 11:03:52 +0000 (13:03 +0200)
Remove the leading p from the following pointer variable names:
- padapter
- pusb_intf
- ppmappedfw
- praw
- pfwpriv
- pdvobj
- pregpriv
- pmappedfw (not in the same scope as ppmappedfw)
- ptmpchar
- ppayload
- ptx_desc
Issue found with Coccinelle.

Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8712/hal_init.c

index 401f0e4..40145c0 100644 (file)
 
 static void rtl871x_load_fw_cb(const struct firmware *firmware, void *context)
 {
-       struct _adapter *padapter = context;
+       struct _adapter *adapter = context;
 
-       complete(&padapter->rtl8712_fw_ready);
+       complete(&adapter->rtl8712_fw_ready);
        if (!firmware) {
-               struct usb_device *udev = padapter->dvobjpriv.pusbdev;
-               struct usb_interface *pusb_intf = padapter->pusb_intf;
+               struct usb_device *udev = adapter->dvobjpriv.pusbdev;
+               struct usb_interface *usb_intf = adapter->pusb_intf;
 
                dev_err(&udev->dev, "r8712u: Firmware request failed\n");
                usb_put_dev(udev);
-               usb_set_intfdata(pusb_intf, NULL);
+               usb_set_intfdata(usb_intf, NULL);
                return;
        }
-       padapter->fw = firmware;
+       adapter->fw = firmware;
        /* firmware available - start netdev */
-       register_netdev(padapter->pnetdev);
+       register_netdev(adapter->pnetdev);
 }
 
 static const char firmware_file[] = "rtlwifi/rtl8712u.bin";
@@ -65,47 +65,47 @@ int rtl871x_load_fw(struct _adapter *padapter)
 }
 MODULE_FIRMWARE("rtlwifi/rtl8712u.bin");
 
-static u32 rtl871x_open_fw(struct _adapter *padapter, const u8 **ppmappedfw)
+static u32 rtl871x_open_fw(struct _adapter *adapter, const u8 **mappedfw)
 {
-       const struct firmware **praw = &padapter->fw;
+       const struct firmware **raw = &adapter->fw;
 
-       if (padapter->fw->size > 200000) {
-               dev_err(&padapter->pnetdev->dev, "r8172u: Badfw->size of %d\n",
-                       (int)padapter->fw->size);
+       if (adapter->fw->size > 200000) {
+               dev_err(&adapter->pnetdev->dev, "r8172u: Badfw->size of %d\n",
+                       (int)adapter->fw->size);
                return 0;
        }
-       *ppmappedfw = (*praw)->data;
-       return (*praw)->size;
+       *mappedfw = (*raw)->data;
+       return (*raw)->size;
 }
 
-static void fill_fwpriv(struct _adapter *padapter, struct fw_priv *pfwpriv)
+static void fill_fwpriv(struct _adapter *adapter, struct fw_priv *fwpriv)
 {
-       struct dvobj_priv *pdvobj = &padapter->dvobjpriv;
-       struct registry_priv *pregpriv = &padapter->registrypriv;
+       struct dvobj_priv *dvobj = &adapter->dvobjpriv;
+       struct registry_priv *regpriv = &adapter->registrypriv;
 
-       memset(pfwpriv, 0, sizeof(struct fw_priv));
+       memset(fwpriv, 0, sizeof(struct fw_priv));
        /* todo: check if needs endian conversion */
-       pfwpriv->hci_sel =  RTL8712_HCI_TYPE_72USB;
-       pfwpriv->usb_ep_num = (u8)pdvobj->nr_endpoint;
-       pfwpriv->bw_40MHz_en = pregpriv->cbw40_enable;
-       switch (pregpriv->rf_config) {
+       fwpriv->hci_sel =  RTL8712_HCI_TYPE_72USB;
+       fwpriv->usb_ep_num = (u8)dvobj->nr_endpoint;
+       fwpriv->bw_40MHz_en = regpriv->cbw40_enable;
+       switch (regpriv->rf_config) {
        case RTL8712_RF_1T1R:
-               pfwpriv->rf_config = RTL8712_RFC_1T1R;
+               fwpriv->rf_config = RTL8712_RFC_1T1R;
                break;
        case RTL8712_RF_2T2R:
-               pfwpriv->rf_config = RTL8712_RFC_2T2R;
+               fwpriv->rf_config = RTL8712_RFC_2T2R;
                break;
        case RTL8712_RF_1T2R:
        default:
-               pfwpriv->rf_config = RTL8712_RFC_1T2R;
+               fwpriv->rf_config = RTL8712_RFC_1T2R;
        }
-       pfwpriv->mp_mode = (pregpriv->mp_mode == 1) ? 1 : 0;
+       fwpriv->mp_mode = (regpriv->mp_mode == 1) ? 1 : 0;
        /* 0:off 1:on 2:auto */
-       pfwpriv->vcs_type = pregpriv->vrtl_carrier_sense;
-       pfwpriv->vcs_mode = pregpriv->vcs_type; /* 1:RTS/CTS 2:CTS to self */
+       fwpriv->vcs_type = regpriv->vrtl_carrier_sense;
+       fwpriv->vcs_mode = regpriv->vcs_type; /* 1:RTS/CTS 2:CTS to self */
        /* default enable turbo_mode */
-       pfwpriv->turbo_mode = ((pregpriv->wifi_test == 1) ? 0 : 1);
-       pfwpriv->low_power_mode = pregpriv->low_power;
+       fwpriv->turbo_mode = ((regpriv->wifi_test == 1) ? 0 : 1);
+       fwpriv->low_power_mode = regpriv->low_power;
 }
 
 static void update_fwhdr(struct fw_hdr *pfwhdr, const u8 *pmappedfw)
@@ -141,7 +141,7 @@ static u8 chk_fwhdr(struct fw_hdr *pfwhdr, u32 ulfilelength)
        return _SUCCESS;
 }
 
-static u8 rtl8712_dl_fw(struct _adapter *padapter)
+static u8 rtl8712_dl_fw(struct _adapter *adapter)
 {
        sint i;
        u8 tmp8, tmp8_a;
@@ -150,56 +150,56 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter)
        uint dump_imem_sz, imem_sz, dump_emem_sz, emem_sz; /* max = 49152; */
        struct fw_hdr fwhdr;
        u32 ulfilelength;       /* FW file size */
-       const u8 *pmappedfw = NULL;
-       u8 *ptmpchar = NULL, *ppayload, *ptr;
-       struct tx_desc *ptx_desc;
+       const u8 *mappedfw = NULL;
+       u8 *tmpchar = NULL, *payload, *ptr;
+       struct tx_desc *txdesc;
        u32 txdscp_sz = sizeof(struct tx_desc);
        u8 ret = _FAIL;
 
-       ulfilelength = rtl871x_open_fw(padapter, &pmappedfw);
-       if (pmappedfw && (ulfilelength > 0)) {
-               update_fwhdr(&fwhdr, pmappedfw);
+       ulfilelength = rtl871x_open_fw(adapter, &mappedfw);
+       if (mappedfw && (ulfilelength > 0)) {
+               update_fwhdr(&fwhdr, mappedfw);
                if (chk_fwhdr(&fwhdr, ulfilelength) == _FAIL)
                        return ret;
-               fill_fwpriv(padapter, &fwhdr.fwpriv);
+               fill_fwpriv(adapter, &fwhdr.fwpriv);
                /* firmware check ok */
                maxlen = (fwhdr.img_IMEM_size > fwhdr.img_SRAM_size) ?
                          fwhdr.img_IMEM_size : fwhdr.img_SRAM_size;
                maxlen += txdscp_sz;
-               ptmpchar = kmalloc(maxlen + FWBUFF_ALIGN_SZ, GFP_KERNEL);
-               if (!ptmpchar)
+               tmpchar = kmalloc(maxlen + FWBUFF_ALIGN_SZ, GFP_KERNEL);
+               if (!tmpchar)
                        return ret;
 
-               ptx_desc = (struct tx_desc *)(ptmpchar + FWBUFF_ALIGN_SZ -
-                           ((addr_t)(ptmpchar) & (FWBUFF_ALIGN_SZ - 1)));
-               ppayload = (u8 *)(ptx_desc) + txdscp_sz;
-               ptr = (u8 *)pmappedfw + FIELD_OFFSET(struct fw_hdr, fwpriv) +
+               txdesc = (struct tx_desc *)(tmpchar + FWBUFF_ALIGN_SZ -
+                           ((addr_t)(tmpchar) & (FWBUFF_ALIGN_SZ - 1)));
+               payload = (u8 *)(txdesc) + txdscp_sz;
+               ptr = (u8 *)mappedfw + FIELD_OFFSET(struct fw_hdr, fwpriv) +
                      fwhdr.fw_priv_sz;
                /* Download FirmWare */
                /* 1. determine IMEM code size and Load IMEM Code Section */
                imem_sz = fwhdr.img_IMEM_size;
                do {
-                       memset(ptx_desc, 0, TXDESC_SIZE);
+                       memset(txdesc, 0, TXDESC_SIZE);
                        if (imem_sz >  MAX_DUMP_FWSZ/*49152*/) {
                                dump_imem_sz = MAX_DUMP_FWSZ;
                        } else {
                                dump_imem_sz = imem_sz;
-                               ptx_desc->txdw0 |= cpu_to_le32(BIT(28));
+                               txdesc->txdw0 |= cpu_to_le32(BIT(28));
                        }
-                       ptx_desc->txdw0 |= cpu_to_le32(dump_imem_sz &
+                       txdesc->txdw0 |= cpu_to_le32(dump_imem_sz &
                                                       0x0000ffff);
-                       memcpy(ppayload, ptr, dump_imem_sz);
-                       r8712_write_mem(padapter, RTL8712_DMA_VOQ,
+                       memcpy(payload, ptr, dump_imem_sz);
+                       r8712_write_mem(adapter, RTL8712_DMA_VOQ,
                                        dump_imem_sz + TXDESC_SIZE,
-                                       (u8 *)ptx_desc);
+                                       (u8 *)txdesc);
                        ptr += dump_imem_sz;
                        imem_sz -= dump_imem_sz;
                } while (imem_sz > 0);
                i = 10;
-               tmp16 = r8712_read16(padapter, TCR);
+               tmp16 = r8712_read16(adapter, TCR);
                while (((tmp16 & _IMEM_CODE_DONE) == 0) && (i > 0)) {
                        usleep_range(10, 1000);
-                       tmp16 = r8712_read16(padapter, TCR);
+                       tmp16 = r8712_read16(adapter, TCR);
                        i--;
                }
                if (i == 0 || (tmp16 & _IMEM_CHK_RPT) == 0)
@@ -208,94 +208,94 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter)
                /* 2.Download EMEM code size and Load EMEM Code Section */
                emem_sz = fwhdr.img_SRAM_size;
                do {
-                       memset(ptx_desc, 0, TXDESC_SIZE);
+                       memset(txdesc, 0, TXDESC_SIZE);
                        if (emem_sz >  MAX_DUMP_FWSZ) { /* max=48k */
                                dump_emem_sz = MAX_DUMP_FWSZ;
                        } else {
                                dump_emem_sz = emem_sz;
-                               ptx_desc->txdw0 |= cpu_to_le32(BIT(28));
+                               txdesc->txdw0 |= cpu_to_le32(BIT(28));
                        }
-                       ptx_desc->txdw0 |= cpu_to_le32(dump_emem_sz &
+                       txdesc->txdw0 |= cpu_to_le32(dump_emem_sz &
                                                       0x0000ffff);
-                       memcpy(ppayload, ptr, dump_emem_sz);
-                       r8712_write_mem(padapter, RTL8712_DMA_VOQ,
+                       memcpy(payload, ptr, dump_emem_sz);
+                       r8712_write_mem(adapter, RTL8712_DMA_VOQ,
                                        dump_emem_sz + TXDESC_SIZE,
-                                       (u8 *)ptx_desc);
+                                       (u8 *)txdesc);
                        ptr += dump_emem_sz;
                        emem_sz -= dump_emem_sz;
                } while (emem_sz > 0);
                i = 5;
-               tmp16 = r8712_read16(padapter, TCR);
+               tmp16 = r8712_read16(adapter, TCR);
                while (((tmp16 & _EMEM_CODE_DONE) == 0) && (i > 0)) {
                        usleep_range(10, 1000);
-                       tmp16 = r8712_read16(padapter, TCR);
+                       tmp16 = r8712_read16(adapter, TCR);
                        i--;
                }
                if (i == 0 || (tmp16 & _EMEM_CHK_RPT) == 0)
                        goto exit_fail;
 
                /* 3.Enable CPU */
-               tmp8 = r8712_read8(padapter, SYS_CLKR);
-               r8712_write8(padapter, SYS_CLKR, tmp8 | BIT(2));
-               tmp8_a = r8712_read8(padapter, SYS_CLKR);
+               tmp8 = r8712_read8(adapter, SYS_CLKR);
+               r8712_write8(adapter, SYS_CLKR, tmp8 | BIT(2));
+               tmp8_a = r8712_read8(adapter, SYS_CLKR);
                if (tmp8_a != (tmp8 | BIT(2)))
                        goto exit_fail;
 
-               tmp8 = r8712_read8(padapter, SYS_FUNC_EN + 1);
-               r8712_write8(padapter, SYS_FUNC_EN + 1, tmp8 | BIT(2));
-               tmp8_a = r8712_read8(padapter, SYS_FUNC_EN + 1);
+               tmp8 = r8712_read8(adapter, SYS_FUNC_EN + 1);
+               r8712_write8(adapter, SYS_FUNC_EN + 1, tmp8 | BIT(2));
+               tmp8_a = r8712_read8(adapter, SYS_FUNC_EN + 1);
                if (tmp8_a != (tmp8 | BIT(2)))
                        goto exit_fail;
 
-               r8712_read32(padapter, TCR);
+               r8712_read32(adapter, TCR);
 
                /* 4.polling IMEM Ready */
                i = 100;
-               tmp16 = r8712_read16(padapter, TCR);
+               tmp16 = r8712_read16(adapter, TCR);
                while (((tmp16 & _IMEM_RDY) == 0) && (i > 0)) {
                        msleep(20);
-                       tmp16 = r8712_read16(padapter, TCR);
+                       tmp16 = r8712_read16(adapter, TCR);
                        i--;
                }
                if (i == 0) {
-                       r8712_write16(padapter, 0x10250348, 0xc000);
-                       r8712_write16(padapter, 0x10250348, 0xc001);
-                       r8712_write16(padapter, 0x10250348, 0x2000);
-                       r8712_write16(padapter, 0x10250348, 0x2001);
-                       r8712_write16(padapter, 0x10250348, 0x2002);
-                       r8712_write16(padapter, 0x10250348, 0x2003);
+                       r8712_write16(adapter, 0x10250348, 0xc000);
+                       r8712_write16(adapter, 0x10250348, 0xc001);
+                       r8712_write16(adapter, 0x10250348, 0x2000);
+                       r8712_write16(adapter, 0x10250348, 0x2001);
+                       r8712_write16(adapter, 0x10250348, 0x2002);
+                       r8712_write16(adapter, 0x10250348, 0x2003);
                        goto exit_fail;
                }
                /* 5.Download DMEM code size and Load EMEM Code Section */
-               memset(ptx_desc, 0, TXDESC_SIZE);
-               ptx_desc->txdw0 |= cpu_to_le32(fwhdr.fw_priv_sz & 0x0000ffff);
-               ptx_desc->txdw0 |= cpu_to_le32(BIT(28));
-               memcpy(ppayload, &fwhdr.fwpriv, fwhdr.fw_priv_sz);
-               r8712_write_mem(padapter, RTL8712_DMA_VOQ,
-                               fwhdr.fw_priv_sz + TXDESC_SIZE, (u8 *)ptx_desc);
+               memset(txdesc, 0, TXDESC_SIZE);
+               txdesc->txdw0 |= cpu_to_le32(fwhdr.fw_priv_sz & 0x0000ffff);
+               txdesc->txdw0 |= cpu_to_le32(BIT(28));
+               memcpy(payload, &fwhdr.fwpriv, fwhdr.fw_priv_sz);
+               r8712_write_mem(adapter, RTL8712_DMA_VOQ,
+                               fwhdr.fw_priv_sz + TXDESC_SIZE, (u8 *)txdesc);
 
                /* polling dmem code done */
                i = 100;
-               tmp16 = r8712_read16(padapter, TCR);
+               tmp16 = r8712_read16(adapter, TCR);
                while (((tmp16 & _DMEM_CODE_DONE) == 0) && (i > 0)) {
                        msleep(20);
-                       tmp16 = r8712_read16(padapter, TCR);
+                       tmp16 = r8712_read16(adapter, TCR);
                        i--;
                }
                if (i == 0)
                        goto exit_fail;
 
-               tmp8 = r8712_read8(padapter, 0x1025000A);
+               tmp8 = r8712_read8(adapter, 0x1025000A);
                if (tmp8 & BIT(4)) /* When boot from EEPROM,
                                    * & FW need more time to read EEPROM
                                    */
                        i = 60;
                else                    /* boot from EFUSE */
                        i = 30;
-               tmp16 = r8712_read16(padapter, TCR);
+               tmp16 = r8712_read16(adapter, TCR);
                while (((tmp16 & _FWRDY) == 0) && (i > 0)) {
                        msleep(100);
-                       tmp16 = r8712_read16(padapter, TCR);
+                       tmp16 = r8712_read16(adapter, TCR);
                        i--;
                }
                if (i == 0)
@@ -306,7 +306,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter)
        ret = _SUCCESS;
 
 exit_fail:
-       kfree(ptmpchar);
+       kfree(tmpchar);
        return ret;
 }