Posted on: 2009-12-13 10:34
hello All,
please help me.
my code is these:
#include <AT91SAM7S64.H>
#include <lib_AT91SAM7S64.h>
void wait(void);
int main (void){
AT91F_PIO_CfgOutput(AT91C_BASE_PIOA,AT91C_PIO_PA0);
AT91F_PIO_SetOutput(AT91C_BASE_PIOA,AT91C_PIO_PA0);
wait();
AT91F_PIO_ClearOutput(AT91C_BASE_PIOA,AT91C_PIO_PA0);
wait();
}
void wait(void){
unsigned int n;
for(n=0;n<7372800;n++);
}
where is my problem?
Posted on: 2009-12-13 11:01
You must make a endless loop in your programm.
while(1)
{
AT91F_PIO_SetOutput(AT91C_BASE_PIOA,AT91C_PIO_PA0);
wait();
AT91F_PIO_ClearOutput(AT91C_BASE_PIOA,AT91C_PIO_PA0);
wait();
}
Posted on: 2009-12-13 11:11
You define
unsigned int n; |
then:
for(n=0;n<7372800;n++); |
unsigned int range from 0 to 65335 You need unsigned long (or unsigned long int) rangng 0 to 4294967295. Its possible the Optimicer will remove the loop
for(n=0;n<7372800;n++); |
because it is empty! avr
Posted on: 2009-12-13 11:18
@ avr It runs on a 32 Bit machine. ARM AT91SAM7S64 there ist "int" = 32 bit
Posted on: 2009-12-13 11:48
> unsigned int range from 0 to 65335
ARM is 32Bit, so it ranges to 4G.
Posted on: 2009-12-13 11:49
helmi1: Hey, why didn't I see your post before? :-)
Posted on: 2009-12-13 19:03
Hossein Heidari wrote:
> where is my problem?
How can we tell if you don't tell us the manner of the failure. You
have simply said that it does not work, but in what way? Does it fail
to compile or link; does it crash, make smoke, not behave in the way you
expect? What do you expect it to do?
Either way an empty loop is a really bad way to implement a time delay;
the device has timer hardware, you should use that. Even if you do use
a loop, it is likely to get removed by the compiler optimiser when it
notices that it does nothing. To solve that you must at least declare
the counter variable as volatile.
Enable email notification