From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 25 May 2007 13:50:14 -0700 From: "Mark A. Greer" To: "Mark A. Greer" , Paul Mackerras , linuxppc-dev Subject: Re: [PATCH 1/13] powerpc: Add Makefile rules to wrap dts file in zImage Message-ID: <20070525205014.GC20831@mag.az.mvista.com> References: <20070512004605.GA2808@mag.az.mvista.com> <20070512005336.GA3365@mag.az.mvista.com> <20070514003332.GE6517@localhost.localdomain> <20070521181043.GB7120@mag.az.mvista.com> <20070522002349.GA25426@localhost.localdomain> <20070524212832.GB32012@mag.az.mvista.com> <20070525013242.GC11789@localhost.localdomain> <20070525013359.GD11789@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20070525013359.GD11789@localhost.localdomain> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, May 25, 2007 at 11:33:59AM +1000, David Gibson wrote: > On Fri, May 25, 2007 at 11:32:42AM +1000, David Gibson wrote: > > On Thu, May 24, 2007 at 02:28:32PM -0700, Mark A. Greer wrote: > > > On Tue, May 22, 2007 at 10:23:49AM +1000, David Gibson wrote: > > > > Just leave them out. As long as you add the correct specific > > > > zImage.dts.% and zImage.dts_initrd.% targets to image-y and initrd-y, > > > > then a "make zImage" from the top-level Makefile will get them built. > > > > > > That defeats the whole purpose of the patch--to allow the selection of > > > wrapping a dtb or not at build time. Its just like initrd, you choose > > > whether to "do it" at build time. > > > > Ah, right. Remind me why that's useful again? > > Sorry, to be more precise, why is that more useful than having "make > zImage" at the top level build both the dts and non-dts versions. Well, that is a good question. This is my reasoning which I put in the patch description: "Having separate rules allows the user to choose whether to include a device tree--and which device tree--at build time. This is useful when one Makefile target builds a zImage that runs on several platforms except for differing device trees. By just setting CONFIG_DEVICE_TREE and running "make zImage.dts" the exact zImage you want is built without Makefile bloat or manually running the wrapper script." Making both seems like a waste but its such a small waste that it doesn't really matter. If you still object to this feature, I'll take it out since I (read: prpmc2800) doesn't really need this feature anyway. Mark