ramoops: add new line to each print
authorMarco Stornelli <marco.stornelli@gmail.com>
Tue, 26 Jul 2011 23:08:57 +0000 (16:08 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 26 Jul 2011 23:49:45 +0000 (16:49 -0700)
Add new line to each print.

Signed-off-by: Marco Stornelli <marco.stornelli@gmail.com>
Reported-by: Stevie Trujillo <stevie.trujillo@gmail.com>
Cc: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Américo Wang <xiyou.wangcong@gmail.com>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/char/ramoops.c

index 32aa233df30058af99228a5709e3feec046dd5e9..c9e1028766a4b6de62688cef5adc59928ef924aa 100644 (file)
@@ -19,6 +19,8 @@
  *
  */
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/kmsg_dump.h>
@@ -111,14 +113,14 @@ static int __init ramoops_probe(struct platform_device *pdev)
        int err = -EINVAL;
 
        if (!pdata->mem_size) {
-               printk(KERN_ERR "ramoops: invalid size specification");
+               pr_err("invalid size specification\n");
                goto fail3;
        }
 
        rounddown_pow_of_two(pdata->mem_size);
 
        if (pdata->mem_size < RECORD_SIZE) {
-               printk(KERN_ERR "ramoops: size too small");
+               pr_err("size too small\n");
                goto fail3;
        }
 
@@ -128,21 +130,21 @@ static int __init ramoops_probe(struct platform_device *pdev)
        cxt->phys_addr = pdata->mem_address;
 
        if (!request_mem_region(cxt->phys_addr, cxt->size, "ramoops")) {
-               printk(KERN_ERR "ramoops: request mem region failed");
+               pr_err("request mem region failed\n");
                err = -EINVAL;
                goto fail3;
        }
 
        cxt->virt_addr = ioremap(cxt->phys_addr,  cxt->size);
        if (!cxt->virt_addr) {
-               printk(KERN_ERR "ramoops: ioremap failed");
+               pr_err("ioremap failed\n");
                goto fail2;
        }
 
        cxt->dump.dump = ramoops_do_dump;
        err = kmsg_dump_register(&cxt->dump);
        if (err) {
-               printk(KERN_ERR "ramoops: registering kmsg dumper failed");
+               pr_err("registering kmsg dumper failed\n");
                goto fail1;
        }
 
@@ -161,7 +163,7 @@ static int __exit ramoops_remove(struct platform_device *pdev)
        struct ramoops_context *cxt = &oops_cxt;
 
        if (kmsg_dump_unregister(&cxt->dump) < 0)
-               printk(KERN_WARNING "ramoops: could not unregister kmsg_dumper");
+               pr_warn("could not unregister kmsg_dumper\n");
 
        iounmap(cxt->virt_addr);
        release_mem_region(cxt->phys_addr, cxt->size);
@@ -185,6 +187,7 @@ static int __init ramoops_init(void)
                 * If we didn't find a platform device, we use module parameters
                 * building platform data on the fly.
                 */
+               pr_info("platform device not found, using module parameters\n");
                dummy_data = kzalloc(sizeof(struct ramoops_platform_data),
                                     GFP_KERNEL);
                if (!dummy_data)