Skip to content

Commit

Permalink
imx8/ls10xx: Use a sane SYS_MALLOC_F_LEN default
Browse files Browse the repository at this point in the history
When adding new features to imx8m boards, such as DM clock support,
the malloc area can be exhausted.

To avoid such issue, provide a reasonable default for the
SYS_MALLOC_F_LEN size.

Quoting Tom Rini:

"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."

Suggested-by: Heinrich Schuchardt <[email protected]>
Signed-off-by: Fabio Estevam <[email protected]>
Tested-by: Adam Ford <[email protected]> #imx8mm-beacon, imx8mn-beacon
Tested-by: Heiko Thiery <[email protected]>
  • Loading branch information
Fabio Estevam authored and trini committed Apr 7, 2022
1 parent f75e516 commit 545eceb
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -249,10 +249,10 @@ config SYS_MALLOC_F_LEN
depends on SYS_MALLOC_F
default 0x1000 if AM33XX
default 0x4000 if SANDBOX || RISCV
default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
ARCH_LS1046A || ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
default 0x2000 if (ARCH_MX7 || ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
default 0x10000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_LS1012A || \
ARCH_LS1021A || ARCH_LS1043A || ARCH_LS1046A)
default 0x400
help
Before relocation, memory is very limited on many platforms. Still,
Expand Down

0 comments on commit 545eceb

Please sign in to comment.