[PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default

Adam Ford aford173 at gmail.com
Wed Mar 23 17:38:18 CET 2022


On Wed, Mar 23, 2022 at 11:27 AM Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Mar 23, 2022 at 01:23:29PM -0300, Fabio Estevam wrote:
> > On Wed, Mar 23, 2022 at 1:16 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > > It should probably also cover the rest of imx8, and how close to you
> > > think imx6 is?  And, git grep'ing I see a bunch of layerscape also
> > > overriding this with something larger.
> >
> > I am not comfortable in changing imx8qxp/imx8qm/layerscape as I cannot
> > test them.
>
> Well, this will be for -next, and we can get some more people on the
> patch list, and this seems to be an area where everyone is either:
> - Kicking the value up a bit for themselves
> - Having hard to figure out problems booting the platform because it's
>   too small a value until they see someone else picked a larger value.
>
> So lets raise these a bit and get some acks, please.

Tested-by: Adam Ford <aford173 at gmail.com> #imx8mm-beacon, imx8mn-beacon

In both cases, after 'make savedefconfig' the updated defconfig file
removed the entry to SYS_MALLOC_F_LEN since it was already the same
value.

Thanks for doing that!

adam

>
> --
> Tom


More information about the U-Boot mailing list