Skip to content

[microTVM] [Fix] reboot include for Zephyr version >=2.6.0#11790

Merged
mehrdadh merged 2 commits intoapache:mainfrom
UlrikHjort:zephyr_reboot_fix
Jun 21, 2022
Merged

[microTVM] [Fix] reboot include for Zephyr version >=2.6.0#11790
mehrdadh merged 2 commits intoapache:mainfrom
UlrikHjort:zephyr_reboot_fix

Conversation

@UlrikHjort
Copy link
Contributor

@UlrikHjort UlrikHjort commented Jun 20, 2022

This is a proposal to fix the changes of header reboot file locations in zephyr version 2.6.0.
Reboot functionality has been moved to subsys/os from subsys/power. A consequence of this movement is that the <power/reboot.h> header has been moved to <sys/reboot.h>. <power/reboot.h> is still provided for compatibility, but it will produce a warning to inform users of the relocation.

Since the microtvm demo zephyr dependency is 2.7.0 this should not be a breaking changes. Problem is that from zephyr version 3 there is no more a relocation warning and compilation will fail with old location of reboot.h

FYI @mehrdadh

cc @alanmacd @gromero

@UlrikHjort UlrikHjort changed the title Fix reboot include for Zephyr version >=2.6.0 [microTVM] [Fix] reboot include for Zephyr version >=2.6.0 Jun 21, 2022
Copy link
Member

@mehrdadh mehrdadh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@mehrdadh mehrdadh merged commit d4ae313 into apache:main Jun 21, 2022
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
)

* Fix reboot include for Zephyr version >=2.6.0
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.

2 participants