ath: Add and use pr_fmt, convert printks to pr_<level>
Use a more current logging style. Make sure all output is prefixed appropriately. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:

committed by
John W. Linville

parent
47107e8444
commit
516304b0f4
@@ -14,6 +14,8 @@
|
||||
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include "ath5k.h"
|
||||
#include "reg.h"
|
||||
#include "debug.h"
|
||||
@@ -728,33 +730,25 @@ void
|
||||
ath5k_ani_print_counters(struct ath5k_hw *ah)
|
||||
{
|
||||
/* clears too */
|
||||
printk(KERN_NOTICE "ACK fail\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_ACK_FAIL));
|
||||
printk(KERN_NOTICE "RTS fail\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_RTS_FAIL));
|
||||
printk(KERN_NOTICE "RTS success\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_RTS_OK));
|
||||
printk(KERN_NOTICE "FCS error\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_FCS_FAIL));
|
||||
pr_notice("ACK fail\t%d\n", ath5k_hw_reg_read(ah, AR5K_ACK_FAIL));
|
||||
pr_notice("RTS fail\t%d\n", ath5k_hw_reg_read(ah, AR5K_RTS_FAIL));
|
||||
pr_notice("RTS success\t%d\n", ath5k_hw_reg_read(ah, AR5K_RTS_OK));
|
||||
pr_notice("FCS error\t%d\n", ath5k_hw_reg_read(ah, AR5K_FCS_FAIL));
|
||||
|
||||
/* no clear */
|
||||
printk(KERN_NOTICE "tx\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PROFCNT_TX));
|
||||
printk(KERN_NOTICE "rx\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PROFCNT_RX));
|
||||
printk(KERN_NOTICE "busy\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PROFCNT_RXCLR));
|
||||
printk(KERN_NOTICE "cycles\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PROFCNT_CYCLE));
|
||||
pr_notice("tx\t%d\n", ath5k_hw_reg_read(ah, AR5K_PROFCNT_TX));
|
||||
pr_notice("rx\t%d\n", ath5k_hw_reg_read(ah, AR5K_PROFCNT_RX));
|
||||
pr_notice("busy\t%d\n", ath5k_hw_reg_read(ah, AR5K_PROFCNT_RXCLR));
|
||||
pr_notice("cycles\t%d\n", ath5k_hw_reg_read(ah, AR5K_PROFCNT_CYCLE));
|
||||
|
||||
printk(KERN_NOTICE "AR5K_PHYERR_CNT1\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PHYERR_CNT1));
|
||||
printk(KERN_NOTICE "AR5K_PHYERR_CNT2\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PHYERR_CNT2));
|
||||
printk(KERN_NOTICE "AR5K_OFDM_FIL_CNT\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_OFDM_FIL_CNT));
|
||||
printk(KERN_NOTICE "AR5K_CCK_FIL_CNT\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_CCK_FIL_CNT));
|
||||
pr_notice("AR5K_PHYERR_CNT1\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PHYERR_CNT1));
|
||||
pr_notice("AR5K_PHYERR_CNT2\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_PHYERR_CNT2));
|
||||
pr_notice("AR5K_OFDM_FIL_CNT\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_OFDM_FIL_CNT));
|
||||
pr_notice("AR5K_CCK_FIL_CNT\t%d\n",
|
||||
ath5k_hw_reg_read(ah, AR5K_CCK_FIL_CNT));
|
||||
}
|
||||
|
||||
#endif
|
||||
|
@@ -76,26 +76,28 @@
|
||||
GENERIC DRIVER DEFINITIONS
|
||||
\****************************/
|
||||
|
||||
#define ATH5K_PRINTF(fmt, ...) \
|
||||
printk(KERN_WARNING "%s: " fmt, __func__, ##__VA_ARGS__)
|
||||
#define ATH5K_PRINTF(fmt, ...) \
|
||||
pr_warn("%s: " fmt, __func__, ##__VA_ARGS__)
|
||||
|
||||
#define ATH5K_PRINTK(_sc, _level, _fmt, ...) \
|
||||
printk(_level "ath5k %s: " _fmt, \
|
||||
((_sc) && (_sc)->hw) ? wiphy_name((_sc)->hw->wiphy) : "", \
|
||||
##__VA_ARGS__)
|
||||
#define ATH5K_PRINTK(_sc, _level, _fmt, ...) \
|
||||
printk(_level pr_fmt("%s%s" _fmt), \
|
||||
((_sc) && (_sc)->hw) ? wiphy_name((_sc)->hw->wiphy) : "", \
|
||||
((_sc) && (_sc)->hw) ? ": " : "", \
|
||||
##__VA_ARGS__)
|
||||
|
||||
#define ATH5K_PRINTK_LIMIT(_sc, _level, _fmt, ...) do { \
|
||||
if (net_ratelimit()) \
|
||||
ATH5K_PRINTK(_sc, _level, _fmt, ##__VA_ARGS__); \
|
||||
} while (0)
|
||||
#define ATH5K_PRINTK_LIMIT(_sc, _level, _fmt, ...) \
|
||||
do { \
|
||||
if (net_ratelimit()) \
|
||||
ATH5K_PRINTK(_sc, _level, _fmt, ##__VA_ARGS__); \
|
||||
} while (0)
|
||||
|
||||
#define ATH5K_INFO(_sc, _fmt, ...) \
|
||||
#define ATH5K_INFO(_sc, _fmt, ...) \
|
||||
ATH5K_PRINTK(_sc, KERN_INFO, _fmt, ##__VA_ARGS__)
|
||||
|
||||
#define ATH5K_WARN(_sc, _fmt, ...) \
|
||||
#define ATH5K_WARN(_sc, _fmt, ...) \
|
||||
ATH5K_PRINTK_LIMIT(_sc, KERN_WARNING, _fmt, ##__VA_ARGS__)
|
||||
|
||||
#define ATH5K_ERR(_sc, _fmt, ...) \
|
||||
#define ATH5K_ERR(_sc, _fmt, ...) \
|
||||
ATH5K_PRINTK_LIMIT(_sc, KERN_ERR, _fmt, ##__VA_ARGS__)
|
||||
|
||||
/*
|
||||
|
@@ -20,6 +20,8 @@
|
||||
* Attach/Detach Functions and helpers *
|
||||
\*************************************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/pci.h>
|
||||
#include <linux/slab.h>
|
||||
#include "ath5k.h"
|
||||
|
@@ -40,6 +40,8 @@
|
||||
*
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/module.h>
|
||||
#include <linux/delay.h>
|
||||
#include <linux/dma-mapping.h>
|
||||
|
@@ -57,6 +57,9 @@
|
||||
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
|
||||
* THE POSSIBILITY OF SUCH DAMAGES.
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/export.h>
|
||||
#include <linux/moduleparam.h>
|
||||
|
||||
@@ -254,10 +257,10 @@ static ssize_t write_file_beacon(struct file *file,
|
||||
|
||||
if (strncmp(buf, "disable", 7) == 0) {
|
||||
AR5K_REG_DISABLE_BITS(ah, AR5K_BEACON, AR5K_BEACON_ENABLE);
|
||||
printk(KERN_INFO "debugfs disable beacons\n");
|
||||
pr_info("debugfs disable beacons\n");
|
||||
} else if (strncmp(buf, "enable", 6) == 0) {
|
||||
AR5K_REG_ENABLE_BITS(ah, AR5K_BEACON, AR5K_BEACON_ENABLE);
|
||||
printk(KERN_INFO "debugfs enable beacons\n");
|
||||
pr_info("debugfs enable beacons\n");
|
||||
}
|
||||
return count;
|
||||
}
|
||||
@@ -457,19 +460,19 @@ static ssize_t write_file_antenna(struct file *file,
|
||||
|
||||
if (strncmp(buf, "diversity", 9) == 0) {
|
||||
ath5k_hw_set_antenna_mode(ah, AR5K_ANTMODE_DEFAULT);
|
||||
printk(KERN_INFO "ath5k debug: enable diversity\n");
|
||||
pr_info("debug: enable diversity\n");
|
||||
} else if (strncmp(buf, "fixed-a", 7) == 0) {
|
||||
ath5k_hw_set_antenna_mode(ah, AR5K_ANTMODE_FIXED_A);
|
||||
printk(KERN_INFO "ath5k debugfs: fixed antenna A\n");
|
||||
pr_info("debug: fixed antenna A\n");
|
||||
} else if (strncmp(buf, "fixed-b", 7) == 0) {
|
||||
ath5k_hw_set_antenna_mode(ah, AR5K_ANTMODE_FIXED_B);
|
||||
printk(KERN_INFO "ath5k debug: fixed antenna B\n");
|
||||
pr_info("debug: fixed antenna B\n");
|
||||
} else if (strncmp(buf, "clear", 5) == 0) {
|
||||
for (i = 0; i < ARRAY_SIZE(ah->stats.antenna_rx); i++) {
|
||||
ah->stats.antenna_rx[i] = 0;
|
||||
ah->stats.antenna_tx[i] = 0;
|
||||
}
|
||||
printk(KERN_INFO "ath5k debug: cleared antenna stats\n");
|
||||
pr_info("debug: cleared antenna stats\n");
|
||||
}
|
||||
return count;
|
||||
}
|
||||
@@ -639,7 +642,7 @@ static ssize_t write_file_frameerrors(struct file *file,
|
||||
st->txerr_fifo = 0;
|
||||
st->txerr_filt = 0;
|
||||
st->tx_all_count = 0;
|
||||
printk(KERN_INFO "ath5k debug: cleared frameerrors stats\n");
|
||||
pr_info("debug: cleared frameerrors stats\n");
|
||||
}
|
||||
return count;
|
||||
}
|
||||
|
@@ -21,6 +21,8 @@
|
||||
Hardware Descriptor Functions
|
||||
\******************************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include "ath5k.h"
|
||||
#include "reg.h"
|
||||
#include "debug.h"
|
||||
|
@@ -29,6 +29,8 @@
|
||||
* status registers (ISR).
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include "ath5k.h"
|
||||
#include "reg.h"
|
||||
#include "debug.h"
|
||||
|
@@ -21,6 +21,8 @@
|
||||
* EEPROM access functions and helpers *
|
||||
\*************************************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/slab.h>
|
||||
|
||||
#include "ath5k.h"
|
||||
|
@@ -19,6 +19,8 @@
|
||||
*
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include "ath5k.h"
|
||||
#include "reg.h"
|
||||
#include "debug.h"
|
||||
@@ -1574,8 +1576,7 @@ ath5k_hw_write_initvals(struct ath5k_hw *ah, u8 mode, bool skip_pcu)
|
||||
|
||||
/* AR5K_MODE_11B */
|
||||
if (mode > 2) {
|
||||
ATH5K_ERR(ah,
|
||||
"unsupported channel mode: %d\n", mode);
|
||||
ATH5K_ERR(ah, "unsupported channel mode: %d\n", mode);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
|
@@ -39,6 +39,8 @@
|
||||
*
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/pci.h>
|
||||
#include "ath5k.h"
|
||||
|
||||
|
@@ -41,6 +41,8 @@
|
||||
*
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <net/mac80211.h>
|
||||
#include <asm/unaligned.h>
|
||||
|
||||
|
@@ -14,6 +14,8 @@
|
||||
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
|
||||
*/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/nl80211.h>
|
||||
#include <linux/pci.h>
|
||||
#include <linux/pci-aspm.h>
|
||||
@@ -347,7 +349,7 @@ init_ath5k_pci(void)
|
||||
|
||||
ret = pci_register_driver(&ath5k_pci_driver);
|
||||
if (ret) {
|
||||
printk(KERN_ERR "ath5k_pci: can't register pci driver\n");
|
||||
pr_err("pci: can't register pci driver\n");
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
@@ -22,6 +22,8 @@
|
||||
* PHY related functions *
|
||||
\***********************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/delay.h>
|
||||
#include <linux/slab.h>
|
||||
#include <asm/unaligned.h>
|
||||
|
@@ -20,6 +20,8 @@
|
||||
Queue Control Unit, DCF Control Unit Functions
|
||||
\********************************************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include "ath5k.h"
|
||||
#include "reg.h"
|
||||
#include "debug.h"
|
||||
|
@@ -23,6 +23,8 @@
|
||||
Reset function and helpers
|
||||
\****************************/
|
||||
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <asm/unaligned.h>
|
||||
|
||||
#include <linux/pci.h> /* To determine if a card is pci-e */
|
||||
|
@@ -1,3 +1,5 @@
|
||||
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||
|
||||
#include <linux/device.h>
|
||||
#include <linux/pci.h>
|
||||
|
||||
|
Reference in New Issue
Block a user