Commit 6d6163c3 authored by Jonathan Jin's avatar Jonathan Jin Committed by Greg Kroah-Hartman

staging: rtl8192e: rejoin split quoted strings

Fix a checkpatch.pl warning regarding quoted string splits across lines.
While each join of these quoted strings results in a new checkpatch.pl
"lines over 80 characters" warning, the regained ability to grep for
these log strings in the codebase is, I would argue, well worth the
trade-off.
Signed-off-by: default avatarJonathan Jin <jjin082693@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7aa36b4b
...@@ -70,8 +70,7 @@ rtllib_frag_cache_find(struct rtllib_device *ieee, unsigned int seq, ...@@ -70,8 +70,7 @@ rtllib_frag_cache_find(struct rtllib_device *ieee, unsigned int seq,
if (entry->skb != NULL && if (entry->skb != NULL &&
time_after(jiffies, entry->first_frag_time + 2 * HZ)) { time_after(jiffies, entry->first_frag_time + 2 * HZ)) {
RTLLIB_DEBUG_FRAG( RTLLIB_DEBUG_FRAG(
"expiring fragment cache entry " "expiring fragment cache entry seq=%u last_frag=%u\n",
"seq=%u last_frag=%u\n",
entry->seq, entry->last_frag); entry->seq, entry->last_frag);
dev_kfree_skb_any(entry->skb); dev_kfree_skb_any(entry->skb);
entry->skb = NULL; entry->skb = NULL;
...@@ -188,8 +187,7 @@ static int rtllib_frag_cache_invalidate(struct rtllib_device *ieee, ...@@ -188,8 +187,7 @@ static int rtllib_frag_cache_invalidate(struct rtllib_device *ieee,
if (entry == NULL) { if (entry == NULL) {
RTLLIB_DEBUG_FRAG( RTLLIB_DEBUG_FRAG(
"could not invalidate fragment cache " "could not invalidate fragment cache entry (seq=%u)\n", seq);
"entry (seq=%u)\n", seq);
return -1; return -1;
} }
...@@ -305,11 +303,9 @@ rtllib_rx_frame_decrypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -305,11 +303,9 @@ rtllib_rx_frame_decrypt(struct rtllib_device *ieee, struct sk_buff *skb,
atomic_dec(&crypt->refcnt); atomic_dec(&crypt->refcnt);
if (res < 0) { if (res < 0) {
RTLLIB_DEBUG_DROP( RTLLIB_DEBUG_DROP(
"decryption failed (SA= %pM" "decryption failed (SA= %pM) res=%d\n", hdr->addr2, res);
") res=%d\n", hdr->addr2, res);
if (res == -2) if (res == -2)
RTLLIB_DEBUG_DROP("Decryption failed ICV " RTLLIB_DEBUG_DROP("Decryption failed ICV mismatch (key %d)\n",
"mismatch (key %d)\n",
skb->data[hdrlen + 3] >> 6); skb->data[hdrlen + 3] >> 6);
ieee->ieee_stats.rx_discards_undecryptable++; ieee->ieee_stats.rx_discards_undecryptable++;
return -1; return -1;
...@@ -345,8 +341,7 @@ rtllib_rx_frame_decrypt_msdu(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -345,8 +341,7 @@ rtllib_rx_frame_decrypt_msdu(struct rtllib_device *ieee, struct sk_buff *skb,
res = crypt->ops->decrypt_msdu(skb, keyidx, hdrlen, crypt->priv); res = crypt->ops->decrypt_msdu(skb, keyidx, hdrlen, crypt->priv);
atomic_dec(&crypt->refcnt); atomic_dec(&crypt->refcnt);
if (res < 0) { if (res < 0) {
printk(KERN_DEBUG "%s: MSDU decryption/MIC verification failed" printk(KERN_DEBUG "%s: MSDU decryption/MIC verification failed (SA= %pM keyidx=%d)\n",
" (SA= %pM keyidx=%d)\n",
ieee->dev->name, hdr->addr2, keyidx); ieee->dev->name, hdr->addr2, keyidx);
return -1; return -1;
} }
...@@ -559,8 +554,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -559,8 +554,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
bool bMatchWinStart = false, bPktInBuf = false; bool bMatchWinStart = false, bPktInBuf = false;
unsigned long flags; unsigned long flags;
RTLLIB_DEBUG(RTLLIB_DL_REORDER, "%s(): Seq is %d, pTS->RxIndicateSeq" RTLLIB_DEBUG(RTLLIB_DL_REORDER, "%s(): Seq is %d, pTS->RxIndicateSeq is %d, WinSize is %d\n", __func__, SeqNum,
" is %d, WinSize is %d\n", __func__, SeqNum,
pTS->RxIndicateSeq, WinSize); pTS->RxIndicateSeq, WinSize);
spin_lock_irqsave(&(ieee->reorder_spinlock), flags); spin_lock_irqsave(&(ieee->reorder_spinlock), flags);
...@@ -600,8 +594,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -600,8 +594,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
pTS->RxIndicateSeq = SeqNum + 1 - WinSize; pTS->RxIndicateSeq = SeqNum + 1 - WinSize;
else else
pTS->RxIndicateSeq = 4095 - (WinSize - (SeqNum + 1)) + 1; pTS->RxIndicateSeq = 4095 - (WinSize - (SeqNum + 1)) + 1;
RTLLIB_DEBUG(RTLLIB_DL_REORDER, "Window Shift! IndicateSeq: %d," RTLLIB_DEBUG(RTLLIB_DL_REORDER, "Window Shift! IndicateSeq: %d, NewSeq: %d\n", pTS->RxIndicateSeq, SeqNum);
" NewSeq: %d\n", pTS->RxIndicateSeq, SeqNum);
} }
/* /*
...@@ -617,8 +610,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -617,8 +610,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
*/ */
if (bMatchWinStart) { if (bMatchWinStart) {
/* Current packet is going to be indicated.*/ /* Current packet is going to be indicated.*/
RTLLIB_DEBUG(RTLLIB_DL_REORDER, "Packets indication!! " RTLLIB_DEBUG(RTLLIB_DL_REORDER, "Packets indication!! IndicateSeq: %d, NewSeq: %d\n",
"IndicateSeq: %d, NewSeq: %d\n",
pTS->RxIndicateSeq, SeqNum); pTS->RxIndicateSeq, SeqNum);
ieee->prxbIndicateArray[0] = prxb; ieee->prxbIndicateArray[0] = prxb;
index = 1; index = 1;
...@@ -636,9 +628,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -636,9 +628,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
if (!AddReorderEntry(pTS, pReorderEntry)) { if (!AddReorderEntry(pTS, pReorderEntry)) {
RTLLIB_DEBUG(RTLLIB_DL_REORDER, RTLLIB_DEBUG(RTLLIB_DL_REORDER,
"%s(): Duplicate packet is " "%s(): Duplicate packet is dropped!! IndicateSeq: %d, NewSeq: %d\n",
"dropped!! IndicateSeq: %d, "
"NewSeq: %d\n",
__func__, pTS->RxIndicateSeq, __func__, pTS->RxIndicateSeq,
SeqNum); SeqNum);
list_add_tail(&pReorderEntry->List, list_add_tail(&pReorderEntry->List,
...@@ -652,8 +642,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -652,8 +642,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
} }
} else { } else {
RTLLIB_DEBUG(RTLLIB_DL_REORDER, RTLLIB_DEBUG(RTLLIB_DL_REORDER,
"Pkt insert into struct buffer!! " "Pkt insert into struct buffer!! IndicateSeq: %d, NewSeq: %d\n",
"IndicateSeq: %d, NewSeq: %d\n",
pTS->RxIndicateSeq, SeqNum); pTS->RxIndicateSeq, SeqNum);
} }
} else { } else {
...@@ -663,9 +652,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -663,9 +652,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
* indicate all the packets in struct buffer and get * indicate all the packets in struct buffer and get
* reorder entries. * reorder entries.
*/ */
RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicatePacket():" RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicatePacket(): There is no reorder entry!! Packet is dropped!!\n");
" There is no reorder entry!! Packet is "
"dropped!!\n");
{ {
int i; int i;
...@@ -687,8 +674,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -687,8 +674,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
SN_EQUAL(pReorderEntry->SeqNum, pTS->RxIndicateSeq)) { SN_EQUAL(pReorderEntry->SeqNum, pTS->RxIndicateSeq)) {
/* This protect struct buffer from overflow. */ /* This protect struct buffer from overflow. */
if (index >= REORDER_WIN_SIZE) { if (index >= REORDER_WIN_SIZE) {
RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicate" RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicatePacket(): Buffer overflow!!\n");
"Packet(): Buffer overflow!!\n");
bPktInBuf = true; bPktInBuf = true;
break; break;
} }
...@@ -699,8 +685,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -699,8 +685,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
pTS->RxIndicateSeq = (pTS->RxIndicateSeq + 1) % 4096; pTS->RxIndicateSeq = (pTS->RxIndicateSeq + 1) % 4096;
ieee->prxbIndicateArray[index] = pReorderEntry->prxb; ieee->prxbIndicateArray[index] = pReorderEntry->prxb;
RTLLIB_DEBUG(RTLLIB_DL_REORDER, "%s(): Indicate SeqNum" RTLLIB_DEBUG(RTLLIB_DL_REORDER, "%s(): Indicate SeqNum %d!\n", __func__, pReorderEntry->SeqNum);
" %d!\n", __func__, pReorderEntry->SeqNum);
index++; index++;
list_add_tail(&pReorderEntry->List, list_add_tail(&pReorderEntry->List,
...@@ -719,8 +704,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee, ...@@ -719,8 +704,7 @@ static void RxReorderIndicatePacket(struct rtllib_device *ieee,
pTS->RxTimeoutIndicateSeq = 0xffff; pTS->RxTimeoutIndicateSeq = 0xffff;
if (index > REORDER_WIN_SIZE) { if (index > REORDER_WIN_SIZE) {
RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicatePacket():" RTLLIB_DEBUG(RTLLIB_DL_ERR, "RxReorderIndicatePacket(): Rx Reorder struct buffer full!!\n");
" Rx Reorder struct buffer full!!\n");
spin_unlock_irqrestore(&(ieee->reorder_spinlock), spin_unlock_irqrestore(&(ieee->reorder_spinlock),
flags); flags);
return; return;
...@@ -809,14 +793,11 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -809,14 +793,11 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb,
(nSubframe_Length << 8); (nSubframe_Length << 8);
if (skb->len < (ETHERNET_HEADER_SIZE + nSubframe_Length)) { if (skb->len < (ETHERNET_HEADER_SIZE + nSubframe_Length)) {
printk(KERN_INFO "%s: A-MSDU parse error!! " printk(KERN_INFO "%s: A-MSDU parse error!! pRfd->nTotalSubframe : %d\n",\
"pRfd->nTotalSubframe : %d\n",\
__func__, rxb->nr_subframes); __func__, rxb->nr_subframes);
printk(KERN_INFO "%s: A-MSDU parse error!! " printk(KERN_INFO "%s: A-MSDU parse error!! Subframe Length: %d\n", __func__,
"Subframe Length: %d\n", __func__,
nSubframe_Length); nSubframe_Length);
printk(KERN_INFO "nRemain_Length is %d and " printk(KERN_INFO "nRemain_Length is %d and nSubframe_Length is : %d\n", skb->len,
"nSubframe_Length is : %d\n", skb->len,
nSubframe_Length); nSubframe_Length);
printk(KERN_INFO "The Packet SeqNum is %d\n", SeqNum); printk(KERN_INFO "The Packet SeqNum is %d\n", SeqNum);
return 0; return 0;
...@@ -844,8 +825,7 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -844,8 +825,7 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb,
sub_skb->dev = ieee->dev; sub_skb->dev = ieee->dev;
rxb->subframes[rxb->nr_subframes++] = sub_skb; rxb->subframes[rxb->nr_subframes++] = sub_skb;
if (rxb->nr_subframes >= MAX_SUBFRAME_COUNT) { if (rxb->nr_subframes >= MAX_SUBFRAME_COUNT) {
RTLLIB_DEBUG_RX("ParseSubframe(): Too many " RTLLIB_DEBUG_RX("ParseSubframe(): Too many Subframes! Packets dropped!\n");
"Subframes! Packets dropped!\n");
break; break;
} }
skb_pull(skb, nSubframe_Length); skb_pull(skb, nSubframe_Length);
...@@ -922,8 +902,7 @@ static int rtllib_rx_check_duplicate(struct rtllib_device *ieee, ...@@ -922,8 +902,7 @@ static int rtllib_rx_check_duplicate(struct rtllib_device *ieee,
pRxTS->RxLastFragNum = frag; pRxTS->RxLastFragNum = frag;
pRxTS->RxLastSeqNum = WLAN_GET_SEQ_SEQ(sc); pRxTS->RxLastSeqNum = WLAN_GET_SEQ_SEQ(sc);
} else { } else {
RTLLIB_DEBUG(RTLLIB_DL_ERR, "ERR!!%s(): No TS!! Skip" RTLLIB_DEBUG(RTLLIB_DL_ERR, "ERR!!%s(): No TS!! Skip the check!!\n", __func__);
" the check!!\n", __func__);
return -1; return -1;
} }
} }
...@@ -996,9 +975,7 @@ static int rtllib_rx_data_filter(struct rtllib_device *ieee, u16 fc, ...@@ -996,9 +975,7 @@ static int rtllib_rx_data_filter(struct rtllib_device *ieee, u16 fc,
stype != RTLLIB_STYPE_QOS_DATA) { stype != RTLLIB_STYPE_QOS_DATA) {
if (stype != RTLLIB_STYPE_NULLFUNC) if (stype != RTLLIB_STYPE_NULLFUNC)
RTLLIB_DEBUG_DROP( RTLLIB_DEBUG_DROP(
"RX: dropped data frame " "RX: dropped data frame with no data (type=0x%02x, subtype=0x%02x)\n",
"with no data (type=0x%02x, "
"subtype=0x%02x)\n",
type, stype); type, stype);
return -1; return -1;
} }
...@@ -1041,8 +1018,7 @@ static int rtllib_rx_get_crypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -1041,8 +1018,7 @@ static int rtllib_rx_get_crypt(struct rtllib_device *ieee, struct sk_buff *skb,
* frames from other than current BSS, so just drop the * frames from other than current BSS, so just drop the
* frames silently instead of filling system log with * frames silently instead of filling system log with
* these reports. */ * these reports. */
RTLLIB_DEBUG_DROP("Decryption failed (not set)" RTLLIB_DEBUG_DROP("Decryption failed (not set) (SA= %pM)\n",
" (SA= %pM)\n",
hdr->addr2); hdr->addr2);
ieee->ieee_stats.rx_discards_undecryptable++; ieee->ieee_stats.rx_discards_undecryptable++;
return -1; return -1;
...@@ -1086,8 +1062,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -1086,8 +1062,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb,
if (!frag_skb) { if (!frag_skb) {
RTLLIB_DEBUG(RTLLIB_DL_RX | RTLLIB_DL_FRAG, RTLLIB_DEBUG(RTLLIB_DL_RX | RTLLIB_DL_FRAG,
"Rx cannot get skb from fragment " "Rx cannot get skb from fragment cache (morefrag=%d seq=%u frag=%u)\n",
"cache (morefrag=%d seq=%u frag=%u)\n",
(fc & RTLLIB_FCTL_MOREFRAGS) != 0, (fc & RTLLIB_FCTL_MOREFRAGS) != 0,
WLAN_GET_SEQ_SEQ(sc), frag); WLAN_GET_SEQ_SEQ(sc), frag);
return -1; return -1;
...@@ -1097,8 +1072,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -1097,8 +1072,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb,
flen -= hdrlen; flen -= hdrlen;
if (frag_skb->tail + flen > frag_skb->end) { if (frag_skb->tail + flen > frag_skb->end) {
printk(KERN_WARNING "%s: host decrypted and " printk(KERN_WARNING "%s: host decrypted and reassembled frame did not fit skb\n",
"reassembled frame did not fit skb\n",
__func__); __func__);
rtllib_frag_cache_invalidate(ieee, hdr); rtllib_frag_cache_invalidate(ieee, hdr);
return -1; return -1;
...@@ -1152,8 +1126,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -1152,8 +1126,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb,
eap_get_type(eap->type)); eap_get_type(eap->type));
} else { } else {
RTLLIB_DEBUG_DROP( RTLLIB_DEBUG_DROP(
"encryption configured, but RX " "encryption configured, but RX frame not encrypted (SA= %pM)\n",
"frame not encrypted (SA= %pM)\n",
hdr->addr2); hdr->addr2);
return -1; return -1;
} }
...@@ -1170,9 +1143,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb, ...@@ -1170,9 +1143,7 @@ static int rtllib_rx_decrypt(struct rtllib_device *ieee, struct sk_buff *skb,
if (crypt && !(fc & RTLLIB_FCTL_WEP) && !ieee->open_wep && if (crypt && !(fc & RTLLIB_FCTL_WEP) && !ieee->open_wep &&
!rtllib_is_eapol_frame(ieee, skb, hdrlen)) { !rtllib_is_eapol_frame(ieee, skb, hdrlen)) {
RTLLIB_DEBUG_DROP( RTLLIB_DEBUG_DROP(
"dropped unencrypted RX data " "dropped unencrypted RX data frame from %pM (drop_unencrypted=1)\n",
"frame from %pM"
" (drop_unencrypted=1)\n",
hdr->addr2); hdr->addr2);
return -1; return -1;
} }
...@@ -1762,9 +1733,7 @@ int rtllib_parse_info_param(struct rtllib_device *ieee, ...@@ -1762,9 +1733,7 @@ int rtllib_parse_info_param(struct rtllib_device *ieee,
while (length >= sizeof(*info_element)) { while (length >= sizeof(*info_element)) {
if (sizeof(*info_element) + info_element->len > length) { if (sizeof(*info_element) + info_element->len > length) {
RTLLIB_DEBUG_MGMT("Info elem: parse failed: " RTLLIB_DEBUG_MGMT("Info elem: parse failed: info_element->len + 2 > left : info_element->len+2=%zd left=%d, id=%d.\n",
"info_element->len + 2 > left : "
"info_element->len+2=%zd left=%d, id=%d.\n",
info_element->len + info_element->len +
sizeof(*info_element), sizeof(*info_element),
length, info_element->id); length, info_element->id);
...@@ -2321,8 +2290,7 @@ static inline int rtllib_network_init( ...@@ -2321,8 +2290,7 @@ static inline int rtllib_network_init(
} }
if (network->mode == 0) { if (network->mode == 0) {
RTLLIB_DEBUG_SCAN("Filtered out '%s (%pM)' " RTLLIB_DEBUG_SCAN("Filtered out '%s (%pM)' network.\n",
"network.\n",
escape_essid(network->ssid, escape_essid(network->ssid,
network->ssid_len), network->ssid_len),
network->bssid); network->bssid);
...@@ -2568,8 +2536,7 @@ static inline void rtllib_process_probe_response( ...@@ -2568,8 +2536,7 @@ static inline void rtllib_process_probe_response(
if (WLAN_FC_GET_STYPE(le16_to_cpu(beacon->header.frame_ctl)) == if (WLAN_FC_GET_STYPE(le16_to_cpu(beacon->header.frame_ctl)) ==
RTLLIB_STYPE_PROBE_RESP) { RTLLIB_STYPE_PROBE_RESP) {
if (IsPassiveChannel(ieee, network->channel)) { if (IsPassiveChannel(ieee, network->channel)) {
printk(KERN_INFO "GetScanInfo(): For Global Domain, " printk(KERN_INFO "GetScanInfo(): For Global Domain, filter probe response at channel(%d).\n",
"filter probe response at channel(%d).\n",
network->channel); network->channel);
goto free_network; goto free_network;
} }
...@@ -2618,8 +2585,7 @@ static inline void rtllib_process_probe_response( ...@@ -2618,8 +2585,7 @@ static inline void rtllib_process_probe_response(
/* If there are no more slots, expire the oldest */ /* If there are no more slots, expire the oldest */
list_del(&oldest->list); list_del(&oldest->list);
target = oldest; target = oldest;
RTLLIB_DEBUG_SCAN("Expired '%s' ( %pM) from " RTLLIB_DEBUG_SCAN("Expired '%s' ( %pM) from network list.\n",
"network list.\n",
escape_essid(target->ssid, escape_essid(target->ssid,
target->ssid_len), target->ssid_len),
target->bssid); target->bssid);
......
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