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
os/board/rtl8730e: Prepare PM for GPIO #6175
base: master
Are you sure you want to change the base?
Conversation
edwakuwaku
commented
May 7, 2024
- Prepare PM for GPIO
ba9611a
to
7c6b0b1
Compare
- Prepare PM for GPIO
os/pm/pm_wakehandler.c
Outdated
@@ -55,7 +55,7 @@ void pm_wakehandler(clock_t missing_tick) | |||
{ | |||
irqstate_t flags = enter_critical_section(); | |||
|
|||
pmllvdbg("missing_tick: %d\n", missing_tick); | |||
pmllvdbg("missing_tick: %lld\n", missing_tick); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ewoodev , the clock_t is using u64, %d is printing wrong value, thus I changed to %lld here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for change edwakuwaku
this value is usinged long long, so Could you please apply %llu
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi EunWoo님, thanks for reminding, changed accordingly~
Hello @ewoodev , @ritesh55555 , other than enabling GPIO as wakeup src, this PR also added informational log to show the wakeup src. For wifi-keepalive, it will also show the packet type received that triggers wakeup. |
build/tools/amebasmart/gnu_utility/km0_km4_app_RELEASE_NOTE.txt
Outdated
Show resolved
Hide resolved
- Enable GPIO channel A as wakeup interrupt src
- Remove some logs, change tick printing to %llu because we are using u64 for clock_t