Thread (15 messages) 15 messages, 5 authors, 2015-10-12

[PATCH 5/5] arm: add missing of_node_put

From: Julia Lawall <hidden>
Date: 2015-10-10 21:13:20
Also in: kernel-janitors, lkml

On Sat, 10 Oct 2015, Thomas Petazzoni wrote:
Arnd,

On Sat, 10 Oct 2015 23:02:15 +0200, Arnd Bergmann wrote:
quoted
The same for_each_child_of_node() loop has three 'return' statements'
aside from the 'break' statement here. I think you should change your
semantic patch to cover both cases.
I think Julia's semantic patch covers both cases, but only the cases
where there is one break or return (though I have essentially zero
Coccinelle knowledge, this is all based on guessing looking at the
semantic patch in the cover letter).
Normally, it should be OK with lots of returns.  And contrary to my 
previous email, even with return;.  Will check on it.

julia
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help