cmake: DTS post-processing of ELF file depends on CONFIG_HAS_DTS

[mtpr-ot]

Only run gen_handles.py if CONFIG_HAS_DTS is set. This enables vendors
without device tree implementations to compile without the generated
dependencies.

Fixes #34416.

Signed-off-by: Morten Priess mtpr@oticon.com

[mtpr-ot]

Closed/reopened to rerun CI

[mtpr-ot]

Can anybody help with the CI errors? Is it because some configurations are using DTS, but somehow set CONFIG_HAS_DTS=n?

[tejlmand]

Can anybody help with the CI errors? Is it because some configurations are using DTS, but somehow set CONFIG_HAS_DTS=n?

@pabigot can you assist ?

[pabigot]

Can anybody help with the CI errors? Is it because some configurations are using DTS, but somehow set CONFIG_HAS_DTS=n?

@pabigot can you assist ?

In arch/Kconfig the SPARC architecture doesn't have select HAS_DTS. Adding a commit that "fixes" that would probably make it work. Not sure why it isn't there @martin-aberg

I suspect platforms that don't have devicetree aren't going to make it very far in the Zephyr of the future. @galak @mbolivar-nordic FYI in case this is something to consider from an architectural level.

[mtpr-ot]

In arch/Kconfig the SPARC architecture doesn't have select HAS_DTS. Adding a commit that "fixes" that would probably make it work. Not sure why it isn't there @martin-aberg

Thanks! I will add a commit with that selected.

[mtpr-ot]

I suspect platforms that don't have devicetree aren't going to make it very far in the Zephyr of the future

I can understand that from a Zephyr architectural perspective. I will inform my colleagues of the deprecation, and hopefully we can go the DTS-way in the next generation.

[mbolivar-nordic]

I suspect platforms that don't have devicetree aren't going to make it very far in the Zephyr of the future. @galak @mbolivar-nordic FYI in case this is something to consider from an architectural level.

That is my general impression as well, as documented in https://docs.zephyrproject.org/latest/guides/dts/design.html#single-source-for-all-hardware-information though this is ongoing work.

Basic things like "be able to create a struct device that isn't from devicetree" will continue to work "forever", though, as far as I know. I never use platforms without DTS though, not sure if we've got project level decisions made on the future of CONFIG_HAS_DTS=n.

[ghost]

@pabigot Thank you for the heads-up regarding the missing HAS_DTS.

@mtpr-ot Thank you for the fix.

[mtpr-ot]

@pabigot, so is this acceptable for the time being?

[pabigot]

@pabigot, so is this acceptable for the time being?

I don't like it, because it seems clear DTS is not an optional part of Zephyr, but until that's written into policy this seems the only way to make progress. Presumably some day somebody will get rid of CONFIG_HAS_DTS entirely and the hack will be removed at that point.