Continue to Site

Welcome to EDAboard.com

Welcome to our site! EDAboard.com is an international Electronics Discussion Forum focused on EDA software, circuits, schematics, books, theory, papers, asic, pld, 8051, DSP, Network, RF, Analog Design, PCB, Service Manuals... and a whole lot more! To participate you need to register. Registration is free. Click here to register now.

Register Log in

[PIC] Problem on looping, The condition get repeated when it' true.

Status
Not open for further replies.
E

Emmanuel1992

Junior Member level 2
Joined
Nov 24, 2016
Messages
22
Helped
0
Reputation
0
Reaction score
0
Trophy points
1
Activity points
253
Hello guys I have the problem, I have tried to make the program on mikroc pro for pic to control street lights automaticaly by using LDR and also the system check for the status of all lights and i have tried to build it and no error on it, but the problem is when I simulate on proteus and when a specific condition met that condition get repeated continuously it doesn't stop for only one condition. any one who have an idea can help me to solve that issue coz am suppose to submit it soon as possible below is my full code.
Am using pic18f452 microcontroller


Code C - [expand]
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238

// LCD module connections
sbit LCD_RS at RC0_bit;
sbit LCD_EN at RC1_bit;
sbit LCD_D4 at RC2_bit;
sbit LCD_D5 at RC3_bit;
sbit LCD_D6 at RC4_bit;
sbit LCD_D7 at RC5_bit;
 
sbit LCD_RS_Direction at TRISC0_bit;
sbit LCD_EN_Direction at TRISC1_bit;
sbit LCD_D4_Direction at TRISC2_bit;
sbit LCD_D5_Direction at TRISC3_bit;
sbit LCD_D6_Direction at TRISC4_bit;
sbit LCD_D7_Direction at TRISC5_bit;
// End LCD module connections
 
void Init_Mobile();
int count=0;
void Init_Mobile()
{
    Delay_ms(1000);
    UART1_Write_Text("AT\r\n");
    Delay_ms(500);
    UART1_Write_Text("ATE0\r\n");
    Delay_ms(500);
    UART1_Write_Text("AT+CMGF=1\r\n");
    Delay_ms(500);
}
 
 
 void main()
{
 
  TRISD=0X00;
  PORTD=0;
  TRISB=0XFF;
  TRISC=0b11000000;
  Lcd_Init();
  UART1_init(9600);
  Delay_ms(1000);
  Lcd_Cmd(_LCD_CLEAR);
  Lcd_Cmd(_LCD_CURSOR_OFF);
  Lcd_Out(2,1,"SYSTEM");
  Lcd_Out(3,-3,"INITIALIZING....");
  Delay_ms(2500);
  Lcd_Cmd(_LCD_CLEAR);
  
  Lcd_Out(1,2,"UNIVERSITY OF");
  Lcd_Out(2,2,"DAR ES SALAAM");
  Lcd_Out(4,-1,"*** UDSM ***");
  Delay_ms(2500);
  Lcd_Cmd(_LCD_CLEAR);
 
  Lcd_Out(1,1,"AUTOMATIC STREET");
  Lcd_Out(2,1,"LIGHTS AND FAULT");
  Lcd_Out(3,-3,"DETECTION SYSTEM");
  Init_Mobile();
 
 
 
while(1)
{
   if(PORTB.F7==0)
   {
   PORTD=0b00000000;
   Delay_ms(100);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"LIGHTS TURNED");
   Lcd_Out(2,1,"OFF SUCCESSFULLY");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LIGHTS TURNDED OFF SUCCESSFULLY");
   UART1_Write(26);
   Delay_ms(100);
   }
   
  if(PORTB.F7==1)
  {
   PORTD=0b00111111;
   Delay_ms(500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"LIGHTS TURNED");
   Lcd_Out(2,1,"ON SUCCESSFULLY");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
    UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LIGHTS TURNDED ON SUCCESSFULLY");
   UART1_Write(26);
   Delay_ms(100);
  }
 if((PORTB.F0==1)&&(PORTD.F0=1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 1");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 1");
   UART1_Write(26);
   Delay_ms(100);
  }
 
  if((PORTB.F1==1)&&(PORTD.F1==1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 2");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 2");
   UART1_Write(26);
  }
 
   if((PORTB.F2==1)&&(PORTD.F2==1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 3");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 3");
   UART1_Write(26);
  }
 
  if((PORTB.F3==1)&&(PORTD.F3==1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 4");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 4");
   UART1_Write(26);
  }
 
  if((PORTB.F4==1)&&(PORTD.F4==1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 5");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 5");
   UART1_Write(26);
  }
 
   if((PORTB.F5==1)&&(PORTD.F5==1))
  {
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"FAULT DETECTED");
   Lcd_Out(2,1,"ON LIGHT 6");
   Lcd_Out(4,-3,"SENDIND SMS.....");
   Delay_ms(1500);
   Lcd_Cmd(_LCD_CLEAR);
   Lcd_Out(1,1,"AUTOMATIC STREET");
   Lcd_Out(2,1,"LIGHTS AND FAULT");
   Lcd_Out(3,-3,"DETECTION SYSTEM");
   UART1_Write_Text("AT+CMGS=");
   UART1_Write(0x22);
   UART1_Write_Text("+255717677776");
   UART1_Write(0x22);
   UART1_Write(0x0d);
   Delay_ms(200);
   UART1_Write_Text("LFAULT DETECTED ON LIGHT 6");
   UART1_Write(26);
  }
 
 }
 }
 
Last edited by a moderator:

but the problem is when I simulate on proteus and when a specific condition met that condition get repeated continuously it doesn't stop for only one condition
Click to expand...

You did not specify which condition. Moreover, did you measured on simulation if these signals are really reaching the true logic (I mean, close to Vcc) ?
 

andre_teprom said:
You did not specify which condition. Moreover, did you measured on simulation if these signals are really reaching the true logic (I mean, close to Vcc) ?
Click to expand...


Thank you for your reply, the problem is that, I want to design a system that will control street lights automatically by using LDR which i have included it on the external ADC circuit and finally the output of of ADC circuit get connected to PORTB.F0 of pic18f452 microcontroller so, in my code that you have seen above if a certain if condition met then action must be taken only on that condition while skipping other condition and the met condition should be executed only once, but on that code above the true condition get executed by repeating itself continuously it doesn't stop please help me solve that issue.
 

hi

is if((PORTB.F0==1)&&(PORTD.F0=1))
this condition is getting executed repeatedly ?

- - - Updated - - -

hi

check for adc is working ok . how is it working you have one ldr and extern adc?

if((PORTB.F0==1)&&(PORTD.F0=1))
here both conditions are getting satisfied hence its executing same code again whats PORTD.F0 ?
 

tasuki said:
hi

is if((PORTB.F0==1)&&(PORTD.F0=1))
this condition is getting executed repeatedly ?

- - - Updated - - -

hi

check for adc is working ok . how is it working you have one ldr and extern adc?

if((PORTB.F0==1)&&(PORTD.F0=1))
here both conditions are getting satisfied hence its executing same code again whats PORTD.F0 ?
Click to expand...


Hi, thank you for the reply, all conditions on the above code get executed by repeating itself when the condition is true they doesn't executed only once when it's true that is the problem am facing too
 

betwixt said:
Read that VERY carefully!
Hint: Is it missing something.

Brian.
Click to expand...

yes its if((PORTB.F0==1)&&(PORTD.F0=1)) it should be if((PORTB.F0==1)&&(PORTD.F0==1))
 

Re: Problem on looping, The condition get repeated when it's true.

tasuki said:
yes it's 
Code VHDL - [expand]
1

if((PORTB.F0==1)&&(PORTD.F0=1))

it should be 
Code VHDL - [expand]
1

if((PORTB.F0==1)&&(PORTD.F0==1))
Click to expand...

Hi, I have tried to change that but nothing happen, the problem is still there any suggestion please
 
Last edited by a moderator:

hi

can you please tell me PORTB.F0 means what is it ? like I have gone through datasheet also it has PORTB RB0:RB7. what is F0?
here it seems two port pins your taking as input and checking its status but what is F0?
also PORTD.F0 whats on this pin ?

- - - Updated - - -

Hi
also instead of checking port pin directly you better set some flag for your conditions and reset them after executing perticular task. that will avoid this situation
 

tasuki said:
hi

can you please tell me PORTB.F0 means what is it ? like I have gone through datasheet also it has PORTB RB0:RB7. what is F0?
here it seems two port pins your taking as input and checking its status but what is F0?
also PORTD.F0 whats on this pin ?

- - - Updated - - -

Hi
also instead of checking port pin directly you better set some flag for your conditions and reset them after executing perticular task. that will avoid this situation
Click to expand...



hi, F0, it's used to specify bit on specific pin port, so PORTB.F0 it mean pin RB0 according to mikroc compiler, then can you help me to set flag on if statements coz am not familiar with it can you try to fix my code above please by putting flag on it as you said ?
 

I want to design a system that will control street lights automatically by using LDR which i have included it on the external ADC circuit and finally the output of of ADC circuit get connected to PORTB.F0 of pic18f452 microcontroller
Click to expand...

What range of values you took from the LDR ?
Have you added any capacitor to suppress noise ?
 

Oh. So many strings in the RAM. You could have placed them in ROM and used CopyConst2Ram() function to get the required string from ROM to RAM.
 

Okada said:
Oh. So many strings in the RAM. You could have placed them in ROM and used CopyConst2Ram() function to get the required string from ROM to RAM.
Click to expand...



Hi, can you show the format on how to use that function CopyConst2Ram() coz am not familiar with it please..

- - - Updated - - -

andre_teprom said:
What range of values you took from the LDR ?
Have you added any capacitor to suppress noise ?
Click to expand...


Hi, I have included LDR on ADC circuit then the output of ADC circuit was connected to digital pin of PIC which is RB0
 

Hi, I have included LDR on ADC circuit then the output of ADC circuit was connected to digital pin of PIC which is RB0
Click to expand...
Even if you are "busy" to attach any drawings, at least can you depict exactly how did you do that? I mean, is not there any pull up or anything else?
 

andre_teprom said:
Even if you are "busy" to attach any drawings, at least can you depict exactly how did you do that? I mean, is not there any pull up or anything else?
Click to expand...



ok i have attached a schematic Screenshot (12).png
 

Looking at the overall logic of your code, you have a main loop that contains a series of conditional statements. Looking at the 'true' condition statement of the if statement you are complaining about, it contains a whole series of statements to output o the LCD, the UART and some delay statements.
As far as I can see, there is nothing in that sequence that will change anything on PORTB0 and PORTD0. Therefore when the loop completes and starts again, those same conditions may well still be there and the statements will be executed again.
If you don't want this to happen then you need to alter the logic so that either you wait until the external conditions have changed, you set a flag to say "I've seen that already' (and clear the flag when the conditions do alter) or whatever is necessary.
General comment: you have an awful lot of repeated code there with the only difference being the bits that are tested and the light number being checked. You could put a lot of that code into a function which is called with the light number that has failed and make it a whole lot easier to write and maintain the code base.
Susan
 

Emmanuel1992 said:
hi, F0, it's used to specify bit on specific pin port, so PORTB.F0 it mean pin RB0 according to mikroc compiler, then can you help me to set flag on if statements coz am not familiar with it can you try to fix my code above please by putting flag on it as you said ?
Click to expand...


hi
here is simple way i could suggest

Code: 
   take some flags 
    // keep initial values  0
    char  status_updated_for_status_1 = 0;
    char  status_updated_for_status_2 = 0;

  // make one function that checkes only stauts of ports 
   void check_port_status()
   {
     
     if(PORTB.F7==0)
      {
            if(status_updated_for_status_1 == 0)
            {
               Status = 1;    // define micros for 1 , 2 like #define 1 LIGHTS_OFF like that in .h file
               status_updated_for_status_2 = 0;
            }
       }

     if(PORTB.F7==1)
     {
          if(status_updated_for_status_2 == 0)
          {
              Status = 2;
              status_updated_for_status_1 = 0;
          }
     }
   // .... other portscomes here

  }

//in your while loop 

  while(1)
  {
     switch(status)
     {
       case 1 : 
        status_updated_for_status_1 = 1;
        // put your LCD code & uart code here. i.e. action to be taken
        Status = 0;
        break ;

     case 2 : 
       
       status_updated_for_status_2 = 1;
         // put your LCD code & uart code here. i.e. action to be taken
         Status = 0;
       break ;

     //....  other cases
   }
  
}
- - - Updated - - -

Hi

one more I will suggest instead of char flags use bitfields , Status 1,2 you define in .h file like #define LIGHTS_OFF 1
make repeated code common , i.e. delays after every uart transmission inlcude that delay into uart transmit code.

//above logic
while checking port your making Status value updated , in while loop in switch statement your taking action according to status and making Status =0; so that perticular set of action wont be executed again. same time your making flag status_updated_for_status_1 = 1 so in void check_port_status() function even if your port is at same condition it wont set Status to 1, next time when port status changes it will reset that flag to zero.

hope it will work try this way, this is logic I could think in short time
 
Last edited by a moderator:

ADC and Comparators are not disabled.
 

tasuki said:
hi
here is simple way i could suggest

Code: 
   take some flags 
    // keep initial values  0
    char  status_updated_for_status_1 = 0;
    char  status_updated_for_status_2 = 0;

  // make one function that checkes only stauts of ports 
   void check_port_status()
   {
     
     if(PORTB.F7==0)
      {
            if(status_updated_for_status_1 == 0)
            {
               Status = 1;    // define micros for 1 , 2 like #define 1 LIGHTS_OFF like that in .h file
               status_updated_for_status_2 = 0;
            }
       }

     if(PORTB.F7==1)
     {
          if(status_updated_for_status_2 == 0)
          {
              Status = 2;
              status_updated_for_status_1 = 0;
          }
     }
   // .... other portscomes here

  }

//in your while loop 

  while(1)
  {
     switch(status)
     {
       case 1 : 
        status_updated_for_status_1 = 1;
        // put your LCD code & uart code here. i.e. action to be taken
        Status = 0;
        break ;

     case 2 : 
       
       status_updated_for_status_2 = 1;
         // put your LCD code & uart code here. i.e. action to be taken
         Status = 0;
       break ;

     //....  other cases
   }
  
}
- - - Updated - - -

Hi

one more I will suggest instead of char flags use bitfields , Status 1,2 you define in .h file like #define LIGHTS_OFF 1
make repeated code common , i.e. delays after every uart transmission inlcude that delay into uart transmit code.

//above logic
while checking port your making Status value updated , in while loop in switch statement your taking action according to status and making Status =0; so that perticular set of action wont be executed again. same time your making flag status_updated_for_status_1 = 1 so in void check_port_status() function even if your port is at same condition it wont set Status to 1, next time when port status changes it will reset that flag to zero.

hope it will work try this way, this is logic I could think in short time
Click to expand...




Hi, Thank you for the reply, Sorry can you show me through my code above by using your suggestion by including few conditions in my code above other condition i will complete myself i need to see that by using few of conditions in my above code please help me..

- - - Updated - - -

Emmanuel1992 said:
Hi, Thank you for the reply, Sorry can you show me through my code above by using your suggestion by including few conditions in my code above other condition i will complete myself i need to see that by using few of conditions in my above code please help me..
Click to expand...


hi again, Then what about other conditions that i have included an AND operator(&&) how will look by using your suggestion? coz you show me only one condition that have only one port.
 

hi

it is using your code only you just need to check carefully follow these steps
1) in void check_port_status() function mentioned all your port checking activities i.e. PORTB.F7 ==0 , PORTB.F7 == 1 etc. set different status 0,1 ,2 .. for them.
2) in while loop put relavent action codes which you have already written in if.. condition
3) set- reset the flags

structure is already ready.

- - - Updated - - -

Emmanuel1992 said:
Hi, Thank you for the reply, Sorry can you show me through my code above by using your suggestion by including few conditions in my code above other condition i will complete myself i need to see that by using few of conditions in my above code please help me..

- - - Updated - - -




hi again, Then what about other conditions that i have included an AND operator(&&) how will look by using your suggestion? coz you show me only one condition that have only one port.
Click to expand...

hi
same you can impleament with all your conditions including (&&) conditions...

- - - Updated - - -

hi
pls refer this hope it will help

Code:
Code: 
#define Light_Turn_on         1
#define Light_Turn_off         2
#define Light_fault_detected 3
//.... other status comes here

// take some flags keep initial values  0
    char  status_updated_for_status_1 = 0;
    char  status_updated_for_status_2 = 0;
     char  status_updated_for_status_3 = 0;
//.... other flags comes here (use bitfields for memory saving ... )


  // make one function that checkes only stauts of ports 
   void check_port_status()
   {
     
     if(PORTB.F7==0)
      {
            if(status_updated_for_status_1 == 0)
            {
               Status = Light_Turn_on ;   
               status_updated_for_status_2 = 0;
               status_updated_for_status_3 = 0;
            }
       }

     if(PORTB.F7==1)
     {
          if(status_updated_for_status_2 == 0)
          {
              Status = Light_Turn_off;
              status_updated_for_status_1 = 0;
              status_updated_for_status_3 = 0;
          }
     }
     if((PORTB.F0==1)&&(PORTD.F0=1))
    {
     if(status_updated_for_status_3 == 0)
          {
              Status =Light_fault_detected;
              status_updated_for_status_1 = 0;
              status_updated_for_status_2 = 0;
          }
     }
   // .... other ports comes here

  }

//in your while loop 

  while(1)
  {
     switch(status)
     {
       case Light_Turn_on : 
        status_updated_for_status_1 = 1;
        // put your LCD code & uart code here. i.e. action to be taken

 [B]  // this part taken from your code please modify this .... too bulky .. for time being taken from your code[/B]
         PORTD=0b00111111;
         Delay_ms(500);
         Lcd_Cmd(_LCD_CLEAR);
         Lcd_Out(1,1,"LIGHTS TURNED");
         Lcd_Out(2,1,"ON SUCCESSFULLY");
         Lcd_Out(4,-3,"SENDIND SMS.....");
         Delay_ms(1500);
         Lcd_Cmd(_LCD_CLEAR);
         Lcd_Out(1,1,"AUTOMATIC STREET");
         Lcd_Out(2,1,"LIGHTS AND FAULT");
         Lcd_Out(3,-3,"DETECTION SYSTEM");
         UART1_Write_Text("AT+CMGS=");
         UART1_Write(0x22);
         UART1_Write_Text("+255717677776");
         UART1_Write(0x22);
         UART1_Write(0x0d);
         Delay_ms(200);
        UART1_Write_Text("LIGHTS TURNDED ON SUCCESSFULLY");
        UART1_Write(26);
         Delay_ms(100);
        Status = 0;
        break ;

     case Light_Turn_off : 
       
       status_updated_for_status_2 = 1;
         // put your LCD code & uart code here. i.e. action to be taken 
         Status = 0;
       break ;

     //....  other cases
   }
  
}
 
Last edited by a moderator:

Status
Not open for further replies.

Similar threads

Part and Inventory Search

Welcome to EDABoard.com

Sponsor

Back
Top