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