Re: [PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Daney
On 04/08/2016 01:15 PM, David Miller wrote: From: David Daney Date: Fri, 8 Apr 2016 09:41:35 -0700 Due to mail server malfunction, this patch was sent twice. Please ignore this duplicate. This submission had another problem too. Do not use the date of your commit as the date that gets put

[PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Daney
From: David Daney commit b7d3e3d3d21a ("net: thunderx: Don't leak phy device references on -EPROBE_DEFER condition.") incorrectly moved the call to of_node_put() outside of the loop. Under normal loop exit, the node has already had of_node_put() called, so the extra call results in: [8.2280

Re: [PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Miller
From: David Daney Date: Fri, 8 Apr 2016 09:41:35 -0700 > Due to mail server malfunction, this patch was sent twice. Please > ignore this duplicate. This submission had another problem too. Do not use the date of your commit as the date that gets put into your email headers. This makes all of

Re: [PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Daney
Due to mail server malfunction, this patch was sent twice. Please ignore this duplicate. Thanks, David Daney On 03/31/2016 06:01 PM, David Daney wrote: From: David Daney commit b7d3e3d3d21a ("net: thunderx: Don't leak phy device references on -EPROBE_DEFER condition.") incorrectly moved th

[PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Daney
From: David Daney commit b7d3e3d3d21a ("net: thunderx: Don't leak phy device references on -EPROBE_DEFER condition.") incorrectly moved the call to of_node_put() outside of the loop. Under normal loop exit, the node has already had of_node_put() called, so the extra call results in: [8.2280

[PATCH] net: thunderx: Fix broken of_node_put() code.

2016-04-08 Thread David Daney
From: David Daney commit b7d3e3d3d21a ("net: thunderx: Don't leak phy device references on -EPROBE_DEFER condition.") incorrectly moved the call to of_node_put() outside of the loop. Under normal loop exit, the node has already had of_node_put() called, so the extra call results in: [8.2280