ACPI / scan: Change the level of _DEP-related messages to KERN_DEBUG

rafaeljw · rafaeljw · commit 80167a24a274 · 2014-12-12T22:48:44.000+01:00
Two _DEP-related failure messages are printed as dev_err() which is unnecessary and annoying. Use dev_dbg() to print them. While at it, one of the messages should actually say it is related to _DEP, so modify it to that effect. Fixes: 40e7fcb (ACPI: Add _DEP support to fix battery issue on Asus T100TA) Reported-by: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
@@ -2107,7 +2107,7 @@ static void acpi_device_dep_initialize(struct acpi_device *adev)
 	status = acpi_evaluate_reference(adev->handle, "_DEP", NULL,
 					&dep_devices);
 	if (ACPI_FAILURE(status)) {
-		dev_err(&adev->dev, "Failed to evaluate _DEP.\n");
+		dev_dbg(&adev->dev, "Failed to evaluate _DEP.\n");
 		return;
 	}
 
@@ -2117,7 +2117,7 @@ static void acpi_device_dep_initialize(struct acpi_device *adev)
 
 		status = acpi_get_object_info(dep_devices.handles[i], &info);
 		if (ACPI_FAILURE(status)) {
-			dev_err(&adev->dev, "Error reading device info\n");
+			dev_dbg(&adev->dev, "Error reading _DEP device info\n");
 			continue;
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -2107,7 +2107,7 @@ static void acpi_device_dep_initialize(struct acpi_device *adev)`
`2107`	`2107`	`status = acpi_evaluate_reference(adev->handle, "_DEP", NULL,`
`2108`	`2108`	`&dep_devices);`
`2109`	`2109`	`if (ACPI_FAILURE(status)) {`
`2110`		`- dev_err(&adev->dev, "Failed to evaluate _DEP.\n");`
	`2110`	`+ dev_dbg(&adev->dev, "Failed to evaluate _DEP.\n");`
`2111`	`2111`	`return;`
`2112`	`2112`	`}`
`2113`	`2113`
`@@ -2117,7 +2117,7 @@ static void acpi_device_dep_initialize(struct acpi_device *adev)`
`2117`	`2117`
`2118`	`2118`	`status = acpi_get_object_info(dep_devices.handles[i], &info);`
`2119`	`2119`	`if (ACPI_FAILURE(status)) {`
`2120`		`- dev_err(&adev->dev, "Error reading device info\n");`
	`2120`	`+ dev_dbg(&adev->dev, "Error reading _DEP device info\n");`
`2121`	`2121`	`continue;`
`2122`	`2122`	`}`
`2123`	`2123`