* [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
@ 2022-03-23 16:10 Fabio Estevam
2022-03-23 16:16 ` Tom Rini
2022-03-24 7:51 ` Heiko Thiery
0 siblings, 2 replies; 7+ messages in thread
From: Fabio Estevam @ 2022-03-23 16:10 UTC (permalink / raw)
To: sbabic; +Cc: u-boot, heiko.thiery, xypron.glpk, trini, aford173, Fabio Estevam
From: Fabio Estevam <festevam@denx.de>
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.
Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
Changes since v1:
- Add the imx8m default inside the main Kconfig - Heinrich
Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Kconfig b/Kconfig
index 9dd9ec7f6dfb..060e327e30c6 100644
--- a/Kconfig
+++ b/Kconfig
@@ -232,10 +232,11 @@ config SYS_MALLOC_F_LEN
depends on SYS_MALLOC_F
default 0x1000 if AM33XX
default 0x4000 if SANDBOX
- default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
+ default 0x2000 if (ARCH_IMX8 || ARCH_MX7 || \
ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
ARCH_LS1046A || ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
+ default 0x10000 if ARCH_IMX8M
default 0x400
help
Before relocation, memory is very limited on many platforms. Still,
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:10 [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default Fabio Estevam
@ 2022-03-23 16:16 ` Tom Rini
2022-03-23 16:23 ` Fabio Estevam
2022-03-24 7:51 ` Heiko Thiery
1 sibling, 1 reply; 7+ messages in thread
From: Tom Rini @ 2022-03-23 16:16 UTC (permalink / raw)
To: Fabio Estevam
Cc: sbabic, u-boot, heiko.thiery, xypron.glpk, aford173, Fabio Estevam
[-- Attachment #1: Type: text/plain, Size: 1358 bytes --]
On Wed, Mar 23, 2022 at 01:10:20PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> 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.
>
> Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> Changes since v1:
> - Add the imx8m default inside the main Kconfig - Heinrich
>
> Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Kconfig b/Kconfig
> index 9dd9ec7f6dfb..060e327e30c6 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -232,10 +232,11 @@ config SYS_MALLOC_F_LEN
> depends on SYS_MALLOC_F
> default 0x1000 if AM33XX
> default 0x4000 if SANDBOX
> - default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
> + default 0x2000 if (ARCH_IMX8 || ARCH_MX7 || \
> ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
> ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
> ARCH_LS1046A || ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
> + default 0x10000 if ARCH_IMX8M
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.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:16 ` Tom Rini
@ 2022-03-23 16:23 ` Fabio Estevam
2022-03-23 16:27 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Fabio Estevam @ 2022-03-23 16:23 UTC (permalink / raw)
To: Tom Rini
Cc: Stefano Babic, U-Boot-Denx, Heiko Thiery, Heinrich Schuchardt,
Adam Ford, Fabio Estevam
On Wed, Mar 23, 2022 at 1:16 PM Tom Rini <trini@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.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:23 ` Fabio Estevam
@ 2022-03-23 16:27 ` Tom Rini
2022-03-23 16:38 ` Adam Ford
2022-03-28 13:56 ` Fabio Estevam
0 siblings, 2 replies; 7+ messages in thread
From: Tom Rini @ 2022-03-23 16:27 UTC (permalink / raw)
To: Fabio Estevam
Cc: Stefano Babic, U-Boot-Denx, Heiko Thiery, Heinrich Schuchardt,
Adam Ford, Fabio Estevam
[-- Attachment #1: Type: text/plain, Size: 819 bytes --]
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@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.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:27 ` Tom Rini
@ 2022-03-23 16:38 ` Adam Ford
2022-03-28 13:56 ` Fabio Estevam
1 sibling, 0 replies; 7+ messages in thread
From: Adam Ford @ 2022-03-23 16:38 UTC (permalink / raw)
To: Tom Rini
Cc: Fabio Estevam, Stefano Babic, U-Boot-Denx, Heiko Thiery,
Heinrich Schuchardt, Fabio Estevam
On Wed, Mar 23, 2022 at 11:27 AM Tom Rini <trini@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@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@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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:10 [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default Fabio Estevam
2022-03-23 16:16 ` Tom Rini
@ 2022-03-24 7:51 ` Heiko Thiery
1 sibling, 0 replies; 7+ messages in thread
From: Heiko Thiery @ 2022-03-24 7:51 UTC (permalink / raw)
To: Fabio Estevam; +Cc: sbabic, u-boot, xypron.glpk, trini, aford173, Fabio Estevam
Hi Fabio,
Am Mi., 23. März 2022 um 17:10 Uhr schrieb Fabio Estevam <festevam@gmail.com>:
>
> From: Fabio Estevam <festevam@denx.de>
>
> 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.
>
> Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
I tested this on an imx8mq board (kontron_pitx_imx8m_defconfig).
Tested-by: Heiko Thiery <heiko.thiery@gmail.com>
> ---
> Changes since v1:
> - Add the imx8m default inside the main Kconfig - Heinrich
>
> Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Kconfig b/Kconfig
> index 9dd9ec7f6dfb..060e327e30c6 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -232,10 +232,11 @@ config SYS_MALLOC_F_LEN
> depends on SYS_MALLOC_F
> default 0x1000 if AM33XX
> default 0x4000 if SANDBOX
> - default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
> + default 0x2000 if (ARCH_IMX8 || ARCH_MX7 || \
> ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
> ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
> ARCH_LS1046A || ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
> + default 0x10000 if ARCH_IMX8M
> default 0x400
> help
> Before relocation, memory is very limited on many platforms. Still,
> --
> 2.25.1
>
--
Heiko
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default
2022-03-23 16:27 ` Tom Rini
2022-03-23 16:38 ` Adam Ford
@ 2022-03-28 13:56 ` Fabio Estevam
1 sibling, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2022-03-28 13:56 UTC (permalink / raw)
To: Tom Rini
Cc: Stefano Babic, U-Boot-Denx, Heiko Thiery, Heinrich Schuchardt,
Adam Ford, Fabio Estevam
Hi Tom,
On Wed, Mar 23, 2022 at 1:27 PM Tom Rini <trini@konsulko.com> wrote:
> 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.
Sure, I sent a v3 as requested.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-03-28 13:56 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-23 16:10 [PATCH v2] imx8m: Use a sane SYS_MALLOC_F_LEN default Fabio Estevam
2022-03-23 16:16 ` Tom Rini
2022-03-23 16:23 ` Fabio Estevam
2022-03-23 16:27 ` Tom Rini
2022-03-23 16:38 ` Adam Ford
2022-03-28 13:56 ` Fabio Estevam
2022-03-24 7:51 ` Heiko Thiery
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.