X-Git-Url: https://defiant.homedns.org/gitweb/?p=ros_wild_thumper.git;a=blobdiff_plain;f=avr%2Fmotor_ctrl%2Fmain.c;h=5d655290a7e8b78cb3efd33b881eaf85fafd3bfe;hp=048fab01bbf645fe7ee949cb20045b8e878b5dab;hb=b301eb19e6487497a8bb946c538a25aa4ee28c1d;hpb=9600299d1815cb14a3f49c72c4266adc1f471bcb diff --git a/avr/motor_ctrl/main.c b/avr/motor_ctrl/main.c index 048fab0..5d65529 100644 --- a/avr/motor_ctrl/main.c +++ b/avr/motor_ctrl/main.c @@ -90,26 +90,39 @@ * 0x95 Aft Handicap * free * 0xA0 Reset reason - * 0xA1 TLE Error status + * 0xA1 Error status * 0xA2 count test * free * 0xff Bootloader */ -#define TWI_ACK TWCR = (1<>8; @@ -570,7 +603,7 @@ ISR(TWI_vect) MCUCSR = 0x0; TWI_ACK; break; - case 0xA1: // TLE Error status + case 0xA1: // Error status TWDR = error_state; TWI_ACK; break; @@ -671,31 +704,31 @@ static void update_motor(void) { static int16_t m3_old=SHRT_MIN; static int16_t m4_old=SHRT_MIN; - error_state = ~((PIND & 0x40)>>3 | (PINB & 0x07)) & 0xf; - - // if error and running: stop - if (motor1_mode == MOTOR_PID && bit_is_set(error_state, 0) && m1_old != 0) motor1 = 0; - if (motor2_mode == MOTOR_PID && bit_is_set(error_state, 1) && m2_old != 0) motor2 = 0; - if (motor3_mode == MOTOR_PID && bit_is_set(error_state, 2) && m3_old != 0) motor3 = 0; - if (motor4_mode == MOTOR_PID && bit_is_set(error_state, 3) && m4_old != 0) motor4 = 0; - - // if we start motor in error state: start with full power - if (motor1_mode == MOTOR_PID && bit_is_set(error_state, 0) && m1_old == 0 && motor1 != 0) motor1 = 255; - if (motor2_mode == MOTOR_PID && bit_is_set(error_state, 1) && m2_old == 0 && motor2 != 0) motor2 = 255; - if (motor3_mode == MOTOR_PID && bit_is_set(error_state, 2) && m3_old == 0 && motor3 != 0) motor3 = 255; - if (motor4_mode == MOTOR_PID && bit_is_set(error_state, 3) && m4_old == 0 && motor4 != 0) motor4 = 255; + error_state &= 0xf0; // clear lower bits + error_state |= ~((PIND & 0x40)>>3 | (PINB & 0x07)) & 0xf; if (m1_old != motor1) { // update only when changed if (motor1 == 0) { // stop + PORTC &= ~(1 << 3) & ~(1 << 2); + DISABLE_PWM_MOTOR1; + } else if (motor1 == PWM_BREAK) { PORTC |= (1 << 3) | (1 << 2); + ENABLE_PWM_MOTOR1; } else if ((!motor1_switch && motor1 > 0) || (motor1_switch && motor1 < 0)) { // forward - PORTC &= ~(1 << 3) & ~(1 << 2); + uint8_t tmp=PORTC; + tmp &= ~(1 << 3); + tmp |= (1 << 2); + PORTC = tmp; + ENABLE_PWM_MOTOR1; } else { // motor1 < 0 // backward - PORTC &= ~(1 << 2); - PORTC |= (1 << 3); + uint8_t tmp=PORTC; + tmp &= ~(1 << 2); + tmp |= (1 << 3); + PORTC = tmp; + ENABLE_PWM_MOTOR1; } m1_old = motor1; @@ -705,14 +738,25 @@ static void update_motor(void) { if (m2_old != motor2) { // update only when changed if (motor2 == 0) { // stop + PORTC &= ~(1 << 5) & ~(1 << 4); + DISABLE_PWM_MOTOR2; + } else if (motor2 == PWM_BREAK) { PORTC |= (1 << 5) | (1 << 4); + ENABLE_PWM_MOTOR2; } else if ((!motor2_switch && motor2 > 0) || (motor2_switch && motor2 < 0)) { // forward - PORTC &= ~(1 << 5) & ~(1 << 4); + uint8_t tmp=PORTC; + tmp &= ~(1 << 5); + tmp |= (1 << 4); + PORTC = tmp; + ENABLE_PWM_MOTOR2; } else { // motor2 < 0 // backward - PORTC &= ~(1 << 4); - PORTC |= (1 << 5); + uint8_t tmp=PORTC; + tmp &= ~(1 << 4); + tmp |= (1 << 5); + PORTC = tmp; + ENABLE_PWM_MOTOR2; } m2_old = motor2; @@ -722,14 +766,25 @@ static void update_motor(void) { if (m3_old != motor3) { // update only when changed if (motor3 == 0) { // stop + PORTC &= ~(1 << 7) & ~(1 << 6); + DISABLE_PWM_MOTOR3; + } else if (motor3 == PWM_BREAK) { PORTC |= (1 << 7) | (1 << 6); + ENABLE_PWM_MOTOR3; } else if ((!motor3_switch && motor3 > 0) || (motor3_switch && motor3 < 0)) { // forward - PORTC &= ~(1 << 7) & ~(1 << 6); + uint8_t tmp=PORTC; + tmp &= ~(1 << 7); + tmp |= (1 << 6); + PORTC = tmp; + ENABLE_PWM_MOTOR3; } else { // motor3 < 0 // backward - PORTC &= ~(1 << 6); - PORTC |= (1 << 7); + uint8_t tmp=PORTC; + tmp &= ~(1 << 6); + tmp |= (1 << 7); + PORTC = tmp; + ENABLE_PWM_MOTOR3; } m3_old = motor3; @@ -739,14 +794,25 @@ static void update_motor(void) { if (m4_old != motor4) { // update only when changed if (motor4 == 0) { // stop + PORTD &= ~(1 << 3) & ~(1 << 2); + DISABLE_PWM_MOTOR4; + } else if (motor4 == PWM_BREAK) { PORTD |= (1 << 3) | (1 << 2); + ENABLE_PWM_MOTOR4; } else if ((!motor4_switch && motor4 > 0) || (motor4_switch && motor4 < 0)) { // forward - PORTD &= ~(1 << 3) & ~(1 << 2); + uint8_t tmp=PORTD; + tmp &= ~(1 << 3); + tmp |= (1 << 2); + PORTD = tmp; + ENABLE_PWM_MOTOR4; } else { // motor4 < 0 // backward - PORTD &= ~(1 << 2); - PORTD |= (1 << 3); + uint8_t tmp=PORTD; + tmp &= ~(1 << 2); + tmp |= (1 << 3); + PORTD = tmp; + ENABLE_PWM_MOTOR4; } m4_old = motor4; @@ -766,7 +832,6 @@ static void update_pos(void) { int16_t pos4_diff; float diff_left_m, diff_right_m, angle_diff, translation; float pos_x_new, pos_y_new, angle_new; - int16_t speed_l, speed_r; float tmp_speed_lin, tmp_speed_rot; int16_t cur_pos1, cur_pos2, cur_pos3, cur_pos4; int16_t new_speed1, new_speed2, new_speed3, new_speed4; @@ -801,10 +866,8 @@ static void update_pos(void) { pos_x_new = pos_x.f + cos(angle_new)*translation; pos_y_new = pos_y.f + sin(angle_new)*translation; - speed_l = (new_speed1+new_speed2)/2; - speed_r = (new_speed3+new_speed4)/2; - tmp_speed_lin = (speed_l + speed_r)/(2.0*STEP_PER_M_AVG); - tmp_speed_rot = (speed_r - speed_l)/(M_PI*WHEEL_DIST*STEP_PER_M_AVG); + tmp_speed_lin = translation/PID_T; + tmp_speed_rot = angle_diff/PID_T; // copy from tmp cli(); @@ -836,63 +899,117 @@ static void update_pid(void) { static int32_t esum3=0; static int32_t esum4=0; + // protect motors from damage if stalling + if (labs(esum1) > STALL_LIMIT && speed1 == 0) { + motor1 = 0; + motor1_mode = MOTOR_MANUAL; + error_state |= (1<<4); + esum1 = 0; + } + if (labs(esum2) > STALL_LIMIT && speed2 == 0) { + motor2 = 0; + motor2_mode = MOTOR_MANUAL; + error_state |= (1<<5); + esum2 = 0; + } + if (labs(esum3) > STALL_LIMIT && speed3 == 0) { + motor3 = 0; + motor3_mode = MOTOR_MANUAL; + error_state |= (1<<6); + esum3 = 0; + } + if (labs(esum4) > STALL_LIMIT && speed4 == 0) { + motor4 = 0; + motor4_mode = MOTOR_MANUAL; + error_state |= (1<<7); + esum4 = 0; + } + if (motor1_mode == MOTOR_PID) { + if (speed1_wish != speed1_wish_old) { + if (abs(speed1_wish - speed1_wish_old) > 500) esum1 = 0; + speed1_wish_old = speed1_wish; + } + if (speed1_wish == 0) { motor1 = 0; eold1 = 0; - esum1 = 0; + error_state &= ~(1<<4); } else { int16_t e = speed1_wish - speed1; esum1+=e; motor1 = KP*e + KI*PID_T*esum1 + KD/PID_T*(e - eold1); eold1 = e; - if (motor1 > 255) motor1 = 255; + if (motor1 > 0 && speed1_wish < 0) motor1=PWM_BREAK; + else if (motor1 < 0 && speed1_wish > 0) motor1=PWM_BREAK; + else if (motor1 > 255) motor1 = 255; else if (motor1 < -255) motor1 = -255; } } if (motor2_mode == MOTOR_PID) { + if (speed2_wish != speed2_wish_old) { + if (abs(speed2_wish - speed2_wish_old) > 500) esum2 = 0; + speed2_wish_old = speed2_wish; + } + if (speed2_wish == 0) { motor2 = 0; eold2 = 0; - esum2 = 0; + error_state &= ~(1<<5); } else { int16_t e = speed2_wish - speed2; esum2+=e; motor2 = KP*e + KI*PID_T*esum2 + KD/PID_T*(e - eold2); eold2 = e; - if (motor2 > 255) motor2 = 255; + if (motor2 > 0 && speed2_wish < 0) motor2=PWM_BREAK; + else if (motor2 < 0 && speed2_wish > 0) motor2=PWM_BREAK; + else if (motor2 > 255) motor2 = 255; else if (motor2 < -255) motor2 = -255; } } if (motor3_mode == MOTOR_PID) { + if (speed3_wish != speed3_wish_old) { + if (abs(speed3_wish - speed3_wish_old) > 500) esum3 = 0; + speed3_wish_old = speed3_wish; + } + if (speed3_wish == 0) { motor3 = 0; eold3 = 0; - esum3 = 0; + error_state &= ~(1<<6); } else { int16_t e = speed3_wish - speed3; esum3+=e; motor3 = KP*e + KI*PID_T*esum3 + KD/PID_T*(e - eold3); eold3 = e; - if (motor3 > 255) motor3 = 255; + if (motor3 > 0 && speed3_wish < 0) motor3=PWM_BREAK; + else if (motor3 < 0 && speed3_wish > 0) motor3=PWM_BREAK; + else if (motor3 > 255) motor3 = 255; else if (motor3 < -255) motor3 = -255; } } if (motor4_mode == MOTOR_PID) { + if (speed4_wish != speed4_wish_old) { + if (abs(speed4_wish - speed4_wish_old) > 500) esum4 = 0; + speed4_wish_old = speed4_wish; + } + if (speed4_wish == 0) { motor4 = 0; eold4 = 0; - esum4 = 0; + error_state &= ~(1<<7); } else { int16_t e = speed4_wish - speed4; esum4+=e; motor4 = KP*e + KI*PID_T*esum4 + KD/PID_T*(e - eold4); eold4 = e; - if (motor4 > 255) motor4 = 255; + if (motor4 > 0 && speed4_wish < 0) motor4=PWM_BREAK; + else if (motor4 < 0 && speed4_wish > 0) motor4=PWM_BREAK; + else if (motor4 > 255) motor4 = 255; else if (motor4 < -255) motor4 = -255; } } @@ -914,7 +1031,7 @@ int main(void) { DDRB = (1 << 3); DDRC = (1 << 7) | (1 << 6) | (1 << 5) | (1 << 4) | (1 << 3) | (1 << 2); DDRD = (1 << 7) | (1 << 5) | (1 << 4) | (1 << 3) | (1 << 2); - // Pullup TLEs EF + // Pullup Diag/Enable PORTB = (1 << 0) | (1 << 1) | (1 << 2); PORTD = (1 << 6); @@ -927,23 +1044,30 @@ int main(void) { TWI_RESET; // Motor 1 & 2 - // Timer 1: Fast PWM inverting mode, Top=256 => 15.625kHz + // Also used for PWM frequency TIMER1_FREQ (F_CPU/256) + // Timer 1: Fast PWM non-inverting mode, Top=255 => 19.531kHz // Prescaler=1 - TCCR1A = (1 << COM1A1) | (1 << COM1B1) | (1 << COM1A0) | (1 << COM1B0) | (1 << WGM10); + //TCCR1A = (1 << COM1A1) | (1 << COM1B1) | (1 << WGM10); + // Avoid narrow spike on extreme pwm value 0 by not setting COM1*1 + TCCR1A = (1 << WGM10); TCCR1B = (1 << WGM12) | (1 << CS10); OCR1A = 0; OCR1B = 0; // Motor 3 - // Timer 2: Fast PWM inverting mode, Top=256 + // Timer 2: Fast PWM non-inverting mode, Top=255 // Prescaler=1 - TCCR2 = (1 << WGM21) | (1 << WGM20) | (1 << COM21) | (1 << COM20) | (1 << CS20); + //TCCR2 = (1 << WGM21) | (1 << WGM20) | (1 << COM21) | (1 << CS20); + // Avoid narrow spike on extreme pwm value 0 by not setting COM21 + TCCR2 = (1 << WGM21) | (1 << WGM20) | (1 << CS20); OCR2 = 0; // Motor 4 - // Timer 0: Fast PWM inverting mode, Top=256 + // Timer 0: Fast PWM non-inverting mode, Top=255 // Prescaler=1 - TCCR0 = (1 << WGM01) | (1 << WGM00) | (1 << COM01) | (1 << COM00) | (1 << CS00); + //TCCR0 = (1 << WGM01) | (1 << WGM00) | (1 << COM01) | (1 << CS00); + // Avoid narrow spike on extreme pwm value 0 by not setting COM01 + TCCR0 = (1 << WGM01) | (1 << WGM00) | (1 << CS00); OCR0 = 0; printf("\r\nStart\r\n"); @@ -976,7 +1100,7 @@ int main(void) { cmd_vel.bUpdate = 0; sei(); - speed_wish_right = angle*M_PI*WHEEL_DIST/2 + speed; + speed_wish_right = (angle*WHEEL_DIST)/2 + speed; speed_wish_left = speed*2-speed_wish_right; speed_wish_left*=STEP_PER_M_LEFT; @@ -984,17 +1108,17 @@ int main(void) { if (aft_handicap > 0) { speed1_wish = speed_wish_left * (100-aft_handicap)/100.0; - speed4_wish = speed_wish_right * (100-aft_handicap)/100.0; + speed3_wish = speed_wish_right * (100-aft_handicap)/100.0; } else { speed1_wish = speed_wish_left; - speed4_wish = speed_wish_right; + speed3_wish = speed_wish_right; } if (front_handicap > 0) { speed2_wish = speed_wish_left * (100-front_handicap)/100.0; - speed3_wish = speed_wish_right * (100-front_handicap)/100.0; + speed4_wish = speed_wish_right * (100-front_handicap)/100.0; } else { speed2_wish = speed_wish_left; - speed3_wish = speed_wish_right; + speed4_wish = speed_wish_right; } motor1_mode = MOTOR_PID; motor2_mode = MOTOR_PID; @@ -1002,7 +1126,7 @@ int main(void) { motor4_mode = MOTOR_PID; } - if (run_update >= 156) { // ~100Hz + if (run_update >= 195) { // TIMER1_FREQ/195 = ~100Hz run_update=0; update_pos();