Skip to content

Commit 57be301

Browse files
author
Richard Unger
committed
hardware 6-PWM is working
1 parent 787cb15 commit 57be301

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/drivers/hardware_specific/renesas/renesas.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ bool configureTimerPin(RenesasHardwareDriverParams* params, uint8_t index, bool
137137
TimerPWMChannel_t pwm_output = IS_PWM_ON_A(pinCfgs[0]) ? CHANNEL_A : CHANNEL_B;
138138
if (complementary) {
139139
TimerPWMChannel_t pwm_output_C = IS_PWM_ON_A(pinCfgs_C[0]) ? CHANNEL_A : CHANNEL_B;
140-
if (pwm_output != CHANNEL_A || pwm_output_C != CHANNEL_B) {
140+
if (pwm_output_C != CHANNEL_A || pwm_output != CHANNEL_B) {
141141
SIMPLEFOC_DEBUG("DRV: output A/B mismatch");
142142
return false;
143143
}
@@ -373,21 +373,21 @@ void* _configure6PWM(long pwm_frequency, float dead_zone, const int pinA_h, cons
373373

374374
bool success = true;
375375
if (isHardware6Pwm(pinA_h, pinA_l)) {
376-
success &= configureTimerPin(params, 0, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, !(SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
376+
success &= configureTimerPin(params, 0, !SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, (SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
377377
}
378378
else {
379379
success &= configureTimerPin(params, 0, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH);
380380
success &= configureTimerPin(params, 1, !(SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH)); // reverse polarity on low side gives desired active high/low behaviour
381381
}
382382
if (isHardware6Pwm(pinB_h, pinB_l)) {
383-
success &= configureTimerPin(params, 2, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, !(SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
383+
success &= configureTimerPin(params, 2, !SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, (SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
384384
}
385385
else {
386386
success &= configureTimerPin(params, 2, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH);
387387
success &= configureTimerPin(params, 3, !(SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
388388
}
389389
if (isHardware6Pwm(pinC_h, pinC_l)) {
390-
success &= configureTimerPin(params, 4, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, !(SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
390+
success &= configureTimerPin(params, 4, !SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH, true, (SIMPLEFOC_PWM_LOWSIDE_ACTIVE_HIGH));
391391
}
392392
else {
393393
success &= configureTimerPin(params, 4, SIMPLEFOC_PWM_HIGHSIDE_ACTIVE_HIGH);

0 commit comments

Comments
 (0)