Commit 286570fb authored by Nishka Dasgupta's avatar Nishka Dasgupta Committed by Greg Kroah-Hartman

staging: rtl8712: hal_init.c: Remove leading p from variable names

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