I did allow the remaining 81 character line behind.  It did not seem
like it was worth changing.  Otherwise, it now passes checkpatch.pl.

Signed-off-by: Robin Holt <h...@sgi.com>
To: Ingo Molnar <mi...@redhat.com>
To: Russ Anderson <r...@sgi.com>
Cc: Shawn Guo <shawn....@linaro.org>
Cc: Oleg Nesterov <o...@redhat.com>
Cc: Andrew Morton <a...@linux-foundation.org>
Cc: "H. Peter Anvin" <h...@zytor.com>
Cc: Lai Jiangshan <la...@cn.fujitsu.com>
Cc: Linus Torvalds <torva...@linux-foundation.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Cc: Michel Lespinasse <wal...@google.com>
Cc: Oleg Nesterov <o...@redhat.com>
Cc: "Paul E. McKenney" <paul...@linux.vnet.ibm.com>
Cc: Paul Mackerras <pau...@samba.org>
Cc: Peter Zijlstra <pet...@infradead.org>
Cc: Robin Holt <h...@sgi.com>
Cc: "ru...@rustcorp.com.au" <ru...@rustcorp.com.au>
Cc: Tejun Heo <t...@kernel.org>
Cc: the arch/x86 maintainers <x...@kernel.org>
Cc: Thomas Gleixner <t...@linutronix.de>
---
 kernel/reboot.c | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/kernel/reboot.c b/kernel/reboot.c
index 00a82ed..187a0d8 100644
--- a/kernel/reboot.c
+++ b/kernel/reboot.c
@@ -96,9 +96,9 @@ void kernel_restart(char *cmd)
        migrate_to_reboot_cpu();
        syscore_shutdown();
        if (!cmd)
-               printk(KERN_EMERG "Restarting system.\n");
+               pr_emerg("Restarting system.\n");
        else
-               printk(KERN_EMERG "Restarting system with command '%s'.\n", 
cmd);
+               pr_emerg("Restarting system with command '%s'.\n", cmd);
        kmsg_dump(KMSG_DUMP_RESTART);
        machine_restart(cmd);
 }
@@ -107,7 +107,7 @@ EXPORT_SYMBOL_GPL(kernel_restart);
 static void kernel_shutdown_prepare(enum system_states state)
 {
        blocking_notifier_call_chain(&reboot_notifier_list,
-               (state == SYSTEM_HALT)?SYS_HALT:SYS_POWER_OFF, NULL);
+               (state == SYSTEM_HALT) ? SYS_HALT : SYS_POWER_OFF, NULL);
        system_state = state;
        usermodehelper_disable();
        device_shutdown();
@@ -122,11 +122,10 @@ void kernel_halt(void)
        kernel_shutdown_prepare(SYSTEM_HALT);
        migrate_to_reboot_cpu();
        syscore_shutdown();
-       printk(KERN_EMERG "System halted.\n");
+       pr_emerg("System halted.\n");
        kmsg_dump(KMSG_DUMP_HALT);
        machine_halt();
 }
-
 EXPORT_SYMBOL_GPL(kernel_halt);
 
 /**
@@ -141,7 +140,7 @@ void kernel_power_off(void)
                pm_power_off_prepare();
        migrate_to_reboot_cpu();
        syscore_shutdown();
-       printk(KERN_EMERG "Power down.\n");
+       pr_emerg("Power down.\n");
        kmsg_dump(KMSG_DUMP_POWEROFF);
        machine_power_off();
 }
@@ -170,10 +169,10 @@ SYSCALL_DEFINE4(reboot, int, magic1, int, magic2, 
unsigned int, cmd,
 
        /* For safety, we require "magic" arguments. */
        if (magic1 != LINUX_REBOOT_MAGIC1 ||
-           (magic2 != LINUX_REBOOT_MAGIC2 &&
-                       magic2 != LINUX_REBOOT_MAGIC2A &&
+                       (magic2 != LINUX_REBOOT_MAGIC2 &&
+                       magic2 != LINUX_REBOOT_MAGIC2A &&
                        magic2 != LINUX_REBOOT_MAGIC2B &&
-                       magic2 != LINUX_REBOOT_MAGIC2C))
+                       magic2 != LINUX_REBOOT_MAGIC2C))
                return -EINVAL;
 
        /*
@@ -282,14 +281,13 @@ static int __orderly_poweroff(bool force)
                ret = call_usermodehelper(argv[0], argv, envp, UMH_WAIT_EXEC);
                argv_free(argv);
        } else {
-               printk(KERN_WARNING "%s failed to allocate memory for \"%s\"\n",
+               pr_warn("%s failed to allocate memory for \"%s\"\n",
                                         __func__, poweroff_cmd);
                ret = -ENOMEM;
        }
 
        if (ret && force) {
-               printk(KERN_WARNING "Failed to start orderly shutdown: "
-                                       "forcing the issue\n");
+               pr_warn("Failed to start orderly shutdown: forcing the 
issue\n");
                /*
                 * I guess this should try to kick off some daemon to sync and
                 * poweroff asap.  Or not even bother syncing if we're doing an
-- 
1.8.1.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to