AT89C51 Alarm clock to Timer

[skmdmasud]

AT89C51 Alarm clock to 24hr repeating ON Off Timer

I finally thought about making a timer using micro controller. I know few things about computer software programming which is a bit related to C for micro controller so I am giving it a go..

I am trying to modify the following code which is a simple alarm clock to on and off timer
View attachment 64927

original CODE:

//Program to make a digital clock with time and alaram setting function

#include<reg51.h>
#define port P1
#define cont_port P3
#define dataport P2  //Data port for LCD
#define m_sec 10
sbit rs = cont_port^0;
sbit rw = cont_port^1;
sbit en = cont_port^6;
sbit dig_hr1=port^0;
sbit dig_min1=port^1;
sbit start=port^2;
sbit am_pm=port^3;
sbit alarm_set=port^4;
sbit alarm=port^7;
int hr ,hr1=0,alarm_hr=0;
int min,min1=0,alarm_min=0;
int sec,sec1=0,dig_am_pm=0,alarm_am_pm=0;

void delay(unsigned int msec) // Time delay function
{
int i,j ;
for(i=0;i<msec;i++)
  for(j=0;j<1275;j++);
}

                
void lcd_cmd(unsigned char item) //Function to send command to LCD
{
dataport = item;
rs= 0;
rw=0;
en=1;
delay(1);
en=0;
return;
}

void lcd_data(unsigned char item) // Function to send data to LCD
{
dataport = item;
rs= 1;
rw=0;
en=1;
delay(1);
en=0;
return;
}

void lcd_data_string(unsigned char *str) // Function to send string to LCD
{
int i=0;
while(str[i]!='\0')
{
  lcd_data(str[i]);
  i++;
  delay(1);
}
return;
}

lcd_data_int(int time_val) // Function to send number to LCD
{
int int_amt;
int_amt=time_val/10;
lcd_data(int_amt+48);
int_amt=time_val%10; 
lcd_data(int_amt+48);
}

void lcd(unsigned char str1[10])
{
lcd_cmd(0x38); 
lcd_cmd(0x0e); 
delay(1);
lcd_data_string(str1);
}

void set_hr1() // // Function to set hours digit of clock
{
hr1++;
if(hr1>11)
hr1=0;
lcd_cmd(0xc3);
lcd_data_int(hr1);
lcd_data(':');
}

void set_min1()  // Function to set minutes digit of clock
{
min1++;
if(min1>59)
min1=0;
lcd_cmd(0xc6);
lcd_data_int(min1);
}

void set_alarm_hr1()  // Function to set hours digit of alarm
{
alarm_hr++;
if(alarm_hr>11)
alarm_hr=0;
lcd_cmd(0xc3);
lcd_data_int(alarm_hr);
lcd_data(':');
}

void set_alarm_min1() // Function to set minutes digit of Alarm
{
alarm_min++;
if(alarm_min>59)
alarm_min=0;
lcd_cmd(0xc6);
lcd_data_int(alarm_min);
}

void alarm_check()   // Function to check alarm
{
if(hr==alarm_hr)
{
  if(min==alarm_min)
  {
   if(dig_am_pm==alarm_am_pm)
   {
   alarm=1;
   lcd_cmd(0xc5);
   lcd("ALARM");
   delay(10);
   alarm = 0;
   }
  }
} 
}

void main()
{
int k;
start=1;
dig_hr1=1;
dig_min1=1;
alarm_set=1;
alarm=0;
lcd_cmd(0x83);
lcd("SET ALARM");
lcd_cmd(0xc3);
lcd_data_int(hr1);
lcd_data(':');
lcd_data_int(min1);

while(alarm_set==0)
{
  delay(10);
  if(dig_hr1==0)
  set_alarm_hr1();
  if(dig_min1==0)
  set_alarm_min1();
}

if(am_pm==0)
{
  lcd_cmd(0xc8);
  lcd_data_string("am");
  alarm_am_pm=0;
}

if(am_pm==1)
{
  lcd_cmd(0xc8);
  lcd_data_string("pm");
  alarm_am_pm=1;
}
delay(200);
lcd_cmd(0x01);
lcd_cmd(0x83);
lcd("SET TIMING");
lcd_cmd(0xc3);
lcd_data_int(hr1);
lcd_data(':');
lcd_data_int(min1);

while(start==0)
{
  delay(10);
  if(dig_hr1==0)
  set_hr1();
  if(dig_min1==0)
  set_min1(); 
}

if(am_pm==0)
{
  lcd_cmd(0xc8);
  lcd_data_string("am");
  dig_am_pm=0;
}

if(am_pm==1)
{
  lcd_cmd(0xc8);
  lcd_data_string("pm");
  dig_am_pm=1;
}
delay(200);
lcd_cmd(0x01);
while(1)
{
  for(k=0;k<2;k++)
  {
   for(hr=hr1;hr<12;hr++)
   {
    for(min=min1;min<60;min++)
    {
     for(sec=0;sec<60;sec++)
     {
      lcd_cmd(0x82);
      delay(1);
      lcd_data_int(hr);
      lcd_data(':');
      lcd_data_int(min);
      lcd_data(':');
      lcd_data_int(sec);
      if(dig_am_pm==0)
      {
       lcd("am");
      }
      else
      {
       lcd("pm");
      }
      delay(80);
      alarm_check();
      lcd_data_string("    ");
     }
    }
   min1=0;
   }
  if(dig_am_pm==0)
  dig_am_pm=1;
  else
  dig_am_pm=0;
  hr1=0;
  }
}
}

circuit modification will be simple, i will just put a relay instead of the buzzer and another switch for alarm2 in pin 6 most probably.

coding part - i need help. i am assuming "alarm as timer on" and "alarm2 as timer off" and did some additions like following:
------------------------------------------------------

sbit alarm_set2=port^5; //----port 5---

//variable declaration
int alarm2_hr=0;
int alarm2_min=0;
int alarm2_am_pm=0;
//---------------------------


void set_alarm2_hr()  // Function to set hours digit of alarm2 or off time for timer
{
alarm2_hr++;
if(alarm_hr2>11)
alarm2_hr=0;
lcd_cmd(0xc3); -----what does this lines do
lcd_data_int(alarm2_hr);
lcd_data(':');
}

void set_alarm2_min() // Function to set minutes digit of Alarm2
{
alarm2_min++;
if(alarm2_min>59)
alarm2_min=0;
lcd_cmd(0xc6);
lcd_data_int(alarm2_min);
}

void alarm2_check()   // Function to check alarm2
{
if(hr==alarm2_hr)
{
  if(min==alarm2_min)
  {
   if(dig_am_pm==alarm2_am_pm)
   {
   alarm2=1;
   lcd_cmd(0xc5); //---------??
   lcd("OFF Time");
   delay(10); //-----some code to keep in a fixed state----??
   alarm2 = 0; //may be this will be 1 
   }
  }
}

---------------------------
inside void main

alarm2_set=1;
alarm2=0;

lcd_cmd(0x83); -------??
lcd("SET ALARM2")
//--------at the end of the code it will check both alarm
alarm_check();
alarm2_check();

-------------------------------------------------------------------------------

I know there are lots of thing missing and wrong as well. Please help to correct the code.


code source: **broken link removed**

Thanks and Regards