ACPI: sbshc: Unify the message printing
authorHanjun Guo <guohanjun@huawei.com>
Wed, 2 Jun 2021 08:54:36 +0000 (16:54 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 7 Jun 2021 13:36:46 +0000 (15:36 +0200)
Using pr_fmt() and pr_*() macros to unify the message printing.

Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/sbshc.c

index 53c2862..d69a2a8 100644 (file)
@@ -5,6 +5,8 @@
  * Copyright (c) 2007 Alexey Starikovskiy
  */
 
+#define pr_fmt(fmt) "ACPI: " fmt
+
 #include <linux/acpi.h>
 #include <linux/wait.h>
 #include <linux/slab.h>
@@ -13,8 +15,6 @@
 #include <linux/interrupt.h>
 #include "sbshc.h"
 
-#define PREFIX "ACPI: "
-
 #define ACPI_SMB_HC_CLASS      "smbus_host_ctl"
 #define ACPI_SMB_HC_DEVICE_NAME        "ACPI SMBus HC"
 
@@ -109,7 +109,7 @@ static int acpi_smbus_transaction(struct acpi_smb_hc *hc, u8 protocol,
        u8 temp, sz = 0;
 
        if (!hc) {
-               printk(KERN_ERR PREFIX "host controller is not configured\n");
+               pr_err("host controller is not configured\n");
                return ret;
        }
 
@@ -254,7 +254,7 @@ static int acpi_smbus_hc_add(struct acpi_device *device)
 
        status = acpi_evaluate_integer(device->handle, "_EC", NULL, &val);
        if (ACPI_FAILURE(status)) {
-               printk(KERN_ERR PREFIX "error obtaining _EC.\n");
+               pr_err("error obtaining _EC.\n");
                return -EIO;
        }