staging: brcm80211: remove static global variables from dhd_sdio.c
Several global variables that were set but never initialized have been removed. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
22a911976f
commit
7dae3a638c
|
@ -761,26 +761,18 @@ module_param(brcmf_poll, uint, 0);
|
|||
uint brcmf_intr = true;
|
||||
module_param(brcmf_intr, uint, 0);
|
||||
|
||||
/* IOCTL response timeout */
|
||||
static int brcmf_ioctl_timeout_msec = IOCTL_RESP_TIMEOUT;
|
||||
|
||||
/* override the RAM size if possible */
|
||||
#define DONGLE_MIN_MEMSIZE (128 * 1024)
|
||||
int brcmf_dongle_memsize;
|
||||
module_param(brcmf_dongle_memsize, int, 0);
|
||||
|
||||
static bool brcmf_alignctl;
|
||||
|
||||
static bool retrydata;
|
||||
#define RETRYCHAN(chan) (((chan) == SDPCM_EVENT_CHANNEL) || retrydata)
|
||||
|
||||
static const uint firstread = BRCMF_FIRSTREAD;
|
||||
#define RETRYCHAN(chan) ((chan) == SDPCM_EVENT_CHANNEL)
|
||||
|
||||
/* Retry count for register access failures */
|
||||
static const uint retry_limit = 2;
|
||||
|
||||
/* Force even SD lengths (some host controllers mess up on odd bytes) */
|
||||
static bool forcealign;
|
||||
/* Limit on rounding up frames */
|
||||
static const uint max_roundup = 512;
|
||||
|
||||
#define ALIGNMENT 4
|
||||
|
||||
|
@ -794,12 +786,6 @@ static void pkt_align(struct sk_buff *p, int len, int align)
|
|||
__skb_trim(p, len);
|
||||
}
|
||||
|
||||
/* Limit on rounding up frames */
|
||||
static const uint max_roundup = 512;
|
||||
|
||||
/* Try doing readahead */
|
||||
static bool brcmf_readahead;
|
||||
|
||||
/* To check if there's window offered */
|
||||
static bool data_ok(struct brcmf_bus *bus)
|
||||
{
|
||||
|
@ -1697,7 +1683,7 @@ static int brcmf_sdbrcm_ioctl_resp_wait(struct brcmf_bus *bus, uint *condition,
|
|||
bool *pending)
|
||||
{
|
||||
DECLARE_WAITQUEUE(wait, current);
|
||||
int timeout = msecs_to_jiffies(brcmf_ioctl_timeout_msec);
|
||||
int timeout = msecs_to_jiffies(IOCTL_RESP_TIMEOUT);
|
||||
|
||||
/* Wait until control frame is available */
|
||||
add_wait_queue(&bus->ioctl_resp_wait, &wait);
|
||||
|
@ -1733,21 +1719,19 @@ brcmf_sdbrcm_read_control(struct brcmf_bus *bus, u8 *hdr, uint len, uint doff)
|
|||
|
||||
/* Set rxctl for frame (w/optional alignment) */
|
||||
bus->rxctl = bus->rxbuf;
|
||||
if (brcmf_alignctl) {
|
||||
bus->rxctl += firstread;
|
||||
pad = ((unsigned long)bus->rxctl % BRCMF_SDALIGN);
|
||||
if (pad)
|
||||
bus->rxctl += (BRCMF_SDALIGN - pad);
|
||||
bus->rxctl -= firstread;
|
||||
}
|
||||
bus->rxctl += BRCMF_FIRSTREAD;
|
||||
pad = ((unsigned long)bus->rxctl % BRCMF_SDALIGN);
|
||||
if (pad)
|
||||
bus->rxctl += (BRCMF_SDALIGN - pad);
|
||||
bus->rxctl -= BRCMF_FIRSTREAD;
|
||||
|
||||
/* Copy the already-read portion over */
|
||||
memcpy(bus->rxctl, hdr, firstread);
|
||||
if (len <= firstread)
|
||||
memcpy(bus->rxctl, hdr, BRCMF_FIRSTREAD);
|
||||
if (len <= BRCMF_FIRSTREAD)
|
||||
goto gotpkt;
|
||||
|
||||
/* Raise rdlen to next SDIO block to avoid tail command */
|
||||
rdlen = len - firstread;
|
||||
rdlen = len - BRCMF_FIRSTREAD;
|
||||
if (bus->roundup && bus->blocksize && (rdlen > bus->blocksize)) {
|
||||
pad = bus->blocksize - (rdlen % bus->blocksize);
|
||||
if ((pad <= bus->roundup) && (pad < bus->blocksize) &&
|
||||
|
@ -1758,11 +1742,11 @@ brcmf_sdbrcm_read_control(struct brcmf_bus *bus, u8 *hdr, uint len, uint doff)
|
|||
}
|
||||
|
||||
/* Satisfy length-alignment requirements */
|
||||
if (forcealign && (rdlen & (ALIGNMENT - 1)))
|
||||
if (rdlen & (ALIGNMENT - 1))
|
||||
rdlen = roundup(rdlen, ALIGNMENT);
|
||||
|
||||
/* Drop if the read is too big or it exceeds our maximum */
|
||||
if ((rdlen + firstread) > bus->drvr->maxctl) {
|
||||
if ((rdlen + BRCMF_FIRSTREAD) > bus->drvr->maxctl) {
|
||||
brcmf_dbg(ERROR, "%d-byte control read exceeds %d-byte buffer\n",
|
||||
rdlen, bus->drvr->maxctl);
|
||||
bus->drvr->rx_errors++;
|
||||
|
@ -1783,7 +1767,7 @@ brcmf_sdbrcm_read_control(struct brcmf_bus *bus, u8 *hdr, uint len, uint doff)
|
|||
sdret = brcmf_sdcard_recv_buf(bus->sdiodev,
|
||||
bus->sdiodev->sbwad,
|
||||
SDIO_FUNC_2,
|
||||
F2SYNC, (bus->rxctl + firstread), rdlen,
|
||||
F2SYNC, (bus->rxctl + BRCMF_FIRSTREAD), rdlen,
|
||||
NULL);
|
||||
bus->f2rxdata++;
|
||||
|
||||
|
@ -1857,7 +1841,7 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
}
|
||||
|
||||
/* Try doing single read if we can */
|
||||
if (brcmf_readahead && bus->nextlen) {
|
||||
if (bus->nextlen) {
|
||||
u16 nextlen = bus->nextlen;
|
||||
bus->nextlen = 0;
|
||||
|
||||
|
@ -1871,7 +1855,7 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
(rdlen % bus->blocksize);
|
||||
if ((pad <= bus->roundup)
|
||||
&& (pad < bus->blocksize)
|
||||
&& ((rdlen + pad + firstread) <
|
||||
&& ((rdlen + pad + BRCMF_FIRSTREAD) <
|
||||
MAX_RX_DATASZ))
|
||||
rdlen += pad;
|
||||
} else if (rdlen % BRCMF_SDALIGN) {
|
||||
|
@ -2055,8 +2039,8 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
|
||||
/* Read frame header (hardware and software) */
|
||||
sdret = brcmf_sdcard_recv_buf(bus->sdiodev, bus->sdiodev->sbwad,
|
||||
SDIO_FUNC_2, F2SYNC, bus->rxhdr, firstread,
|
||||
NULL);
|
||||
SDIO_FUNC_2, F2SYNC, bus->rxhdr,
|
||||
BRCMF_FIRSTREAD, NULL);
|
||||
bus->f2rxhdrs++;
|
||||
|
||||
if (sdret < 0) {
|
||||
|
@ -2162,24 +2146,24 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
SDPCM_GLOM_CHANNEL */
|
||||
|
||||
/* Length to read */
|
||||
rdlen = (len > firstread) ? (len - firstread) : 0;
|
||||
rdlen = (len > BRCMF_FIRSTREAD) ? (len - BRCMF_FIRSTREAD) : 0;
|
||||
|
||||
/* May pad read to blocksize for efficiency */
|
||||
if (bus->roundup && bus->blocksize &&
|
||||
(rdlen > bus->blocksize)) {
|
||||
pad = bus->blocksize - (rdlen % bus->blocksize);
|
||||
if ((pad <= bus->roundup) && (pad < bus->blocksize) &&
|
||||
((rdlen + pad + firstread) < MAX_RX_DATASZ))
|
||||
((rdlen + pad + BRCMF_FIRSTREAD) < MAX_RX_DATASZ))
|
||||
rdlen += pad;
|
||||
} else if (rdlen % BRCMF_SDALIGN) {
|
||||
rdlen += BRCMF_SDALIGN - (rdlen % BRCMF_SDALIGN);
|
||||
}
|
||||
|
||||
/* Satisfy length-alignment requirements */
|
||||
if (forcealign && (rdlen & (ALIGNMENT - 1)))
|
||||
if (rdlen & (ALIGNMENT - 1))
|
||||
rdlen = roundup(rdlen, ALIGNMENT);
|
||||
|
||||
if ((rdlen + firstread) > MAX_RX_DATASZ) {
|
||||
if ((rdlen + BRCMF_FIRSTREAD) > MAX_RX_DATASZ) {
|
||||
/* Too long -- skip this frame */
|
||||
brcmf_dbg(ERROR, "too long: len %d rdlen %d\n",
|
||||
len, rdlen);
|
||||
|
@ -2189,7 +2173,8 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
continue;
|
||||
}
|
||||
|
||||
pkt = brcmu_pkt_buf_get_skb(rdlen + firstread + BRCMF_SDALIGN);
|
||||
pkt = brcmu_pkt_buf_get_skb(rdlen +
|
||||
BRCMF_FIRSTREAD + BRCMF_SDALIGN);
|
||||
if (!pkt) {
|
||||
/* Give up on data, request rtx of events */
|
||||
brcmf_dbg(ERROR, "brcmu_pkt_buf_get_skb failed: rdlen %d chan %d\n",
|
||||
|
@ -2200,7 +2185,7 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
}
|
||||
|
||||
/* Leave room for what we already read, and align remainder */
|
||||
skb_pull(pkt, firstread);
|
||||
skb_pull(pkt, BRCMF_FIRSTREAD);
|
||||
pkt_align(pkt, rdlen, BRCMF_SDALIGN);
|
||||
|
||||
/* Read the remaining frame data */
|
||||
|
@ -2221,8 +2206,8 @@ brcmf_sdbrcm_readframes(struct brcmf_bus *bus, uint maxframes, bool *finished)
|
|||
}
|
||||
|
||||
/* Copy the already-read portion */
|
||||
skb_push(pkt, firstread);
|
||||
memcpy(pkt->data, bus->rxhdr, firstread);
|
||||
skb_push(pkt, BRCMF_FIRSTREAD);
|
||||
memcpy(pkt->data, bus->rxhdr, BRCMF_FIRSTREAD);
|
||||
|
||||
#ifdef BCMDBG
|
||||
if (BRCMF_BYTES_ON() && BRCMF_DATA_ON()) {
|
||||
|
@ -2328,7 +2313,6 @@ static int brcmf_sdbrcm_txpkt(struct brcmf_bus *bus, struct sk_buff *pkt,
|
|||
u8 *frame;
|
||||
u16 len, pad = 0;
|
||||
u32 swheader;
|
||||
uint retries = 0;
|
||||
struct sk_buff *new;
|
||||
int i;
|
||||
|
||||
|
@ -2413,48 +2397,44 @@ static int brcmf_sdbrcm_txpkt(struct brcmf_bus *bus, struct sk_buff *pkt,
|
|||
}
|
||||
|
||||
/* Some controllers have trouble with odd bytes -- round to even */
|
||||
if (forcealign && (len & (ALIGNMENT - 1)))
|
||||
if (len & (ALIGNMENT - 1))
|
||||
len = roundup(len, ALIGNMENT);
|
||||
|
||||
do {
|
||||
ret = brcmf_sdbrcm_send_buf(bus, bus->sdiodev->sbwad,
|
||||
SDIO_FUNC_2, F2SYNC, frame,
|
||||
len, pkt);
|
||||
bus->f2txdata++;
|
||||
ret = brcmf_sdbrcm_send_buf(bus, bus->sdiodev->sbwad,
|
||||
SDIO_FUNC_2, F2SYNC, frame,
|
||||
len, pkt);
|
||||
bus->f2txdata++;
|
||||
|
||||
if (ret < 0) {
|
||||
/* On failure, abort the command
|
||||
and terminate the frame */
|
||||
brcmf_dbg(INFO, "sdio error %d, abort command and terminate frame\n",
|
||||
ret);
|
||||
bus->tx_sderrs++;
|
||||
if (ret < 0) {
|
||||
/* On failure, abort the command and terminate the frame */
|
||||
brcmf_dbg(INFO, "sdio error %d, abort command and terminate frame\n",
|
||||
ret);
|
||||
bus->tx_sderrs++;
|
||||
|
||||
brcmf_sdcard_abort(bus->sdiodev, SDIO_FUNC_2);
|
||||
brcmf_sdcard_cfg_write(bus->sdiodev, SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_FRAMECTRL, SFC_WF_TERM,
|
||||
NULL);
|
||||
bus->f1regdata++;
|
||||
|
||||
for (i = 0; i < 3; i++) {
|
||||
u8 hi, lo;
|
||||
hi = brcmf_sdcard_cfg_read(bus->sdiodev,
|
||||
SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_WFRAMEBCHI,
|
||||
NULL);
|
||||
lo = brcmf_sdcard_cfg_read(bus->sdiodev,
|
||||
SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_WFRAMEBCLO,
|
||||
NULL);
|
||||
bus->f1regdata += 2;
|
||||
if ((hi == 0) && (lo == 0))
|
||||
break;
|
||||
}
|
||||
brcmf_sdcard_abort(bus->sdiodev, SDIO_FUNC_2);
|
||||
brcmf_sdcard_cfg_write(bus->sdiodev, SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_FRAMECTRL, SFC_WF_TERM,
|
||||
NULL);
|
||||
bus->f1regdata++;
|
||||
|
||||
for (i = 0; i < 3; i++) {
|
||||
u8 hi, lo;
|
||||
hi = brcmf_sdcard_cfg_read(bus->sdiodev,
|
||||
SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_WFRAMEBCHI,
|
||||
NULL);
|
||||
lo = brcmf_sdcard_cfg_read(bus->sdiodev,
|
||||
SDIO_FUNC_1,
|
||||
SBSDIO_FUNC1_WFRAMEBCLO,
|
||||
NULL);
|
||||
bus->f1regdata += 2;
|
||||
if ((hi == 0) && (lo == 0))
|
||||
break;
|
||||
}
|
||||
if (ret == 0)
|
||||
bus->tx_seq = (bus->tx_seq + 1) % SDPCM_SEQUENCE_WRAP;
|
||||
|
||||
} while ((ret < 0) && retrydata && retries++ < TXRETRIES);
|
||||
}
|
||||
if (ret == 0)
|
||||
bus->tx_seq = (bus->tx_seq + 1) % SDPCM_SEQUENCE_WRAP;
|
||||
|
||||
done:
|
||||
/* restore pkt buffer pointer before calling tx complete routine */
|
||||
|
@ -3259,16 +3239,14 @@ brcmf_sdbrcm_bus_txctl(struct brcmf_bus *bus, unsigned char *msg, uint msglen)
|
|||
len = (msglen += SDPCM_HDRLEN);
|
||||
|
||||
/* Add alignment padding (optional for ctl frames) */
|
||||
if (brcmf_alignctl) {
|
||||
doff = ((unsigned long)frame % BRCMF_SDALIGN);
|
||||
if (doff) {
|
||||
frame -= doff;
|
||||
len += doff;
|
||||
msglen += doff;
|
||||
memset(frame, 0, doff + SDPCM_HDRLEN);
|
||||
}
|
||||
/* precondition: doff < BRCMF_SDALIGN */
|
||||
doff = ((unsigned long)frame % BRCMF_SDALIGN);
|
||||
if (doff) {
|
||||
frame -= doff;
|
||||
len += doff;
|
||||
msglen += doff;
|
||||
memset(frame, 0, doff + SDPCM_HDRLEN);
|
||||
}
|
||||
/* precondition: doff < BRCMF_SDALIGN */
|
||||
doff += SDPCM_HDRLEN;
|
||||
|
||||
/* Round send length to next SDIO block */
|
||||
|
@ -3281,7 +3259,7 @@ brcmf_sdbrcm_bus_txctl(struct brcmf_bus *bus, unsigned char *msg, uint msglen)
|
|||
}
|
||||
|
||||
/* Satisfy length-alignment requirements */
|
||||
if (forcealign && (len & (ALIGNMENT - 1)))
|
||||
if (len & (ALIGNMENT - 1))
|
||||
len = roundup(len, ALIGNMENT);
|
||||
|
||||
/* precondition: IS_ALIGNED((unsigned long)frame, 2) */
|
||||
|
@ -4825,14 +4803,9 @@ void *brcmf_sdbrcm_probe(u16 bus_no, u16 slot, u16 func, uint bustype,
|
|||
*/
|
||||
brcmf_txbound = BRCMF_TXBOUND;
|
||||
brcmf_rxbound = BRCMF_RXBOUND;
|
||||
brcmf_alignctl = true;
|
||||
brcmf_readahead = true;
|
||||
retrydata = false;
|
||||
brcmf_dongle_memsize = 0;
|
||||
brcmf_txminmax = BRCMF_TXMINMAX;
|
||||
|
||||
forcealign = true;
|
||||
|
||||
brcmf_c_init();
|
||||
|
||||
brcmf_dbg(TRACE, "Enter\n");
|
||||
|
|
Loading…
Reference in New Issue