diff mbox series

of: base: improve error message in of_phandle_iterator_next()

Message ID 20210208165806.25466-1-info@metux.net
State Superseded, archived
Headers show
Series of: base: improve error message in of_phandle_iterator_next() | expand

Checks

Context Check Description
robh/checkpatch success

Commit Message

Enrico Weigelt, metux IT consult Feb. 8, 2021, 4:58 p.m. UTC
Also print out the phandle ID on error message, as a debug aid.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 drivers/of/base.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Rob Herring Feb. 11, 2021, 8:13 p.m. UTC | #1
On Mon, Feb 8, 2021 at 10:58 AM Enrico Weigelt, metux IT consult
<info@metux.net> wrote:
>
> Also print out the phandle ID on error message, as a debug aid.

I already have this patch applied in my tree. Why do you keep sending it?

Rob
Enrico Weigelt, metux IT consult Feb. 12, 2021, 10:19 a.m. UTC | #2
On 11.02.21 21:13, Rob Herring wrote:
> On Mon, Feb 8, 2021 at 10:58 AM Enrico Weigelt, metux IT consult
> <info@metux.net> wrote:
>>
>> Also print out the phandle ID on error message, as a debug aid.
> 
> I already have this patch applied in my tree. Why do you keep sending it?

Sorry, didn't know that when sending it the 2nd time, missed your last
mail in the mail flood :o

Seems my mail sorting rules are still quite imperfect. I suspect I'm not
the only one with those problems - how do you folks handle that ?


--mtx
diff mbox series

Patch

diff --git a/drivers/of/base.c b/drivers/of/base.c
index 161a23631472..8a348f0d3c5e 100644
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -1297,8 +1297,8 @@  int of_phandle_iterator_next(struct of_phandle_iterator *it)
 
 		if (it->cells_name) {
 			if (!it->node) {
-				pr_err("%pOF: could not find phandle\n",
-				       it->parent);
+				pr_err("%pOF: could not find phandle %d\n",
+				       it->parent, it->phandle);
 				goto err;
 			}