[PATCH 4/4] rtl8192u: fix printk calls in r8192U_core.c
Xenia Ragiadakou
burzalodowa at gmail.com
Thu May 23 02:14:49 UTC 2013
This patch fixes the following checkpatch warning:
WARNING: printk() should include KERN_ facility level
and also replaces calls to printk(KERN_ fmt, ...) with
their corresponding calls to pr_ (fmt, ...), defined in
include/linux/printk.h, to save space.
Signed-off-by: Xenia Ragiadakou <burzalodowa at gmail.com>
---
drivers/staging/rtl8192u/r8192U_core.c | 88 ++++++++++++++++----------------
1 file changed, 43 insertions(+), 45 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c
index f790292..f798c95 100644
--- a/drivers/staging/rtl8192u/r8192U_core.c
+++ b/drivers/staging/rtl8192u/r8192U_core.c
@@ -285,7 +285,7 @@ void write_nic_byte_E(struct net_device *dev, int indx, u8 data)
if (status < 0)
{
- printk("write_nic_byte_E TimeOut! status:%d\n", status);
+ pr_err("write_nic_byte_E TimeOut! status:%d\n", status);
}
}
@@ -302,7 +302,7 @@ u8 read_nic_byte_E(struct net_device *dev, int indx)
if (status < 0)
{
- printk("read_nic_byte_E TimeOut! status:%d\n", status);
+ pr_err("read_nic_byte_E TimeOut! status:%d\n", status);
}
return data;
@@ -321,7 +321,7 @@ void write_nic_byte(struct net_device *dev, int indx, u8 data)
if (status < 0)
{
- printk("write_nic_byte TimeOut! status:%d\n", status);
+ pr_err("write_nic_byte TimeOut! status:%d\n", status);
}
@@ -342,7 +342,7 @@ void write_nic_word(struct net_device *dev, int indx, u16 data)
if (status < 0)
{
- printk("write_nic_word TimeOut! status:%d\n", status);
+ pr_err("write_nic_word TimeOut! status:%d\n", status);
}
}
@@ -363,7 +363,7 @@ void write_nic_dword(struct net_device *dev, int indx, u32 data)
if (status < 0)
{
- printk("write_nic_dword TimeOut! status:%d\n", status);
+ pr_err("write_nic_dword TimeOut! status:%d\n", status);
}
}
@@ -383,7 +383,7 @@ u8 read_nic_byte(struct net_device *dev, int indx)
if (status < 0)
{
- printk("read_nic_byte TimeOut! status:%d\n", status);
+ pr_err("read_nic_byte TimeOut! status:%d\n", status);
}
return data;
@@ -404,7 +404,7 @@ u16 read_nic_word(struct net_device *dev, int indx)
&data, 2, HZ / 2);
if (status < 0)
- printk("read_nic_word TimeOut! status:%d\n", status);
+ pr_err("read_nic_word TimeOut! status:%d\n", status);
return data;
}
@@ -421,7 +421,7 @@ u16 read_nic_word_E(struct net_device *dev, int indx)
indx|0xfe00, 0, &data, 2, HZ / 2);
if (status < 0)
- printk("read_nic_word TimeOut! status:%d\n", status);
+ pr_err("read_nic_word TimeOut! status:%d\n", status);
return data;
}
@@ -446,7 +446,7 @@ u32 read_nic_dword(struct net_device *dev, int indx)
*/
if (status < 0)
- printk("read_nic_dword TimeOut! status:%d\n", status);
+ pr_err("read_nic_dword TimeOut! status:%d\n", status);
return data;
}
@@ -706,17 +706,17 @@ void print_buffer(u32 *buffer, int len)
int i;
u8 *buf = (u8 *)buffer;
- printk("ASCII BUFFER DUMP (len: %x):\n", len);
+ pr_devel("ASCII BUFFER DUMP (len: %x):\n", len);
for (i = 0; i < len; i++)
- printk("%c", buf[i]);
+ pr_devel("%c", buf[i]);
- printk("\nBINARY BUFFER DUMP (len: %x):\n", len);
+ pr_devel("\nBINARY BUFFER DUMP (len: %x):\n", len);
for (i = 0; i < len; i++)
- printk("%x", buf[i]);
+ pr_devel("%x", buf[i]);
- printk("\n");
+ pr_devel("\n");
}
//short check_nic_enough_desc(struct net_device *dev, priority_t priority)
@@ -757,11 +757,11 @@ void rtl8192_dump_reg(struct net_device *dev)
for (n = 0; n <= max;)
{
- printk("\nD: %2x> ", n);
+ pr_devel("\nD: %2x> ", n);
for (i = 0; i < 16 && n <= max; i++, n++)
- printk("%2x ", read_nic_byte(dev, n));
+ pr_devel("%2x ", read_nic_byte(dev, n));
}
- printk("\n");
+ pr_devel("\n");
}
/****************************************************************************
@@ -999,7 +999,7 @@ void rtl8192_rtx_disable(struct net_device *dev)
}
if (skb_queue_len(&priv->skb_queue)) {
- printk(KERN_WARNING "skb_queue not empty\n");
+ pr_warn("skb_queue not empty\n");
}
skb_queue_purge(&priv->skb_queue);
@@ -1080,7 +1080,7 @@ static void rtl8192_rx_isr(struct urb *urb)
skb = dev_alloc_skb(RX_URB_SIZE);
if (unlikely(!skb)) {
usb_free_urb(urb);
- printk("%s():can,t alloc skb\n", __func__);
+ pr_err("%s():can,t alloc skb\n", __func__);
/* TODO check rx queue length and refill *somewhere* */
return;
}
@@ -1099,7 +1099,7 @@ static void rtl8192_rx_isr(struct urb *urb)
skb_queue_tail(&priv->rx_queue, skb);
err = usb_submit_urb(urb, GFP_ATOMIC);
if (err && err != EPERM)
- printk("can not submit rxurb, err is %x,URB status is %x\n", err, urb->status);
+ pr_err("can not submit rxurb, err is %x,URB status is %x\n", err, urb->status);
}
u32
@@ -1257,7 +1257,7 @@ struct sk_buff *DrvAggr_Aggregation(struct net_device *dev, struct ieee80211_drv
tcb_desc->drv_agg_enable = 1;
tcb_desc->pkt_size = skb->len;
tcb_desc->DrvAggrNum = pSendList->nr_drv_agg_frames;
- printk("DrvAggNum = %d\n", tcb_desc->DrvAggrNum);
+ pr_devel("DrvAggNum = %d\n", tcb_desc->DrvAggrNum);
// RT_DEBUG_DATA(COMP_SEND, skb->cb, sizeof(skb->cb));
// printk("========>skb->data ======> \n");
// RT_DEBUG_DATA(COMP_SEND, skb->data, skb->len);
@@ -1794,10 +1794,10 @@ short rtl819xU_tx_cmd(struct net_device *dev, struct sk_buff *skb)
#endif
#ifdef JOHN_DUMP_TXDESC
int i;
- printk("<Tx descriptor>--rate %x---", rate);
+ pr_devel("<Tx descriptor>--rate %x---", rate);
for (i = 0; i < 8; i++)
- printk("%8x ", tx[i]);
- printk("\n");
+ pr_devel("%8x ", tx[i]);
+ pr_devel("\n");
#endif
usb_fill_bulk_urb(tx_urb, priv->udev, usb_sndbulkpipe(priv->udev, idx_pipe), \
skb->data, skb->len, rtl8192_tx_isr, skb);
@@ -1951,7 +1951,7 @@ short rtl8192_tx(struct net_device *dev, struct sk_buff *skb)
* !!! For debug purpose
*/
if (pend > MAX_TX_URB){
- printk("To discard skb packet!\n");
+ pr_devel("To discard skb packet!\n");
dev_kfree_skb_any(skb);
return -1;
}
@@ -2191,7 +2191,7 @@ short rtl8192_usb_initendpoints(struct net_device *dev)
return -ENOMEM;
}
- printk("End of initendpoints\n");
+ pr_devel("End of initendpoints\n");
return 0;
}
@@ -3074,7 +3074,7 @@ short rtl8192_get_channel_map(struct net_device *dev)
{
struct r8192_priv *priv = ieee80211_priv(dev);
if (priv->ChannelPlan > COUNTRY_CODE_GLOBAL_DOMAIN){
- printk("rtl8180_init:Error channel plan! Set to default.\n");
+ pr_err("rtl8180_init:Error channel plan! Set to default.\n");
priv->ChannelPlan = 0;
}
RT_TRACE(COMP_INIT, "Channel plan is %d\n", priv->ChannelPlan);
@@ -3886,7 +3886,7 @@ RESET_START:
if (ieee->state == IEEE80211_LINKED)
{
down(&ieee->wx_sem);
- printk("ieee->state is IEEE80211_LINKED\n");
+ pr_devel("ieee->state is IEEE80211_LINKED\n");
ieee80211_stop_send_beacons(priv->ieee80211);
del_timer_sync(&ieee->associate_timer);
cancel_delayed_work(&ieee->associate_retry_wq);
@@ -3895,7 +3895,7 @@ RESET_START:
up(&ieee->wx_sem);
}
else{
- printk("ieee->state is NOT LINKED\n");
+ pr_devel("ieee->state is NOT LINKED\n");
ieee80211_softmac_stop_protocol(priv->ieee80211); }
up(&priv->wx_sem);
RT_TRACE(COMP_RESET, "%s():<==========down process is finished\n", __func__);
@@ -3989,7 +3989,7 @@ void CAM_read_entry(
RT_TRACE(COMP_SEC, "CAM_read_entry(): WRITE A8: %x \n", target_content);
// printk("CAM_read_entry(): WRITE A8: %lx \n",target_content);
}
- printk("\n");
+ pr_devel("\n");
}
void rtl819x_update_rxcounts(
@@ -4054,7 +4054,7 @@ extern void rtl819x_watchdog_wqcallback(struct work_struct *work)
if (rfState == eRfOff)
RT_TRACE(COMP_ERR, "========>%s()\n", __func__);
#endif
- printk("===>%s(): AP is power off,connect another one\n", __func__);
+ pr_devel("===>%s(): AP is power off,connect another one\n", __func__);
// Dot11d_Reset(dev);
priv->ieee80211->state = IEEE80211_ASSOCIATING;
notify_wx_assoc_event(priv->ieee80211);
@@ -4376,12 +4376,12 @@ int rtl8192_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
}
#ifdef JOHN_HWSEC_DEBUG
//john's test 0711
- printk("@@ wrq->u pointer = ");
+ pr_devel("@@ wrq->u pointer = ");
for (i = 0; i < wrq->u.data.length; i++){
- if (i%10 == 0) printk("\n");
- printk("%8x|", ((u32 *)wrq->u.data.pointer)[i]);
+ if (i%10 == 0) pr_devel("\n");
+ pr_devel("%8x|", ((u32 *)wrq->u.data.pointer)[i]);
}
- printk("\n");
+ pr_devel("\n");
#endif /*JOHN_HWSEC_DEBUG*/
ret = ieee80211_wpa_supplicant_ioctl(priv->ieee80211, &wrq->u.data);
break;
@@ -5514,7 +5514,7 @@ void rtl8192_rx_nomal(struct sk_buff *skb)
#endif
} else {
priv->stats.rxurberr++;
- printk("actual_length:%d\n", skb->len);
+ pr_devel("actual_length:%d\n", skb->len);
dev_kfree_skb_any(skb);
}
@@ -5815,38 +5815,36 @@ static int __init rtl8192_usb_module_init(void)
#ifdef CONFIG_IEEE80211_DEBUG
ret = ieee80211_debug_init();
if (ret) {
- printk(KERN_ERR "ieee80211_debug_init() failed %d\n", ret);
+ pr_err("ieee80211_debug_init() failed %d\n", ret);
return ret;
}
#endif
ret = ieee80211_crypto_init();
if (ret) {
- printk(KERN_ERR "ieee80211_crypto_init() failed %d\n", ret);
+ pr_err("ieee80211_crypto_init() failed %d\n", ret);
return ret;
}
ret = ieee80211_crypto_tkip_init();
if (ret) {
- printk(KERN_ERR "ieee80211_crypto_tkip_init() failed %d\n",
- ret);
+ pr_err("ieee80211_crypto_tkip_init() failed %d\n", ret);
return ret;
}
ret = ieee80211_crypto_ccmp_init();
if (ret) {
- printk(KERN_ERR "ieee80211_crypto_ccmp_init() failed %d\n",
- ret);
+ pr_err("ieee80211_crypto_ccmp_init() failed %d\n", ret);
return ret;
}
ret = ieee80211_crypto_wep_init();
if (ret) {
- printk(KERN_ERR "ieee80211_crypto_wep_init() failed %d\n", ret);
+ pr_err("ieee80211_crypto_wep_init() failed %d\n", ret);
return ret;
}
- printk(KERN_INFO "\nLinux kernel driver for RTL8192 based WLAN cards\n");
- printk(KERN_INFO "Copyright (c) 2007-2008, Realsil Wlan\n");
+ pr_info("\nLinux kernel driver for RTL8192 based WLAN cards\n");
+ pr_info("Copyright (c) 2007-2008, Realsil Wlan\n");
RT_TRACE(COMP_INIT, "Initializing module");
RT_TRACE(COMP_INIT, "Wireless extensions version %d", WIRELESS_EXT);
rtl8192_proc_module_init();
--
1.7.10.4
More information about the devel
mailing list