driver core: silence device link messages unless debugging

This change “driver core: silence device link messages unless debugging” in Linux kernel is authored by Jerome Brunet <jbrunet [at] baylibre.com> on Fri Dec 21 17:23:41 2018 +0100.

driver core: silence device link messages unless debugging

On platforms making a fair use of regulators, the dev_info() messages
coming from the device link function are a bit too verbose. The amount
of message will increase further with the clock framework joining the
device link party.

These messages looks valuable for people debugging device link related
issues, so dev_dbg() looks more appropriate than dev_info().

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

This Linux change may have been applied to various maintained Linux releases and you can find Linux releases including commit 8a4b326.

There are 6 lines of Linux source code added/deleted in this change. Code changes to Linux kernel are as follows.

 drivers/base/core.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/base/core.c b/drivers/base/core.c
index 4a4b6f8..4e57e16 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -302,7 +302,7 @@ struct device_link *device_link_add(struct device *consumer,
 	list_add_tail_rcu(&link->s_node, &supplier->links.consumers);
 	list_add_tail_rcu(&link->c_node, &consumer->links.suppliers);
 
-	dev_info(consumer, "Linked as a consumer to %sn", dev_name(supplier));
+	dev_dbg(consumer, "Linked as a consumer to %sn", dev_name(supplier));
 
  out:
 	device_pm_unlock();
@@ -328,8 +328,8 @@ static void __device_link_del(struct kref *kref)
 {
 	struct device_link *link = container_of(kref, struct device_link, kref);
 
-	dev_info(link->consumer, "Dropping the link to %sn",
-		 dev_name(link->supplier));
+	dev_dbg(link->consumer, "Dropping the link to %sn",
+		dev_name(link->supplier));
 
 	if (link->flags & DL_FLAG_PM_RUNTIME)
 		pm_runtime_drop_link(link->consumer);

The commit for this change in Linux stable tree is 8a4b326 (patch).

Leave a Reply

Your email address will not be published. Required fields are marked *