Skip to contentSkip to conversation

ACPI: power: Use dev_dbg() to print some messages

PATCH In review Rafael J. Wysocki 2 participants 2 messages Jun 11, 2021 ACPI

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ACPI: power: Use dev_dbg() to print some messages

From: Rafael J. Wysocki <[email protected]>

The messages printed by acpi_resume_power_resources() and acpi_turn_off_unused_power_resources() are not important enough to be printed with pr_info(), so use dev_dbg() instead of it to get rid of some noise in the kernel log.

---
 drivers/acpi/power.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/acpi/power.c +2 -2
Index: linux-pm/drivers/acpi/power.c
===================================================================
@@ -1004,7 +1004,7 @@ void acpi_resume_power_resources(void)
 
 		if (state == ACPI_POWER_RESOURCE_STATE_OFF
 		    && resource->ref_count) {
-			dev_info(&resource->device.dev, "Turning ON\n");
+			dev_dbg(&resource->device.dev, "Turning ON\n");
 			__acpi_power_on(resource);
 		}
 
@@ -1034,7 +1034,7 @@ void acpi_turn_off_unused_power_resource
 		 */
 		if (!resource->ref_count &&
 		    resource->state != ACPI_POWER_RESOURCE_STATE_OFF) {
-			dev_info(&resource->device.dev, "Turning OFF\n");
+			dev_dbg(&resource->device.dev, "Turning OFF\n");
 			__acpi_power_off(resource);
 		}
 
Acked-by on v1 1/1
#1 lore raw
Quoted 26 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

The messages printed by acpi_resume_power_resources() and acpi_turn_off_unused_power_resources() are not important enough to be printed with pr_info(), so use dev_dbg() instead of it to get rid of some noise in the kernel log.

---
 drivers/acpi/power.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Index: linux-pm/drivers/acpi/power.c
===================================================================

drivers/acpi/power.c +1 -1
@@ -1004,7 +1004,7 @@ void acpi_resume_power_resources(void)
 
 		if (state == ACPI_POWER_RESOURCE_STATE_OFF
 		    && resource->ref_count) {
-			dev_info(&resource->device.dev, "Turning ON\n");
+			dev_dbg(&resource->device.dev, "Turning ON\n");
 			__acpi_power_on(resource);
 		}
 

drivers/acpi/power.c void acpi_turn_off_unused_power_resource · line 1039

Quoted 7 lines
drivers/acpi/power.c +1 -1
@@ -1034,7 +1034,7 @@ void acpi_turn_off_unused_power_resource
 		 */
 		if (!resource->ref_count &&
 		    resource->state != ACPI_POWER_RESOURCE_STATE_OFF) {
-			dev_info(&resource->device.dev, "Turning OFF\n");
+			dev_dbg(&resource->device.dev, "Turning OFF\n");
 			__acpi_power_off(resource);
 		}

Thx!

Signature · 4 lines
Regards/Gruss,
    Boris.

SUSE Software Solutions Germany GmbH, GF: Felix Imendörffer, HRB 36809, AG Nürnberg