Cleanup code and mark init pins argument as const (which it is)

This commit is contained in:
Andrej Krutak 2017-03-30 21:59:35 +02:00 committed by Andrej Krutak
parent 174dba8c8b
commit c03622871f
2 changed files with 7 additions and 7 deletions

View file

@ -29,7 +29,7 @@ typedef struct pwmInfoDefinition
uint8_t running; uint8_t running;
uint16_t freq; uint16_t freq;
uint16_t dutyCicle; uint16_t dutyCycle;
/* private */ /* private */
uint32_t _maxLoad; uint32_t _maxLoad;
@ -66,7 +66,7 @@ static void frc1_interrupt_handler(void)
pwmInfo._step = step; pwmInfo._step = step;
} }
void pwm_init(uint8_t npins, uint8_t* pins) void pwm_init(uint8_t npins, const uint8_t* pins)
{ {
/* Assert number of pins is correct */ /* Assert number of pins is correct */
if (npins > MAX_PWM_PINS) if (npins > MAX_PWM_PINS)
@ -127,7 +127,7 @@ void pwm_set_duty(uint16_t duty)
{ {
bool output; bool output;
pwmInfo.dutyCicle = duty; pwmInfo.dutyCycle = duty;
if (duty > 0 && duty < UINT16_MAX) { if (duty > 0 && duty < UINT16_MAX) {
pwm_restart(); pwm_restart();
return; return;
@ -154,7 +154,7 @@ void pwm_restart()
void pwm_start() void pwm_start()
{ {
pwmInfo._onLoad = pwmInfo.dutyCicle * pwmInfo._maxLoad / UINT16_MAX; pwmInfo._onLoad = pwmInfo.dutyCycle * pwmInfo._maxLoad / UINT16_MAX;
pwmInfo._offLoad = pwmInfo._maxLoad - pwmInfo._onLoad; pwmInfo._offLoad = pwmInfo._maxLoad - pwmInfo._onLoad;
pwmInfo._step = PERIOD_ON; pwmInfo._step = PERIOD_ON;

View file

@ -16,7 +16,7 @@
extern "C" { extern "C" {
#endif #endif
void pwm_init(uint8_t npins, uint8_t* pins); void pwm_init(uint8_t npins, const uint8_t* pins);
void pwm_set_freq(uint16_t freq); void pwm_set_freq(uint16_t freq);
void pwm_set_duty(uint16_t duty); void pwm_set_duty(uint16_t duty);