Skip to content

xtensa-build-zephyr.py: cosmetic: rename "overlays" variable#9435

Merged
lgirdwood merged 2 commits intothesofproject:mainfrom
marc-hb:rename-overlays
Sep 9, 2024
Merged

xtensa-build-zephyr.py: cosmetic: rename "overlays" variable#9435
lgirdwood merged 2 commits intothesofproject:mainfrom
marc-hb:rename-overlays

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Sep 4, 2024

The Zephyr documentation uses the term "overlay" for Device Tree, not for Kconfig.

This commit does not change the wrongly named --overlay option not to break anything but it stops that wrong name from leaking all over the code. Also tweak the --help text.

@marc-hb marc-hb marked this pull request as ready for review September 4, 2024 07:29
@marc-hb marc-hb marked this pull request as draft September 6, 2024 16:24
@marc-hb

This comment was marked as resolved.

The Zephyr documentation uses the term "overlay" for Device Tree, not
for Kconfig.

This commit does not change the wrongly named --overlay option not to
break anything but it stops that wrong name from leaking all over the
code. Also tweak the --help text.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Zephyr does not call extra configs "overlays", it uses overlays only for
Device Tree.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb marked this pull request as ready for review September 6, 2024 17:07
@marc-hb
Copy link
Collaborator Author

marc-hb commented Sep 6, 2024

Usual LNL failures in https://sof-ci.01.org/sofpr/PR9435/build7759/devicetest/index.html, everything else green.

@lgirdwood lgirdwood merged commit 76c976f into thesofproject:main Sep 9, 2024
@marc-hb marc-hb deleted the rename-overlays branch September 10, 2024 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants