Help w/ code change from L298 to AFMotorShield v2.3

Adafruit Ethernet, Motor, Proto, Wave, Datalogger, GPS Shields - etc!

Moderators: adafruit_support_bill, adafruit

Please be positive and constructive with your questions and comments.
Locked
User avatar
mjosbesh
 
Posts: 3
Joined: Fri Sep 15, 2017 2:05 pm

Help w/ code change from L298 to AFMotorShield v2.3

Post by mjosbesh »

I was using a particle photon with a L298 motor driver. Now I am trying to modify the code written for the L298, which requires jumpers to control PWM and direction, to be used w/ AF Motor Shield v 2.3 and Arduino Uno, which controls PWM & Direction another way which I don't fully understand yet. Obviously, I am still very new to coding so I need a little help if someone would be so kind. I've attached my code below. I've commented out the code I believe is no longer needed and added what I think I'm supposed to. I'm still getting a code error related to mySpeed.

Code: Select all

#include <Adafruit_MotorShield.h>
Adafruit_MotorShield AFMS = Adafruit_MotorShield();
Adafruit_DCMotor *myMotor = AFMS.getMotor (1);

// Global variables to set motor timing
//    Each funcion (opening, closing) has a max run time and two phases, before middle limit switch (1) and after (2)
//    Each phase has a time that the speed ramps up and a beginnig speed and end speed.
//    Times are in milliseconds.  Speeds are in PWM value from 0 - 255.

const int openingMaxRunTime = 1000*9;  
const int openingRampTime1 = 2500;  
const int openingStartSpeed1 = 40; 
const int openingEndSpeed1 = 80; 
const int openingRampTime2 = 2000; 
const int openingStartSpeed2 = 80; 
const int openingEndSpeed2 = 255; 

const int closingMaxRunTime = 1000*9; 
const int closingRampTime1 = 2000;
const int closingStartSpeed1 = 100;
const int closingEndSpeed1 = 255;  
const int closingRampTime2 = 1500;
const int closingStartSpeed2 = 150; 
const int closingEndSpeed2 = 110; 

const int closingTimeAfterLimitSwitch = 1000;  //Time after closed limit switch hit before motors are stopped

const int demoOpenDelay = 5000;  //5 second delay after opening
const int demoCloseDelay = 5000;  //5 second delay after closing

//Set to 2 if controller uses 2 pins to control motor direction (i.e. HIGH,LOW for Forward), 
//  set to 1 if motor controller uses 1 pin (i.e. HIGH for forward, LOW for reverse)
const int motorControllerPins = 2;  

// connect motor controller pins to Arduino digital pins
// motor one and two connected to same pins
//int enA = D0;
//int in1 = D2;
//int in2 = D3;

// limit switches
int lr = A3;  //Close/fully retracted limit switch
int ls = A4;  //Limit switch used to trigger speed change
int le = A5;  //Open/fully extended limit switch

// buttons
int bo = A0;  //opening button
int bs = A1;  //stop button
int bc = A2;  //close button
int bd = D5;  //demo button


int allStop = 0; //Global variable to stop all actions
int working = 0; //Global variable in process of performing action

void setup()
{
  // set all the motor control pins to outputs
       //pinMode(enA, OUTPUT);
       //pinMode(in1, OUTPUT);
       //pinMode(in2, OUTPUT);
  pinMode(lr, INPUT_PULLUP);
  pinMode(ls, INPUT_PULLUP);
  pinMode(le, INPUT_PULLUP);
  pinMode(bo, INPUT_PULLUP);
  pinMode(bs, INPUT_PULLUP);
  pinMode(bc, INPUT_PULLUP);
  //pinMode(bd, INPUT_PULLUP);
  //pinMode(setupButton, INPUT_PULLUP);
  
  controlMotors(0,0);
}

void loop() {
  allStop = 0;
  if(digitalRead(bo)==LOW)
    openMount();
  if(digitalRead(bc)==LOW)
    closeMount();
  if(digitalRead(bd)==LOW)
    demoButtonPushed();
  delay(10);
}

/*void demoButtonPushed() {
    while(!allStop){
        if(!allStop) openMount();
        for(int x=0; x<demoOpenDelay/20 && digitalRead(bs); x++)
            delay(20);
        if(!allStop) closeMount();
        for(int x=0; x<demoCloseDelay/20 && digitalRead(bs); x++)
            delay(20);
    }
}*/
// this code is not run.  it was used when an interrupt routine ran on stop button pushed.
void stopButtonPushed() {
    controlMotors(0,0);
    allStop = 1;
}

void openMount() {
    //Particle.publish("opening");
    int startTime = millis();
    int startTime2 = 0;
    int mySpeed = 0;
    while(!allStop && (millis()-startTime)<openingMaxRunTime && digitalRead(le) == HIGH) {
        if(digitalRead(ls)==LOW) {
            if(millis()-startTime<openingRampTime1)
            {
                if(openingEndSpeed1>openingStartSpeed1)
                    mySpeed = openingStartSpeed1+(millis()-startTime)/(openingRampTime1/(openingEndSpeed1-openingStartSpeed1));
                else
                    mySpeed = openingStartSpeed1-(millis()-startTime)/(openingRampTime1/(openingStartSpeed1-openingEndSpeed1));
            }
            else
                mySpeed = openingEndSpeed1;
            controlMotors(1, mySpeed);
        }
        else {
            if(startTime2 == 0) startTime2 = millis();
            if(millis()-startTime2<openingRampTime2)
           {
                if(openingEndSpeed2>openingStartSpeed2)
                    mySpeed = openingStartSpeed2+(millis()-startTime2)/(openingRampTime2/(openingEndSpeed2-openingStartSpeed2));
                else
                    mySpeed = openingStartSpeed2-(millis()-startTime2)/(openingRampTime2/(openingStartSpeed2-openingEndSpeed2));
            }
            else
                mySpeed = openingEndSpeed2;
            controlMotors(1, mySpeed);
        }
        if(digitalRead(bs)==LOW){
          allStop = 1;
          controlMotors(0,0);
        }
        delay(10);
    }
    controlMotors(0,0);
}

void closeMount() {
    //Particle.publish("closing");
    int startTime = millis();
    int startTime2 = 0;
    int mySpeed = 0;
    while(!allStop && (millis()-startTime)<closingMaxRunTime && digitalRead(lr) == HIGH) {
        if(digitalRead(ls)==HIGH) {
            if(millis()-startTime<closingRampTime1)
            {
                if(closingEndSpeed1>closingStartSpeed1)
                    mySpeed = closingStartSpeed1+(millis()-startTime)/(closingRampTime1/(closingEndSpeed1-closingStartSpeed1));
                else
                    mySpeed = closingStartSpeed1-(millis()-startTime)/(closingRampTime1/(closingStartSpeed1-closingEndSpeed1));
            }
            else
                mySpeed = closingEndSpeed1;
            controlMotors(2, mySpeed);
        }
        else {
            if(startTime2 == 0) startTime2 = millis();
            if(millis()-startTime2<closingRampTime2)
            {
                if(closingEndSpeed2>closingStartSpeed2)
                    mySpeed = closingStartSpeed2+(millis()-startTime2)/(closingRampTime2/(closingEndSpeed2-closingStartSpeed2));
                else
                    mySpeed = closingStartSpeed2-(millis()-startTime2)/(closingRampTime2/(closingStartSpeed2-closingEndSpeed2));
            }
            else
                mySpeed = closingEndSpeed2;
            controlMotors(2, mySpeed);
        }
        if(digitalRead(bs)==LOW){
          allStop = 1;
          controlMotors(0,0);
        }
        delay(10);
    }
    delay(closingTimeAfterLimitSwitch);
    controlMotors(0,0);
}

// controlMotors
// Direction 0 = brake, 1 = forward, 2 = reverse
// Speed 0 - 255
void controlMotors(int Direction, int Speed) {
    if(Direction == 1){
        myMotor->run(FORWARD); //AFMS
        //digitalWrite(in1, HIGH);
        //digitalWrite(in2, LOW);
    }
    else if(Direction == 2){
        myMotor->run(BACKWARD); //AFMS
        //digitalWrite(in1, LOW);
        //digitalWrite(in2, HIGH);
    }
  /*  else {
        digitalWrite(in1, HIGH);
        digitalWrite(in2, HIGH);
    }*/
    //analogWrite(enA, Speed);
    myMotor->setSpeed(speed); //AFMS // I took a couple guesses and changed this 
}

User avatar
adafruit_support_bill
 
Posts: 88142
Joined: Sat Feb 07, 2009 10:11 am

Re: Help w/ code change from L298 to AFMotorShield v2.3

Post by adafruit_support_bill »

... which controls PWM & Direction another way which I don't fully understand yet.
It has an on-board PWM generator chip. The library sends commands to the chip via i2c to control speed and direction.

Code: Select all

 I'm still getting a code error related to mySpeed.
If you post the error message we can help you debug it.

User avatar
mjosbesh
 
Posts: 3
Joined: Fri Sep 15, 2017 2:05 pm

Re: Help w/ code change from L298 to AFMotorShield v2.3

Post by mjosbesh »

Thanks. Here's the error message

Arduino: 1.8.5 (Windows 8.1), Board: "Arduino/Genuino Uno"

C:\Users\Mark\Documents\Arduino\TVMount\TVMount_v0.1.6.AFv2\TVMount_v0.1.6.AFv2.ino: In function 'void controlMotors(int, int)':

TVMount_v0.1.6.AFv2:203: error: 'speed' was not declared in this scope

myMotor->setSpeed(speed); //AFMS // I took a couple guesses and changed this

^

exit status 1
'speed' was not declared in this scope

This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

User avatar
adafruit_support_bill
 
Posts: 88142
Joined: Sat Feb 07, 2009 10:11 am

Re: Help w/ code change from L298 to AFMotorShield v2.3

Post by adafruit_support_bill »

'Speed' should be capitalized - to match the parameter in your function.

User avatar
mjosbesh
 
Posts: 3
Joined: Fri Sep 15, 2017 2:05 pm

Re: Help w/ code change from L298 to AFMotorShield v2.3

Post by mjosbesh »

Thanks for catching that. I've made the change. There are no errors however it still doesn't work. I assume I'm still missing some connection in the code to tell the shield which direction and speed of the motor. I just don't have enough knowledge yet to pick out the issue. Believe me... I've tried. My head hurts.

User avatar
adafruit_support_bill
 
Posts: 88142
Joined: Sat Feb 07, 2009 10:11 am

Re: Help w/ code change from L298 to AFMotorShield v2.3

Post by adafruit_support_bill »

You probably want to set the speed before telling the motors to run.

I'd add some serial output to the "controlMotors()" function so you can see in the monitor what motor commands are actually being executed.

Locked
Please be positive and constructive with your questions and comments.

Return to “Arduino Shields from Adafruit”