Skip to content

RP2040 driver improvements, StepDirListener fix #209

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Aug 27, 2022
Merged

RP2040 driver improvements, StepDirListener fix #209

merged 10 commits into from
Aug 27, 2022

Conversation

runger1101001
Copy link
Member

@runger1101001 runger1101001 commented Aug 20, 2022 •
edited
Loading

Several improvements:

  • RP2040 driver improvements: sets a default PWM frequency (24kHz) if input frequency is NOT_SET
  • RP2040 driver improvements: minimum allowed frequency lowered to 5kHz (formerly 20kHz, for no reason)
  • RP2040 driver improvements: introduce 3 build-flags to control the PWM polarity:
    • SIMPLEFOC_PWM_ACTIVE_HIGH (for all PWM modes except 6-PWM)
    • SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH (for 6-PWM, high side FETs)
    • SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH (for 6-PWM, low side FETs)
  • StepDirListener: change to listen on RISING edge by default, remove double-checking code
    Fix for: Interrupt code timing bug - Encoder, StepDirListener #169
  • StepDirListener: introduce field "polarity" in class, to control whether RISING or FALLING edge triggers interrupt

@runger1101001 runger1101001 changed the title Dev RP2040 driver improvements, StepDirListener fix Aug 20, 2022
@runger1101001 runger1101001 merged commit c9fd8d0 into simplefoc:dev Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@runger1101001