From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 2 Aug 2006 10:23:11 -0700 From: Tom Rini To: "Mark A. Greer" Subject: Re: [PATCH 3/6] bootwrapper: Add device tree ops for flattened device tree Message-ID: <20060802172311.GM3075@smtp.west.cox.net> References: <20060719230544.GD3887@mag.az.mvista.com> <20060802161054.GG3075@smtp.west.cox.net> <20060802170514.GE17652@mag.az.mvista.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20060802170514.GE17652@mag.az.mvista.com> Cc: linuxppc-dev List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, Aug 02, 2006 at 10:05:14AM -0700, Mark A. Greer wrote: > On Wed, Aug 02, 2006 at 09:10:54AM -0700, Tom Rini wrote: > > On Wed, Jul 19, 2006 at 04:05:44PM -0700, Mark A. Greer wrote: > > > > > This patch adds the device tree operations (dt_ops) for a flattened > > > device tree (fdt). > > [snip] > > > +/* Definitions used by the flattened device tree */ > > [snip] > > ... > > > +struct fdt_bus { > > > + u64 (*map)(u32 *addr, u32 *range, int na, int ns, int pna); > > > + int (*translate)(u32 *addr, u64 offset, int na); > > > +}; > > > > All of that should live in an fdt.h. > > Why? Its never used outside of fdt.c and therefore should not be in > fdt.h. Meh, I'll drop it then.. -- Tom Rini