Flashing LED will not flash!

[jaco1982]

I think I'm going slightly mad! I'm quite new to embedded development (got a few years experience in server/client devving).
What I'm trying to do, is make a LED flash. Code is as follows:

void main() {
     TRISC = 0x00;
     PORTC = 0x00;
     
     for(; ;) {
           PORTC.F0 = 0xFF;
           delay_ms(500);
           PORTC.F0 = 0x00;
     }
     
}

I'm writing the code in MikroC, and simulating it in Proteus using a PIC18F4553 device. The LED goes on, but does not flash. Am I missing something?

 

[Rob B]

Put another delay in before the end of the for loop.

 

[millwood]

you will need to read the datasheet a few times, specifically the sections on IO Ports and Configuration Bits.

and then read your compiler manual to see how to implement what you learned there.

 

[john blue]

Can you post the whole code? You need to configure your configuration pins properly as well as the clock

 

[jaco1982]

Put another delay in before the end of the for loop.

That seems to have worked!

 

[Rob B]

That seems to have worked!

The LED was flashing but it was not in the off state long enough to see without an oscilloscope.

 

[jaco1982]

The LED was flashing but it was not in the off state long enough to see without an oscilloscope.

That was exactly it. I saw it when I put a scope on the output.

 

[Luciferus]

What if you change the "for (; " instruction with "while(1)"

 

[vinodstanur]

I think I'm going slightly mad! I'm quite new to embedded development (got a few years experience in server/client devving).
What I'm trying to do, is make a LED flash. Code is as follows:

void main() {
     TRISC = 0x00;
     PORTC = 0x00;
     
     for(; ;) {
           PORTC.F0 = 0xFF;
           delay_ms(500);
           PORTC.F0 = 0x00;
          [B] delay_ms(500);  //add this to make it off[/B]
 }
     
}

I'm writing the code in MikroC, and simulating it in Proteus using a PIC18F4553 device. The LED goes on, but does not flash. Am I missing something?

after PORTC.F0=0x00; U R NOT GIVING ANY DELAY. SO u cant see the led in OFF state since its OFF only for very few clock periods(i mean the clocks required to reach PORTC.F0=0xFF....

 

[incomplete_jerin]

Hi. You don't need to write

PORTC.F0 = 0xFF;

Only

PORTC.F0 = 1;

&

PORTC.F0 = 0;

are required.
Also just change the infinity loop you have given to

while(1);

. That will be more nice to see.
Regards,
Jerin.