Odd counter problems

[Ryan M.]

Originally Posted by Catastrophy

ok I did and now theres nothing there, its not printing anything.

Hum, well, try just using the following line. It's guarranteed to print and if it doesn't, it means that it is never getting called and you'll have to look at your code.

printf("Test");

--EDIT--
If it doesn't work then post your code and people can look at it.

[Catastrophy]

It printed test but here it is. I'm using a basic auto right now so i can debug everything first.

[Ryan M.]

Originally Posted by Catastrophy

It printed test but here it is. I'm using a basic auto right now so i can debug everything first.

What exactly is the output say? Does the 'arm value of armcounter' print anything? What does it print?

--EDIT--
Try changing the %i in the printf to %d.

[Joe Ross]

Originally Posted by Texan

Try changing the %i in the printf to %d.

Remeber that IFI implemented printf and that it isn't a "standard" implementation. Only the %s, %lx, %d, %x, %u, %X directives are parsed.

Catastrophy, is your problem fixed now that it is counting, or do you have more problems?

[Ryan M.]

Originally Posted by Joe Ross

Remeber that IFI implemented printf and that it isn't a "standard" implementation. Only the %s, %lx, %d, %x, %u, %X directives are parsed.

Catastrophy, is your problem fixed now that it is counting, or do you have more problems?

Oh, didn't know those limitations. Thanks

[Catastrophy]

well the motors still arnt stopping.

[Joe Ross]

Originally Posted by Catastrophy

well the motors still arnt stopping.

Please post the code that you are using currently.

If you are using the corrections that Rob posted, pwm06 will go forward for about 1/3 a second and then go in reverse forever. You also have a similar behavior for pwm07 (assuming you corrected the errors)

[rbayer]

A few things I saw after glancing at it:

if (armcounter = 15) //will raise arm.
   pwm06=0;
else 
   armcounter++;
   pwm06=255;

sould be

if (armcounter == 15) //will raise arm.
   pwm06=0;
else {
   armcounter++;
   pwm06=255; 
}

It is VERY rare that you'll actually want to use a single = in an if, so watch out for that, and also be careful with your curly braces.

That said, I believe the code in its original form should have printf'd 15 over and over and over. Was it doing that, or just not printing anything?

[Ryan M.]

Originally Posted by rbayer

A few things I saw after glancing at it:

if (armcounter = 15) //will raise arm.
   pwm06=0;
else 
   armcounter++;
   pwm06=255;

sould be

if (armcounter == 15) //will raise arm.
   pwm06=0;
else {
   armcounter++;
   pwm06=255; 
}

It is VERY rare that you'll actually want to use a single = in an if, so watch out for that, and also be careful with your curly braces.

That said, I believe the code in its original form should have printf'd 15 over and over and over. Was it doing that, or just not printing anything?

Missing a = is an easy thing to do. It's one of those powerful, useful things that C/C++ lets you do, but isn't what you want most of the time.

[Catastrophy]

Well the = was something i accidently did when moving it i had it in the code i was using. and btw its just not printing anything.

[Ryan M.]

Originally Posted by Catastrophy

Well the = was something i accidently did when moving it i had it in the code i was using. and btw its just not printing anything.

So, the "test" prints, but the other thing doesn't?

[deltacoder1020]

Originally Posted by rbayer

A few things I saw after glancing at it:

if (armcounter = 15) //will raise arm.
   pwm06=0;
else 
   armcounter++;
   pwm06=255;

sould be

if (armcounter == 15) //will raise arm.
   pwm06=0;
else {
   armcounter++;
   pwm06=255; 
}

It is VERY rare that you'll actually want to use a single = in an if, so watch out for that, and also be careful with your curly braces.

That said, I believe the code in its original form should have printf'd 15 over and over and over. Was it doing that, or just not printing anything?

remember that you need to add the curly braces, not just change the equals sign to a double-equals. w/o the braces, the code is equivalent to this:

if (armcounter == 15) //will raise arm.
   pwm06=0;
else {
   armcounter++; 
}

pwm06=255;

note how pwm06 will always end up as 255, no matter what the value is.

you said that the "motors weren't stopping" - of course they aren't: 0 is full reverse, 255 is full forward. if you want a motor to stop, set the PWM to 127.

[Catastrophy]

Well i got it working i was messign with the code when i posted it so i accidently left stuff out. Thanks Tex helped a lot.

[steven114]

Originally Posted by Texan

Hum, well, try just using the following line. It's guarranteed to print and if it doesn't, it means that it is never getting called and you'll have to look at your code.

printf("Test");

--EDIT--
If it doesn't work then post your code and people can look at it.

Try

printf("Test\n");

instead. If it works anything like printf on any other system, it won't flush the buffer until it gets a newline...

[Ryan M.]

Originally Posted by steven114

Try

printf("Test\n");

instead. If it works anything like printf on any other system, it won't flush the buffer until it gets a newline...

Yeah, that is true, but it did print for him. The wierd thing is, at least from what he has said, the line that is right below it doesn't say print anything, even though it should.