Re: [PATCH v3 2/2] kgdboc: Change printk to the right fashion
From: Daniel Thompson <hidden>
Date: 2018-08-17 11:23:33
Also in:
lkml
On Tue, Aug 14, 2018 at 10:57:40PM +0800, zhe.he@windriver.com wrote:
From: He Zhe <redacted> pr_* is preferred according to scripts/checkpatch.pl. Signed-off-by: He Zhe <redacted>
I thought I'd already done this but it looks like maybe I didn't get round to it... Acked-by: Daniel Thompson <redacted>
quoted hunk ↗ jump to hunk
--- v2: - Split printk cleanups into a single patch - Add cc to stable@vger.kernel.org v3: - Add pr_fmt for mod name drivers/tty/serial/kgdboc.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)diff --git a/drivers/tty/serial/kgdboc.c b/drivers/tty/serial/kgdboc.c index f4d0ef6..371357d 100644 --- a/drivers/tty/serial/kgdboc.c +++ b/drivers/tty/serial/kgdboc.c@@ -8,6 +8,9 @@ * * 2007-2008 (c) Jason Wessel - Wind River Systems, Inc. */ + +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + #include <linux/kernel.h> #include <linux/ctype.h> #include <linux/kgdb.h>@@ -131,12 +134,12 @@ static void kgdboc_unregister_kbd(void) static int kgdboc_option_setup(char *opt) { if (!opt) { - pr_err("kgdboc: config string not provided\n"); + pr_err("config string not provided\n"); return -EINVAL; } if (strlen(opt) >= MAX_CONFIG_LEN) { - printk(KERN_ERR "kgdboc: config string too long\n"); + pr_err("config string too long\n"); return -ENOSPC; } strcpy(config, opt);@@ -253,7 +256,7 @@ static int param_set_kgdboc_var(const char *kmessage, int len = strlen(kmessage); if (len >= MAX_CONFIG_LEN) { - printk(KERN_ERR "kgdboc: config string too long\n"); + pr_err("config string too long\n"); return -ENOSPC; }@@ -264,8 +267,7 @@ static int param_set_kgdboc_var(const char *kmessage, } if (kgdb_connected) { - printk(KERN_ERR - "kgdboc: Cannot reconfigure while KGDB is connected.\n"); + pr_err("Cannot reconfigure while KGDB is connected.\n"); return -EBUSY; }-- 2.7.4