ACPI: scan: Replace ACPI_DEBUG_PRINT() with pr_debug()

Tian Tao · rafaeljw · commit 05de068614ac · 2020-10-02T19:10:24.000+02:00
Fix the following W=1 kernel build warning(s):

drivers/acpi/scan.c: In function ‘acpi_bus_get_wakeup_device_flags’:
drivers/acpi/scan.c:902:43: warning: suggest braces around empty body in
an ‘if’ statement [-Wempty-body]

by using pr_debug() to instead of the ACPI_DEBUG_PRINT() macro, which
should only be used by ACPICA code, to print a debug message.

Signed-off-by: Tian Tao &lt;tiantao6@hisilicon.com&gt;
[ rjw: Subject and changelog edits ]
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
@@ -898,8 +898,7 @@ static void acpi_bus_get_wakeup_device_flags(struct acpi_device *device)
 	 */
 	err = acpi_device_sleep_wake(device, 0, 0, 0);
 	if (err)
-		ACPI_DEBUG_PRINT((ACPI_DB_INFO,
-				"error in _DSW or _PSW evaluation\n"));
+		pr_debug("error in _DSW or _PSW evaluation\n");
 }
 
 static void acpi_bus_init_power_state(struct acpi_device *device, int state)

Original file line number	Diff line number	Diff line change
`@@ -898,8 +898,7 @@ static void acpi_bus_get_wakeup_device_flags(struct acpi_device *device)`
`898`	`898`	`*/`
`899`	`899`	`err = acpi_device_sleep_wake(device, 0, 0, 0);`
`900`	`900`	`if (err)`
`901`		`- ACPI_DEBUG_PRINT((ACPI_DB_INFO,`
`902`		`- "error in _DSW or _PSW evaluation\n"));`
	`901`	`+ pr_debug("error in _DSW or _PSW evaluation\n");`
`903`	`902`	`}`
`904`	`903`
`905`	`904`	`static void acpi_bus_init_power_state(struct acpi_device *device, int state)`