Re: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-17 Thread Grant Likely
On Wed, Apr 17, 2013 at 10:52 PM, Timur Tabi wrote: > On Wed, Apr 17, 2013 at 9:57 AM, Grant Likely > wrote: >> >>> I really preciate if you can spend some times to review this patch. >> >> Applied, thanks. > > Pff. Why do I bother? Relax Timur: http://git.secretlab.ca/?p=linux.git;a=commitdi

Re: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-17 Thread Timur Tabi
On Wed, Apr 17, 2013 at 9:57 AM, Grant Likely wrote: > >> I really preciate if you can spend some times to review this patch. > > Applied, thanks. Pff. Why do I bother? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.or

RE: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-17 Thread Grant Likely
On Tue, 16 Apr 2013 06:54:40 +, Tang Yuantian-B29983 wrote: > Hi Grant.likely, > > I really preciate if you can spend some times to review this patch. Applied, thanks. g. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.k

Re: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-17 Thread Timur Tabi
Tang Yuantian-B29983 wrote: It was placed on ELSE statement originally, I moved it to IF statement. Why is it so wrong? Because the logic of the comment applies to the else-condition, not the if-condtion. -- Timur Tabi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel

RE: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-16 Thread Tang Yuantian-B29983
> -Original Message- > From: Timur Tabi [mailto:ti...@tabi.org] > Sent: 2013年4月17日 11:31 > To: Tang Yuantian-B29983 > Cc: Grant Likely; devicetree-discuss; linuxppc-...@lists.ozlabs.org; lkml; > Rob Herring > Subject: Re: [PATCH v2] of/base: release

Re: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-16 Thread Timur Tabi
Tang Yuantian-B29983 wrote: >On Tue, Apr 9, 2013 at 10:36 PM, wrote: > > > >+ /* Found it! return success */ > >I'm pretty sure this comment is in the wrong place. It is not perfect, but acceptable. Like I said, I'm pretty sure it's in the wrong place. -- Timur Tabi

RE: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-16 Thread Tang Yuantian-B29983
> -Original Message- > From: Timur Tabi [mailto:ti...@tabi.org] > Sent: 2013年4月16日 19:37 > To: Tang Yuantian-B29983 > Cc: Grant Likely; devicetree-discuss; linuxppc-...@lists.ozlabs.org; lkml; > Rob Herring > Subject: Re: [PATCH v2] of/base: release

Re: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-16 Thread Timur Tabi
On Tue, Apr 9, 2013 at 10:36 PM, wrote: > > + /* Found it! return success */ I'm pretty sure this comment is in the wrong place. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo i

RE: [PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-15 Thread Tang Yuantian-B29983
labs.org; linux- > ker...@vger.kernel.org; linuxppc-...@lists.ozlabs.org; Tang Yuantian- > B29983; Tang Yuantian-B29983 > Subject: [PATCH v2] of/base: release the node correctly in > of_parse_phandle_with_args() > > From: Tang Yuantian > > Call of_node_put() only when the out_a

[PATCH v2] of/base: release the node correctly in of_parse_phandle_with_args()

2013-04-09 Thread Yuantian.Tang
From: Tang Yuantian Call of_node_put() only when the out_args is NULL on success, or the node's reference count will not be correct because the caller will call of_node_put() again. Signed-off-by: Tang Yuantian --- v2: - modified the title and description. the 1st patch title is: