A very small problem with delay

[Vodka]

Hi all ccs coders...i have a small issue with delay_ms() function.My code should blink 5 leds connected to port D0-D4 on a pic16f877;my code is here:


#include "D:\CCS WORK\asta.h"


void main()
{

setup_adc_ports(NO_ANALOGS);
setup_adc(ADC_OFF);
setup_psp(PSP_DISABLED);
setup_spi(SPI_SS_DISABLED);
setup_timer_0(RTCC_EXT_H_TO_L|RTCC_DIV_1);
setup_timer_1(T1_DISABLED);
setup_timer_2(T2_DISABLED,0,1);

// TODO: USER CODE!!

set_tris_d(0x00);
while(1){
output_high(pin_d0); //leds are off
output_high(pin_d1);
output_high(pin_d2);
output_high(pin_d3);
output_high(pin_d4);
delay_ms(1000); //delay 1 sec
output_low(pin_d0); //led on
delay_ms(1000); //delay 1 sec
output_high(pin_d1);
delay_ms(1000); //delay 1 sec
output_low(pin_d2);
delay_ms(1000); //delay 1 sec
output_high(pin_d3);
delay_ms(1000); //delay 1 sec
output_low(pin_d4);
delay_ms(1000); //delay 1 sec

}
}


When i don`t use delay_ms() sintax the leds are on as they should...but when i put delay_ms(1000) they aren`t doing anything(leds are off).

Please guide me where my mistake is. thank you all

[rnielsen]

First, when using output_high() or output_low() you don't need to set the tris. The compiler will take care of this for you.

Next, post the #fuses you are using. Are you using the internal oscillator or do you have a crystal attached?

Ronald

[Vodka]

i use :

#fuses HS,NOLVP, NOWDT,NOPROTECT

the crystal is external at 20Mhz

[Ttelmah]

You have:
#use_delay(clock=20000000)

I hope.

Best Wishes

[Vodka]

no i have:

#use delay (clock=20000000)

[Vodka]

seems like no one can help me?? i have to try other compliers?? but i love ccs .....should i choose another version???

[Ttelmah]

Seriously, I have compiled the code, with the fuses as quoted, and the delay, and it works.
One thing, where is the delay statement?. It must be after the fuse statement.
What is the compiler version?.
I have tried (just for the hell of it), 3.212, 3.249, 4.051. All worked fine.

Best Wishes

[Ttelmah]

As a couple more comments.
The placement of the delay statement, doesn't seem to matter on the newer compilers, but better to put it on the next line to 'play safe'. Obvious comment though, how are the LED's connected?. Non operation, could be caused by something electrical, like having the LED's directly connected to the output pins, with no current limiting resistors. Without the delay, they would only be 'on' for a few uSec, and the current internal limiting, and capacitances on the supply line, might allow it to work, but with the delay, they may well simply droop the supply rail, and restart the processor, so quickly, that it looks as if they are not lighting.

Best Wishes

[Vodka]

Well i have tried with ccs ver 4.013 but i can`t find any of the version you mentioned... if you please help me with one of those versions just to be sure that the compiler is ok i`ll be very grateful. My e-mail is [email protected]

[ckielstra]