From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <46152D90.5070209@smiths-aerospace.com> Date: Thu, 05 Apr 2007 13:10:40 -0400 From: Jerry Van Baren MIME-Version: 1.0 To: Scott Wood Subject: Re: [PATCH dtc] Implement the -R option and add a -S option. References: <20070405020433.GA13891@dellserver.lan> <20070405161113.GA28063@ld0162-tx32.am.freescale.net> In-Reply-To: <20070405161113.GA28063@ld0162-tx32.am.freescale.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: Linuxppc-dev@ozlabs.org, jdl@jdl.com, Jerry Van Baren , david@gibson.dropbear.id.au List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Scott Wood wrote: > On Wed, Apr 04, 2007 at 10:04:33PM -0400, Jerry Van Baren wrote: >> Implement the -R option to add memory reserve slots. >> Add a -S option makes the blob at least this number of bytes. > > Wouldn't it be better to just specify the amount of extra space, instead > of the minimum total space? That way, you only need to know what you > intend to add, not how much is already there. > > -Scott I thought briefly about this, but decided to implement a fixed size so that someone could allocate, say, 8K of memory in their memory map (likely flash) and know their blob would fit. Maybe we need a little -s option to say "add -s bytes". Jon suggested a --stats option to print out the important statistics. That would also be a good enhancement. gvb