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

Refactoring/exit startup power #173

Merged
merged 1 commit into from
Oct 5, 2023
Merged

Conversation

damosvil
Copy link
Contributor

Refactored exit_startup_power to improve coherency.
Updated switch_power_off to also set Pwm_Limits to zero to enforce shut down
Renamed Pwm_Limit to Pwm_Limit_Startup_n_Temp to make more clear the use of this variable

@damosvil damosvil added the enhancement New feature or request label Sep 23, 2023
@damosvil damosvil added this to the v0.21.0 milestone Sep 23, 2023
@damosvil damosvil self-assigned this Sep 23, 2023
@damosvil damosvil changed the base branch from main to v0.21 September 23, 2023 19:07
@damosvil damosvil linked an issue Sep 23, 2023 that may be closed by this pull request
@github-actions
Copy link

Build artifacts:

src/Bluejay.asm Outdated Show resolved Hide resolved
src/Modules/Isrs.asm Outdated Show resolved Hide resolved
src/Modules/Power.asm Outdated Show resolved Hide resolved
@damosvil damosvil requested a review from stylesuxx September 24, 2023 07:13
src/Bluejay.asm Outdated Show resolved Hide resolved
@@ -1068,9 +1068,6 @@ ENDIF
ljmp motor_start ; Go back and try starting motors again

exit_run_mode_is_stall:
; Enable all interrupts (disabled above, in exit_run_mode)
setb IE_EA
Copy link
Contributor

@hyp0dermik-code hyp0dermik-code Sep 29, 2023

Choose a reason for hiding this comment

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

Can the tidyup of interrupt handling be included in the comment message too please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In principle that code has been removed. So you mean to the commit message?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes my apologies

Copy link
Contributor Author

@damosvil damosvil Oct 4, 2023

Choose a reason for hiding this comment

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

Done: added interrupt handling explanation in commit comment.
I also squashed the commits in the branch to keep a single comment.
imagen

…sabled when entering the function and enabled again when returning.

Updated switch_power_off to also set Pwm_Limits to zero to enforce shut down
Renamed Pwm_Limit to Pwm_Limit_Startup_n_Temp to make more clear the use of this variable
Fixed review findings
@damosvil damosvil force-pushed the refactoring/exit_startup_power branch from 046e949 to 303f638 Compare October 4, 2023 16:59
@stylesuxx stylesuxx merged commit de87570 into v0.21 Oct 5, 2023
@stylesuxx stylesuxx deleted the refactoring/exit_startup_power branch July 23, 2024 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve stall protection
3 participants