From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Fri, 17 Aug 2012 01:32:38 +0200
Subject: [U-Boot] Early malloc() summary
In-Reply-To: <CALButCKOdX-XDunFDj9Os4udx0PKCrRpLxrquSqX2WQHA_rfpA@mail.gmail.com>
References: <CALButCL9btmPEW3Uiv9=tU4Yk7fdTUruzpCcVqMLynpzWv6Pug@mail.gmail.com>
	<201208161650.27985.marex@denx.de>
	<CALButCKOdX-XDunFDj9Os4udx0PKCrRpLxrquSqX2WQHA_rfpA@mail.gmail.com>
Message-ID: <201208170132.39232.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Graeme Russ,

> Hi Marek,
> 
> On Fri, Aug 17, 2012 at 12:50 AM, Marek Vasut <marex@denx.de> wrote:
> > Dear Graeme Russ,
> > 
> > [...]
> > 
> >> >>  2) We can pass a pointer to a driver_core struct (or whatever struct
> >> >>  it
> >> >>  
> >> >>     is that holds the 'reloc' helper function pointer). We can't pick
> >> >>     up misuse at compile time, but dm_malloc() can print a
> >> >>     meaningful message if it is called pre-relocation with no
> >> >>     relocation function. (We should add a flag to indicate that no
> >> >>     relocation helper is required which may be the case for very
> >> >>     simple drivers)
> >> > 
> >> > Yes, but it would prevent using dm_malloc(size_t size, driver *drv)
> >> > for one-time buffers inside helper functions - strdup() for instance,
> >> 
> >> Hmm, I hadn't thought of that
> > 
> > So, we're back to square 1 ?
> 
> Nope - just a slight tweak is all

What tweak ?

> Regards,
> 
> Graeme

Best regards,
Marek Vasut