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.

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

SW Module HAL_PWM #8

Closed
pastorpflores opened this issue May 22, 2023 · 2 comments · Fixed by #45
Closed

SW Module HAL_PWM #8

pastorpflores opened this issue May 22, 2023 · 2 comments · Fixed by #45
Assignees
@RobertoAldea
Labels
good first issue Good for newcomers

Comments

@pastorpflores
Copy link
Member

pastorpflores commented May 22, 2023 •
edited
Loading

Module functionalities

  • Enables/disables PWM
  • Set duty
@pastorpflores pastorpflores added the good first issue Good for newcomers label May 23, 2023
@pastorpflores
Copy link
Member Author

Delays in Week 21. Therefore, moved to Week 23.

@pastorpflores pastorpflores linked a pull request Jun 5, 2023 that will close this issue
Add PWM configuration #45
Merged
@RobertoAldea
Copy link
Member

#5 @mariuscrsn Change interface of the module, its neccessary to use a uint32_t instead of a uint8_t, so the final interface will be as following:

HAL_PWM_result_e HAL_PwmInit(void)
HAL_PWM_result_e HAL_PwmSetDuty(const uint32_t duty)
HAL_PWM_result_e HAL_PwmStart (void)
HAL_PWM_result_e HAL_PwmStop (void)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RobertoAldea RobertoAldea

Labels
good first issue Good for newcomers
Projects
EPC - Electronic Power Converter for battery cycling
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add PWM configuration WattRex/Regenerative-Battery-Cycler
2 participants
@pastorpflores @RobertoAldea