Hi;I have a motor.c code in which pin 13 and 14 are set\used for clockwise and anti-clockwise direcion as original firmware. I want to use only one pin 14 for DIR and then set the values to HIGH and LOW for clockwise and anti-clockwise direction.
I attached the original code below.
Please tell me how to use the lines below in original code.

#undef F_CPU 
#define F_CPU 1000000UL 
#include <avr/io.h> 
#include <util/delay.h> 
int main(void) { 
  DDRB = (1 << 2); 
  while(1) { 
    PORTB |= (1 <<2); 
    _delay_ms(200); 
    PORTB &= ~(1 << 2); 
    _delay_ms(200); 
  } 
}

 

Please someone help me. I want to fix the 10 lines above in the original motor.c code.So that the motor can rotate in both directions from only one Pin 14.

 

Depends on the compiler you are using. What are you using?

 

I am using avr studio 5.1

 

It still depends on which C compiler you are using. Some compilers allow you to create bit fields to address individual bits. Others have a predefined header that defines bit fields.
Check the header files for things like PORTB.0 or PB0.

Rather than messing around it is just as easy to address single bits as you have done.
I would keep it simple.

If pin-14 is PB2, here is your code modified to explicitly identify BIT2
(but I think pin-14 is PB0).

#undef F_CPU 
#define F_CPU 1000000UL 
#include <avr/io.h> 
#include <util/delay.h> 

#define BIT2 2

int main(void) { 
  DDRB = (1 << BIT2); 
  while(1) { 
    PORTB |= (1 << BIT2); 
    _delay_ms(200); 
    PORTB &= ~(1 <<  BIT2); 
    _delay_ms(200); 
  } 
}

If pin-14 is PB0 and that is all you want to control, then do this:

#undef F_CPU 
#define F_CPU 1000000UL 
#include <avr/io.h> 
#include <util/delay.h> 

int main(void) { 
  DDRB = 1; 
  while(1) { 
    PORTB |= 1; 
    _delay_ms(200); 
    PORTB &= ~1; 
    _delay_ms(200); 
  } 
}

 

Thanks but i want to modify motor.c code. Yes the pin 14 is PB2 SS/OC1B.
Can you please modify motor.txt ?

 

You have to define what is Motor_Port, Motor_Clockwise and Motor_Anticlockwise.
They are controlling two bits to define clockwise and anticlockwise.

 

Here is the full motor.h file:-

#include <avr/delay.h>
#include <avr/io.h>
#include <avr/iom8.h>
#include <avr/interrupt.h>
#include <ctype.h>
#include "UtilsAndDefines.h"
#define Motor_Clockwise  BIT2
#define Motor_Anticlockwise BIT1
#define Motor_Port    PORTB
#define Motor_Dir    DDRB
 
/**
*@brief simply control PWM for the motor.\n
The pwm must be givem with values between 0 and 2000.\n
The 1000 is the motor stopped. The 0 is the 100% PWM rotating Anticlockwise and the 2000 is the 100% PWM rotating Clockwise
The PWM has increments of one step with 1000 steps of resolution.
*@var pwm This is the pwm to set on the motor
*@return 2 if rotating clockwise.\n
1 if stopped.\n
0 if anticlockwise.\n
-1 if fail or bad value or limit reached.
*/
int8_t SetPWM(uint16_t pwm);

/**
*@brief Init the motor low level control, on this case pins, timers, pwm variables, etc.
*/
void InitMotor(void);
uint16_t PWM;

 

There you go. So they are using BIT2 and BIT1. You can modify that to whatever you wish.

 

Please modify it to use pin 14. OC1B PB2.

 

You need two pins, not one.

 

I want only 1 pin 14 because my dc motor shield has only 1 DIR pin.

 

pin14 is PB2 (ie BIT2)

I want BIT2 to go to 0 instead of 1

 

the way i read it, you need to set value to pwm, interrupt handler is controlling direction pins so your code should NOT touch them.

@brief simply control PWM for the motor.\n
The pwm must be givem with values between 0 and 2000.\n
The 1000 is the motor stopped. The 0 is the 100% PWM rotating Anticlockwise and the 2000 is the 100% PWM rotating Clockwise
The PWM has increments of one step with 1000 steps of resolution.

so you need to assign value to pwm that is in 0-2000 range, something like

#undef F_CPU 
#define F_CPU 1000000UL 
#include <avr/io.h> 
#include <util/delay.h> 
int main(void) { 
  // DDRB = (1 << 2); 
  while(1) { 
   pwm=500; 
    _delay_ms(2000); 
    pwm=1500; 
    _delay_ms(2000); 
  } 
}

also, you don't really expect motor to change direction 5 times in a second, do you? 200ms is way too short to get motor running, it would only twitch and hum in one place, so i just changed it to 2 seconds forward and 2 second reverse.

as for using one bit to control direction, that should not be problem (unless you are not telling something) because only one of the two bits is high. if the shield expects one bit, and ignores other, you are all set.