From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from zipcode.az.mvista.com (unknown [65.200.49.156]) by ozlabs.org (Postfix) with ESMTP id DADD6DDF08 for ; Wed, 16 May 2007 04:11:35 +1000 (EST) Date: Tue, 15 May 2007 11:11:40 -0700 From: "Mark A. Greer" To: Scott Wood , Paul Mackerras , "Mark A. Greer" , linuxppc-dev@ozlabs.org Subject: Re: Fix bug adding properties with flatdevtree.c's ft_set_prop() Message-ID: <20070515181140.GD17643@mag.az.mvista.com> References: <20070514031357.GM6517@localhost.localdomain> <20070514162816.GB371@ld0162-tx32.am.freescale.net> <20070515002648.GA31585@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20070515002648.GA31585@localhost.localdomain> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, May 15, 2007 at 10:26:48AM +1000, David Gibson wrote: > On Mon, May 14, 2007 at 11:28:16AM -0500, Scott Wood wrote: > > On Mon, May 14, 2007 at 01:13:57PM +1000, David Gibson wrote: > > > This patch corrects ft_set_prop() so that it adds new properties > > > before the first subnode, instead of before the END_NODE tag. > > > > Is there any reason not to just insert at the beginning of the node? > > Only that we have to search through the node first to see if the > property already exists, so we've already reached the end by the time > we know we need to insert. David, Jon, Does dtc guarantee that there aren't any properties after a subnode? Mark