Hi Sughosh,

On Mon, 29 Jul 2024 at 01:46, Sughosh Ganu <sughosh.g...@linaro.org> wrote:
>
> hi Simon,
>
> On Fri, 26 Jul 2024 at 05:02, Simon Glass <s...@chromium.org> wrote:
> >
> > Hi Sughosh,
> >
> > On Wed, 24 Jul 2024 at 00:03, Sughosh Ganu <sughosh.g...@linaro.org> 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 <sughosh.g...@linaro.org>
> > > ---
> > > Changes since rfc: New patch
> > >
> > >  common/malloc_simple.c | 13 +++++++++++++
> > >  1 file changed, 13 insertions(+)

Regards,
Simon

Reply via email to