* [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk
@ 2015-03-06 14:18 Hatice ERTÜRK
2015-03-06 15:40 ` [Outreachy kernel] " Julia Lawall
2015-03-16 14:50 ` Greg KH
0 siblings, 2 replies; 3+ messages in thread
From: Hatice ERTÜRK @ 2015-03-06 14:18 UTC (permalink / raw)
To: outreachy-kernel; +Cc: Hatice ERTURK
This patch fixes the following warning using checkpatch.pl
WARNING: Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ...
Signed-off-by: Hatice ERTURK <haticeerturk27@gmail.com>
---
v4:
Changed patch number.
---
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
index ecd8542..8d4ade4 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
@@ -197,7 +197,7 @@ int ieee80211_encrypt_fragment(
struct ieee80211_hdr_3addrqos *header;
header = (struct ieee80211_hdr_3addrqos *)frag->data;
- dev_debug(KERN_DEBUG "%s: TKIP countermeasures: dropped "
+ dev_debug("%s: TKIP countermeasures: dropped "
"TX packet to %pM\n",
ieee->dev->name, header->addr1);
}
@@ -219,7 +219,7 @@ int ieee80211_encrypt_fragment(
atomic_dec(&crypt->refcnt);
if (res < 0) {
- dev_info(KERN_INFO "%s: Encryption failed: len=%d.\n",
+ dev_info("%s: Encryption failed: len=%d.\n",
ieee->dev->name, frag->len);
ieee->ieee_stats.tx_discards++;
return -1;
@@ -623,7 +623,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
* creating it... */
if ((!ieee->hard_start_xmit && !(ieee->softmac_features & IEEE_SOFTMAC_TX_QUEUE))||
((!ieee->softmac_data_hard_start_xmit && (ieee->softmac_features & IEEE_SOFTMAC_TX_QUEUE)))) {
- dev_warn(KERN_WARNING "%s: No xmit handler.\n",
+ dev_warn("%s: No xmit handler.\n",
ieee->dev->name);
goto success;
}
@@ -631,7 +631,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
if(likely(ieee->raw_tx == 0)){
if (unlikely(skb->len < SNAP_SIZE + sizeof(u16))) {
- dev_warn(KERN_WARNING "%s: skb too small (%d).\n",
+ dev_warn("%s: skb too small (%d).\n",
ieee->dev->name, skb->len);
goto success;
}
@@ -747,7 +747,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
* postfix, header, FCS, etc.) */
txb = ieee80211_alloc_txb(nr_frags, frag_size + ieee->tx_headroom, GFP_ATOMIC);
if (unlikely(!txb)) {
- dev_warn(KERN_WARNING "%s: Could not allocate TXB\n",
+ dev_warn("%s: Could not allocate TXB\n",
ieee->dev->name);
goto failed;
}
@@ -846,14 +846,14 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
}
}else{
if (unlikely(skb->len < sizeof(struct ieee80211_hdr_3addr))) {
- dev_warn(KERN_WARNING "%s: skb too small (%d).\n",
+ dev_warn("%s: skb too small (%d).\n",
ieee->dev->name, skb->len);
goto success;
}
txb = ieee80211_alloc_txb(1, skb->len, GFP_ATOMIC);
if(!txb){
- dev_warn(KERN_WARNING "%s: Could not allocate TXB\n",
+ dev_warn("%s: Could not allocate TXB\n",
ieee->dev->name);
goto failed;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Outreachy kernel] [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk
2015-03-06 14:18 [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk Hatice ERTÜRK
@ 2015-03-06 15:40 ` Julia Lawall
2015-03-16 14:50 ` Greg KH
1 sibling, 0 replies; 3+ messages in thread
From: Julia Lawall @ 2015-03-06 15:40 UTC (permalink / raw)
To: Hatice ERTÜRK; +Cc: outreachy-kernel
[-- Attachment #1: Type: TEXT/PLAIN, Size: 3748 bytes --]
On Fri, 6 Mar 2015, Hatice ERTÜRK wrote:
> This patch fixes the following warning using checkpatch.pl
> WARNING: Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
> dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ...
>
> Signed-off-by: Hatice ERTURK <haticeerturk27@gmail.com>
> ---
> v4:
> Changed patch number.
> ---
> drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> index ecd8542..8d4ade4 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> @@ -197,7 +197,7 @@ int ieee80211_encrypt_fragment(
> struct ieee80211_hdr_3addrqos *header;
>
> header = (struct ieee80211_hdr_3addrqos *)frag->data;
> - dev_debug(KERN_DEBUG "%s: TKIP countermeasures: dropped "
> + dev_debug("%s: TKIP countermeasures: dropped "
This code will not compile.
julia
> "TX packet to %pM\n",
> ieee->dev->name, header->addr1);
> }
> @@ -219,7 +219,7 @@ int ieee80211_encrypt_fragment(
>
> atomic_dec(&crypt->refcnt);
> if (res < 0) {
> - dev_info(KERN_INFO "%s: Encryption failed: len=%d.\n",
> + dev_info("%s: Encryption failed: len=%d.\n",
> ieee->dev->name, frag->len);
> ieee->ieee_stats.tx_discards++;
> return -1;
> @@ -623,7 +623,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
> * creating it... */
> if ((!ieee->hard_start_xmit && !(ieee->softmac_features & IEEE_SOFTMAC_TX_QUEUE))||
> ((!ieee->softmac_data_hard_start_xmit && (ieee->softmac_features & IEEE_SOFTMAC_TX_QUEUE)))) {
> - dev_warn(KERN_WARNING "%s: No xmit handler.\n",
> + dev_warn("%s: No xmit handler.\n",
> ieee->dev->name);
> goto success;
> }
> @@ -631,7 +631,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
>
> if(likely(ieee->raw_tx == 0)){
> if (unlikely(skb->len < SNAP_SIZE + sizeof(u16))) {
> - dev_warn(KERN_WARNING "%s: skb too small (%d).\n",
> + dev_warn("%s: skb too small (%d).\n",
> ieee->dev->name, skb->len);
> goto success;
> }
> @@ -747,7 +747,7 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
> * postfix, header, FCS, etc.) */
> txb = ieee80211_alloc_txb(nr_frags, frag_size + ieee->tx_headroom, GFP_ATOMIC);
> if (unlikely(!txb)) {
> - dev_warn(KERN_WARNING "%s: Could not allocate TXB\n",
> + dev_warn("%s: Could not allocate TXB\n",
> ieee->dev->name);
> goto failed;
> }
> @@ -846,14 +846,14 @@ int ieee80211_xmit(struct sk_buff *skb, struct net_device *dev)
> }
> }else{
> if (unlikely(skb->len < sizeof(struct ieee80211_hdr_3addr))) {
> - dev_warn(KERN_WARNING "%s: skb too small (%d).\n",
> + dev_warn("%s: skb too small (%d).\n",
> ieee->dev->name, skb->len);
> goto success;
> }
>
> txb = ieee80211_alloc_txb(1, skb->len, GFP_ATOMIC);
> if(!txb){
> - dev_warn(KERN_WARNING "%s: Could not allocate TXB\n",
> + dev_warn("%s: Could not allocate TXB\n",
> ieee->dev->name);
> goto failed;
> }
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1425651496-14666-1-git-send-email-haticeerturk27%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Outreachy kernel] [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk
2015-03-06 14:18 [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk Hatice ERTÜRK
2015-03-06 15:40 ` [Outreachy kernel] " Julia Lawall
@ 2015-03-16 14:50 ` Greg KH
1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2015-03-16 14:50 UTC (permalink / raw)
To: Hatice ERTÜRK; +Cc: outreachy-kernel
On Fri, Mar 06, 2015 at 04:18:16PM +0200, Hatice ERT�RK wrote:
> This patch fixes the following warning using checkpatch.pl
> WARNING: Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
> dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ...
>
> Signed-off-by: Hatice ERTURK <haticeerturk27@gmail.com>
> ---
> v4:
> Changed patch number.
> ---
> drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> index ecd8542..8d4ade4 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
> @@ -197,7 +197,7 @@ int ieee80211_encrypt_fragment(
> struct ieee80211_hdr_3addrqos *header;
>
> header = (struct ieee80211_hdr_3addrqos *)frag->data;
> - dev_debug(KERN_DEBUG "%s: TKIP countermeasures: dropped "
> + dev_debug("%s: TKIP countermeasures: dropped "
> "TX packet to %pM\n",
> ieee->dev->name, header->addr1);
This doesn't even apply, let alone compile, what are you making it
against?
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-03-16 14:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-06 14:18 [PATCH v4] Staging: rtl8192u: ieee80211: pr_warn instead printk Hatice ERTÜRK
2015-03-06 15:40 ` [Outreachy kernel] " Julia Lawall
2015-03-16 14:50 ` Greg KH
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.