Message ID | 20240223104721.4140880-1-m.felsch@pengutronix.de |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:a81b:b0:108:e6aa:91d0 with SMTP id bq27csp501645dyb; Fri, 23 Feb 2024 02:47:49 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWweITghqd0acrKzWvsudssBko2Lpi/GK52hPCvV0xmbQjAMiKWXyAoC9LsmU4IG8C64skRMVZbCmFVEjjKXgksZF9k0w== X-Google-Smtp-Source: AGHT+IGO++dIXFpwI1/QJiucZuxkGpLk7GH0QLlIglDVFxlIQ5rBZ1VqMquibtAKcjZo4MRjknxw X-Received: by 2002:a17:906:b09:b0:a3f:da63:49fb with SMTP id u9-20020a1709060b0900b00a3fda6349fbmr560011ejg.73.1708685268832; Fri, 23 Feb 2024 02:47:48 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708685268; cv=pass; d=google.com; s=arc-20160816; b=YMTASBBvKG1896EPWQSsiiVw36omd9w3AMRnPjAd9uzgSrw22kWkOg5QnYguS5qagC UE64KW8y83zRiMWx1M9Gv3xKDSGL8i/sZxRZzTBd5p8yKzk+XjKZdOdSIaeku8UZBX4f rXiXM5P5PwM5CgvcLDJQ14kmOdMxtisWLRUgB+jZLn8MUj2ixs3jmSDFsB0IOXJVVOPQ GzS1tRb/SYDoOgJhL9cKqJ1oiToOjUsoEr9SJlmWYPjYnVKgW9/G6o8tJA1ChfcPsKEw iviWiCKBAMcOXA8ljyvbki+GBdyFE1HBPfrCvviAN+6SpWo8ExBYQqP2/eY2kJtQ5CuR HqJA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from; bh=xwU5HziSB60v6uDoq6w9MpN2pQlRxsjaN7ZxYO9XDhI=; fh=9knZMCUc4usxSF8X5Ui1V4bpmVB0sC45rXxbtHzhAx4=; b=ANVZA/D3fKSFMCqBsZMD8lXkzgKTpsZ1yQeNxPUudBdH8HN97fd4hcuc7v85aMxI12 8QPzW1ukQ7IQguy3KS65XBaBYMeg/IuRMrgCa6wkbQ1uFwzYli2bim2bOh7aI3S+dYap twd9Noop6YOJUriyOdYFNogzBlxewrRGFdRbO3M2U4kEjNhoa4IC/sPoDMV6VBUnMFu7 UdT7kl/JgDJDN0XByGv9Wk4H2dxQIRM+gqaQHptFvCwSTn0wSwvNyqXx0KtYj53+HWSg iYguPgMfXE3i1p2GAQE9rwEmwk6fUqZBKB51b5+TnwvrLiNI86kg1qjhA4hDDlQy6Y4g Ws/Q==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=pengutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org" Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id he42-20020a1709073daa00b00a3e8ca48eadsi4826356ejc.166.2024.02.23.02.47.48 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Feb 2024 02:47:48 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=pengutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78174-ouuuleilei=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 6FBBE1F21BD6 for <ouuuleilei@gmail.com>; Fri, 23 Feb 2024 10:47:48 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8FD63657A8; Fri, 23 Feb 2024 10:47:37 +0000 (UTC) Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [185.203.201.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D15C8D29B for <linux-kernel@vger.kernel.org>; Fri, 23 Feb 2024 10:47:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.203.201.7 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708685256; cv=none; b=SgMwgve3f14eb+Jr5X3/e+j6JUy0U0QQiKtBz0wLqsEFV2LGzHzesCE+3V6UujPWrppVrJFH0xqz+tqZNZ7kZ53EFXCVsdyEV+/3J8LxnIaHNLK8yorTgVvnGHrOqjdBgV0wTxdGBFrVJZYDyay/IKoeTFUTXezcOo0jttSowyI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708685256; c=relaxed/simple; bh=s4MMU7+jsaZJjz9oik9m7Bh8k11l3m1ojNylen52K70=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=ViknMJTw5C2xKMI5Jjy4rgQ8Trc/fH0Me8Zq4Z4vu81+rIgxa/DfdH4RGtTQwUHA1T23SU2xwambPj0tkx8o2Kv1Xy47UlNDn0yI6Q/gnONDmYrHoMKBI7XMX/kkRoWMWmyd6uz7fcJ/HLNIN/CKlO3innD4jUeiK+RSyGisFbg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de; spf=pass smtp.mailfrom=pengutronix.de; arc=none smtp.client-ip=185.203.201.7 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de Received: from dude02.red.stw.pengutronix.de ([2a0a:edc0:0:1101:1d::28]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from <m.felsch@pengutronix.de>) id 1rdT5S-0007T7-5B; Fri, 23 Feb 2024 11:47:26 +0100 From: Marco Felsch <m.felsch@pengutronix.de> To: robh+dt@kernel.org, frowand.list@gmail.com Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@pengutronix.de Subject: [PATCH] of: property: lower loglevel of of_graph_get_next_endpoint Date: Fri, 23 Feb 2024 11:47:21 +0100 Message-Id: <20240223104721.4140880-1-m.felsch@pengutronix.de> X-Mailer: git-send-email 2.39.2 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 2a0a:edc0:0:1101:1d::28 X-SA-Exim-Mail-From: m.felsch@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791686364531342280 X-GMAIL-MSGID: 1791686364531342280 |
Series |
of: property: lower loglevel of of_graph_get_next_endpoint
|
|
Commit Message
Marco Felsch
Feb. 23, 2024, 10:47 a.m. UTC
Drivers like the tcpm.c do search for a remote endpoint on different
places to be dt-bindings compatible. The search is done on the device
itself or on the child fwnode in case it was not found the first time.
This indicates that not finding the remote endpoint at the first try is
a valid use-case and should not cause an error printing.
Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
Hi,
I'm not 100% certain if this is the correct place but if our platform
follows the dt-bindings we receive
| OF: graph: no port node found in /soc@0/bus@30800000/i2c@30a30000/tcpc@50
a few times because of the below pr_err() and EPROBE_DEFER.
Regards,
Marco
drivers/of/property.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Fri, Feb 23, 2024 at 11:47:21AM +0100, Marco Felsch wrote: > Drivers like the tcpm.c do search for a remote endpoint on different > places to be dt-bindings compatible. The search is done on the device > itself or on the child fwnode in case it was not found the first time. > > This indicates that not finding the remote endpoint at the first try is > a valid use-case and should not cause an error printing. > > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> > --- > Hi, > > I'm not 100% certain if this is the correct place but if our platform > follows the dt-bindings we receive > > | OF: graph: no port node found in /soc@0/bus@30800000/i2c@30a30000/tcpc@50 > > a few times because of the below pr_err() and EPROBE_DEFER. > > Regards, > Marco > > drivers/of/property.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/of/property.c b/drivers/of/property.c > index 641a40cf5cf3..155df04a9512 100644 > --- a/drivers/of/property.c > +++ b/drivers/of/property.c > @@ -665,7 +665,7 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent, > of_node_put(node); > > if (!port) { > - pr_err("graph: no port node found in %pOF\n", parent); > + pr_notice("graph: no port node found in %pOF\n", parent); Already changed to pr_debug. Rob
On 24-03-01, Rob Herring wrote: > On Fri, Feb 23, 2024 at 11:47:21AM +0100, Marco Felsch wrote: > > Drivers like the tcpm.c do search for a remote endpoint on different > > places to be dt-bindings compatible. The search is done on the device > > itself or on the child fwnode in case it was not found the first time. > > > > This indicates that not finding the remote endpoint at the first try is > > a valid use-case and should not cause an error printing. > > > > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> > > --- > > Hi, > > > > I'm not 100% certain if this is the correct place but if our platform > > follows the dt-bindings we receive > > > > | OF: graph: no port node found in /soc@0/bus@30800000/i2c@30a30000/tcpc@50 > > > > a few times because of the below pr_err() and EPROBE_DEFER. > > > > Regards, > > Marco > > > > drivers/of/property.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/of/property.c b/drivers/of/property.c > > index 641a40cf5cf3..155df04a9512 100644 > > --- a/drivers/of/property.c > > +++ b/drivers/of/property.c > > @@ -665,7 +665,7 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent, > > of_node_put(node); > > > > if (!port) { > > - pr_err("graph: no port node found in %pOF\n", parent); > > + pr_notice("graph: no port node found in %pOF\n", parent); > > Already changed to pr_debug. Ah.. didn't noticed that albeit I was sitting on rc4 while preparing this patch. Thanks for the info :) Regards, Marco > > Rob >
diff --git a/drivers/of/property.c b/drivers/of/property.c index 641a40cf5cf3..155df04a9512 100644 --- a/drivers/of/property.c +++ b/drivers/of/property.c @@ -665,7 +665,7 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent, of_node_put(node); if (!port) { - pr_err("graph: no port node found in %pOF\n", parent); + pr_notice("graph: no port node found in %pOF\n", parent); return NULL; } } else {