Search found 682 matches

This must be a joke or a homework assignment.
sahu77has answered his own question within the question itself.
There are no variable definitions.
There is an error in the third condition in the inte ...

is this a homework project?

As the opto's are one way (a slave has no way of knowing if another slave is transmitting) and you turn ALL slaves on and off at the same time then there is only one way ...

I biggest error I can see is that your function Final_Delay_Time () definition does not include a return type!

Not sure how the compiler would handle that.

float Final_Delay_Time () ?????????

You expect me to download that zip file? Why?
it would be nice if you actually asked a question and explained why you posted this.

If it is, as I expect a driver you have written which you wish to ...

So errrr, whats wrong with it ?

also
output_toggle(PIN_B3);

should work just as well Smile

And set_tris should not be required in this situation.

So if you dont call the wait_for_one_press() function in main the leds flash ?

I suspect your putton is not working, prob due to hardware.
how have you got it wired up?
Pull UP, Pull down?

What are the warnings?
What does it do?

Your problem is most likely due to bounce.
Do you find that when you press the button that the LEDS count up realy fast, prob too fast to notice and then s ...

This is always true

if (cmd != 0x09 || cmd !=0x11)

cmd cannot be 0x09 and 0x011 at the same time so at least one of these conditions will allways be true Smile

What are the symptoms of i ...

As PCM programmer states, it is almost impossible to help you without a test program but saying that I would say your problem is in the LED_On function ;)

I expect it is doing something it shouldn ...

This line will compile but is most likely wrong :-

if(r1=r2)

What you are doing is an assignment, set r1 to have the same value as r2 and if it is not 0 (not FALSE but TRUE >0) ...

is lenbuff2 a var or a #define or const ?

in this case it would usually be a #define which would reduce the code required when compiled. if it is a var then the compiler has to treat it as a variab ...

You also have another problem
From the help.

Syntax:
i8 = MAKE8(var, offset)

Parameters:
var is a 16 or 32 bit integer.

offset is a byte offset of 0,1,2 or 3.

It appears MAKE8 does ...

void send_packet (float *data) //
{
int i;
tx_buffer[0]=stx;
tx_buffer[1]=0x12;//cop_send temp
tx_buffer[2]=address;
tx_buffer[3]= ...

Something like this should do what you want.
I have not tested it. I don't know if it reduces your code.
It may be possible to improve on this based on the rest of your code.
Someone here may have ...