Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Split reboot reasons due to wakeup #254

Merged
merged 5 commits into from
Feb 25, 2024

Conversation

cap9qd
Copy link
Contributor

@cap9qd cap9qd commented Feb 21, 2024

Per #234

Tested on several BK7231N devices with ESPHome 2024.1.0-dev

cores/common/base/api/lt_device.h Outdated Show resolved Hide resolved
cores/common/base/api/lt_device.h Show resolved Hide resolved
cores/common/base/api/lt_device.h Outdated Show resolved Hide resolved
cores/common/base/api/lt_device.h Outdated Show resolved Hide resolved
@kuba2k2
Copy link
Member

kuba2k2 commented Feb 22, 2024

clang-format needs fixes here:
https://github.com/libretiny-eu/libretiny/actions/runs/7991708089/job/21879639566?pr=254

@cap9qd
Copy link
Contributor Author

cap9qd commented Feb 23, 2024

clang-format needs fixes here: https://github.com/libretiny-eu/libretiny/actions/runs/7991708089/job/21879639566?pr=254

I have the clang-format fixed. The python one still seems to fail but I didn't change anything there...?

EDIT: Fixed the python files to pass black.

@cap9qd cap9qd requested a review from kuba2k2 February 23, 2024 16:09
@kuba2k2 kuba2k2 changed the title Updates to break out wakeup reasons. [core] Split reboot reasons due to wakeup Feb 25, 2024
@kuba2k2 kuba2k2 merged commit cf52021 into libretiny-eu:master Feb 25, 2024
2 checks passed
@kuba2k2 kuba2k2 added enhancement New feature or request quality Code quality or safety improvements labels Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request quality Code quality or safety improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants