Hi,

On 09/08/2017 11:28 AM, Robert Hartung wrote:
> Looks like it's not that easy. Many platforms define pm_reboot in the
> board's file(s).

Only mips-malta has it's own "pm_reboot()" implementation. The other two
define stubs.

> Additionally pm_layered does not define pm_reboot, the same applies for
> pm_off (pm_off can be modeled as pm_set_lowest(); irq_disable();
> while(1) in pm_layered I guess ?).

pm_layered does define pm_set_lowest() as weak exactly like that.

> Therefore I will work on removing pm_reboot() from pm_fallback
> implementation and create additional modules if needed (at some points
> pm_reboot is defined outside of pm anyway).

When designing periph/pm, we intentionally moved reboot from a core
include into periph/pm, as it seemed to fit together with pm_off().

Do you have a WIP branch somewhere? While working on #7241, I had to
implement a lot of what we've discussed, in order to make anything
compile with sumbodulized periph. Maybe you can take a look? the
requirements have changed a little.

Kaspar
_______________________________________________
devel mailing list
[email protected]
https://lists.riot-os.org/mailman/listinfo/devel

Reply via email to