Zephyr integration: Use K_HEAP_MEM_POOL_SIZE

[jhedberg]

This define should be used instead of the Kconfig option to determine if the k_malloc family of functions are available.

[jhedberg]

See zephyrproject-rtos/zephyr#65908

[arnopo]

Could you confirm if it is compatible with legacy versions of Zephyr?

[jhedberg]

Could you confirm if it is compatible with legacy versions of Zephyr?

@arnopo no, older Zephyr versions don't have a K_HEAP_MEM_POOL_SIZE define. However, older versions of Zephyr also wouldn't reference a newer version of the libmetal tree through west.yml, so I don't think there should be any issue with that.

Btw, note that the discussion in zephyrproject-rtos/zephyr#65908 is still ongoing, so I would suggest to wait for a consensus to be reached there before merging this PR.

[jhedberg]

There was sufficient consensus in today's Arch WG meeting on how to proceed, that I think this can be merged now. After that I'll update the main tree PR west.yml to reference the right commit.

[arnopo]

There was sufficient consensus in today's Arch WG meeting on how to proceed, that I think this can be merged now. After that I'll update the main tree PR west.yml to reference the right commit.

The zephyrproject-rtos/zephyr#65523 PR would need to be addressed first. the rebase on the v2023.10 released is blocked due to a test that fails

[arnopo]

@jhedberg :
Please Could you send a PR to libmetal github?
The normal process is to integrate it in lib metal and then rebase this repo.
if something takes to much time in the libmetal review process we will merge this one to unblock #65908

[jhedberg]

@jhedberg : Please Could you send a PR to libmetal github? The normal process is to integrate it in lib metal and then rebase this repo. if something takes to much time in the libmetal review process we will merge this one to unblock #65908

Done: OpenAMP/libmetal#280

[arnopo]

@carlocaione : Please could you review OpenAMP/libmetal#280. Then I will merge it and create a PR to rebase the Zephyr libmetal module on top of.

@jhedberg : I assume that you will update the west.yml in zephyrproject-rtos/zephyr#65908 then to integrate it, right?

[jhedberg]

@jhedberg : I assume that you will update the west.yml in zephyrproject-rtos/zephyr#65908 then to integrate it, right?

Correct.

[arnopo]

@jhedberg : I created #25

[jhedberg]

Closing, now that this has been superseded by #25