Hi Sughosh,

On Mon, 29 Jul 2024 at 01:46, Sughosh Ganu <[email protected]> wrote:
>
> hi Simon,
>
> On Fri, 26 Jul 2024 at 05:02, Simon Glass <[email protected]> wrote:
> >
> > Hi Sughosh,
> >
> > On Wed, 24 Jul 2024 at 00:03, Sughosh Ganu <[email protected]> wrote:
> > >
> > > If the malloc simple functionality is enabled in SPL, it is not
> > > possible to call the full-implementation alloc functions even after
> > > the heap is set-up in ram memory. Check for this condition and
> > > call the functions when enabled.
> >
> > Is this because you want to use lmb in SPL. Is that needed?
>
> Yes, there was a discussion on this, and Tom [1], and you also [2] had
> mentioned on the earlier series that we do need lmb in SPL. You had
> mentioned that this would subsequently be needed for VPL too.

OK, well at least this way we can make sure it is disabled if not
needed. We can worry about how to pass information through the phases,
later.


>
> -sughosh
>
> [1] - https://lists.denx.de/pipermail/u-boot/2024-July/558250.html
> [2] - https://lists.denx.de/pipermail/u-boot/2024-July/558644.html
>
> >
> > BTW I'll send a patch to allow alist to run without realloc().
> >
> > >
> > > Signed-off-by: Sughosh Ganu <[email protected]>
> > > ---
> > > Changes since rfc: New patch
> > >
> > >  common/malloc_simple.c | 13 +++++++++++++
> > >  1 file changed, 13 insertions(+)

Regards,
Simon

Reply via email to