From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from AGRXSUSMAILB.smiths.aero (host241-chi.smiths-group.com [65.216.75.241]) by ozlabs.org (Postfix) with ESMTP id 7963DDE0DB for ; Thu, 25 Jan 2007 09:02:08 +1100 (EST) Message-ID: <45B7D044.7020007@smiths-aerospace.com> Date: Wed, 24 Jan 2007 16:31:48 -0500 From: Jerry Van Baren MIME-Version: 1.0 To: Scott Wood Subject: Re: [PATCH 05/18] Add ft_first_node(). References: <20070124210707.GE10527@ld0162-tx32.am.freescale.net> In-Reply-To: <20070124210707.GE10527@ld0162-tx32.am.freescale.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Scott Wood wrote: > Clean up some of the open-coded data structure references by providing a > function to return a pointer to the first node in the tree. This is only > used in high-level functions trying to access the beginning of the tree, > not in low-level code that is actually manipulating the data structure. > > Signed-off-by: Scott Wood > --- > arch/powerpc/boot/flatdevtree.c | 13 +++++++++---- > 1 files changed, 9 insertions(+), 4 deletions(-) Hi Scott, Have you considered replacing flatdevtree.c with David Gibson's fdtlib? The interface is a lot cleaner, which likely carries through to the underlying code. Best regards, gvb