ibm_rtl: Use pr_fmt and pr_<level>
Remove hard coded prefixes from logging messages. Neaten RTL_DEBUG macro and uses. Convert __FUNCTION__ to __func__. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Matthew Garrett <mjg@redhat.com>
This commit is contained in:
committed by
Matthew Garrett
parent
b5a4223c1c
commit
323623a717
@@ -22,6 +22,8 @@
|
|||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
||||||
|
|
||||||
#include <linux/kernel.h>
|
#include <linux/kernel.h>
|
||||||
#include <linux/delay.h>
|
#include <linux/delay.h>
|
||||||
#include <linux/module.h>
|
#include <linux/module.h>
|
||||||
@@ -69,9 +71,10 @@ struct ibm_rtl_table {
|
|||||||
#define RTL_SIGNATURE 0x0000005f4c54525fULL
|
#define RTL_SIGNATURE 0x0000005f4c54525fULL
|
||||||
#define RTL_MASK 0x000000ffffffffffULL
|
#define RTL_MASK 0x000000ffffffffffULL
|
||||||
|
|
||||||
#define RTL_DEBUG(A, ...) do { \
|
#define RTL_DEBUG(fmt, ...) \
|
||||||
|
do { \
|
||||||
if (debug) \
|
if (debug) \
|
||||||
pr_info("ibm-rtl: " A, ##__VA_ARGS__ ); \
|
pr_info(fmt, ##__VA_ARGS__); \
|
||||||
} while (0)
|
} while (0)
|
||||||
|
|
||||||
static DEFINE_MUTEX(rtl_lock);
|
static DEFINE_MUTEX(rtl_lock);
|
||||||
@@ -114,7 +117,7 @@ static int ibm_rtl_write(u8 value)
|
|||||||
int ret = 0, count = 0;
|
int ret = 0, count = 0;
|
||||||
static u32 cmd_port_val;
|
static u32 cmd_port_val;
|
||||||
|
|
||||||
RTL_DEBUG("%s(%d)\n", __FUNCTION__, value);
|
RTL_DEBUG("%s(%d)\n", __func__, value);
|
||||||
|
|
||||||
value = value == 1 ? RTL_CMD_ENTER_PRTM : RTL_CMD_EXIT_PRTM;
|
value = value == 1 ? RTL_CMD_ENTER_PRTM : RTL_CMD_EXIT_PRTM;
|
||||||
|
|
||||||
@@ -144,7 +147,7 @@ static int ibm_rtl_write(u8 value)
|
|||||||
while (ioread8(&rtl_table->command)) {
|
while (ioread8(&rtl_table->command)) {
|
||||||
msleep(10);
|
msleep(10);
|
||||||
if (count++ > 500) {
|
if (count++ > 500) {
|
||||||
pr_err("ibm-rtl: Hardware not responding to "
|
pr_err("Hardware not responding to "
|
||||||
"mode switch request\n");
|
"mode switch request\n");
|
||||||
ret = -EIO;
|
ret = -EIO;
|
||||||
break;
|
break;
|
||||||
@@ -250,7 +253,7 @@ static int __init ibm_rtl_init(void) {
|
|||||||
int ret = -ENODEV, i;
|
int ret = -ENODEV, i;
|
||||||
|
|
||||||
if (force)
|
if (force)
|
||||||
pr_warning("ibm-rtl: module loaded by force\n");
|
pr_warn("module loaded by force\n");
|
||||||
/* first ensure that we are running on IBM HW */
|
/* first ensure that we are running on IBM HW */
|
||||||
else if (efi_enabled || !dmi_check_system(ibm_rtl_dmi_table))
|
else if (efi_enabled || !dmi_check_system(ibm_rtl_dmi_table))
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
Reference in New Issue
Block a user