Filtering $GPRMC string

Please can you rectify the code for me, its already too late i need to submit the project, my college is going to open next week, please do help me, am electrical student, i know only "C" but embedded c is quite complicated for me, as this project technical sounded good i choosed this project, but if i can't able to complete it my team members will sout me. please understand my pressure and complete the code for me.

Than you.

Please can you rectify the code for me, its already too late i need to submit the project, my college is going to open next week, please do help me, am electrical student, i know only "C" but embedded c is quite complicated for me, as this project technical sounded good i choosed this project, but if i can't able to complete it my team members will sout me. please understand my pressure and complete the code for me.

Than you.

Sorry, it's your task - the object of the exercise is for you to demonstrate your competence in the subject.

To do it for you would be cheating.

If you can't complete the required assignment, then you need to discuss the matter urgently with your tutors...

Please andy,

I know this is cheating but there is no time in my hand, i gothrough your faq and tutorial links, but i can able to understand nothing, i have learned about armature, generators, magnetic flux, ac and dc but i never go through buffer, timers, tI, RI, interrupts, in my life time. My tutors also not well sound for guiding me in coding, They guided my in designing hardware by GODs grace i made it sucessfully. The code logic that i wrote was guided by my senior, but he also can't able to solve this problem.
Please slove this problem to me i will be very much thankful to you.

So just don't do it!

Come clean and admit that you cannot complete the assignment.
If you bluff this assignment, how will you cope with the next assignment?

"i have learned about armature, generators, magnetic flux, ac and dc but i never go through buffer, timers, tI, RI, interrupts, in my life time."

So why on earth are you doing a project that so totally depends upon it?!

If you really feel that you have followed all the course requirements, and this assignment is totally unrelated to any of the course content, then you need to appeal to your school authorities.

"Please slove this problem to me i will be very much thankful to you."

Actually, you probably won't: If you cheat this assignment, then it will be assumed that you can do this stuff - so you will receive more and harder programming tasks!

Dear all,

Please correct if am going wrong.

/*------------------------------------------------------------------------------
Serial.c
------------------------------------------------------------------------------*/
#include <reg52.H>                /* special function register declarations  */
#include <stdio.h>                /* prototype declarations for I/O functions */
  sbit mybit=P1^1 ;

unsigned char RMC_Pos,GPSReceiveCtr,GPSData,Ctr;
unsigned char  GPSArray[80];
void main(void)
{
RMC_Pos=0;
SCON = 0x50; /* SCON: mode 1, 8-bit UART, enable rcvr */
TMOD |= 0x20; /* TMOD: timer 1, mode 2, 8-bit reload */
TH1 = 0xFa; /* TH1: reload value for 4800 baud @ 11.0592 MHz */
TR1 = 1; /* TR1: timer 1 run */
ES=1;  /* Serial interrupt Enable*/
EA=1;  /* global interrupt enable*/
while(1)
{
if(RMC_Pos==7)
{
  for(Ctr=0;Ctr<GPSReceiveCtr;Ctr++)
  {
    SBUF=GPSArray[Ctr];
        while(TI==0);
    TI=0;
  }
}
RMC_Pos=0;
}
}

void serial (void) interrupt 4

{
  mybit=1;         //making P1.1 high for debugging
while(RI==0);//if(RI)
{
     // RI=0;
      GPSData = SBUF;  //gps datas are stored in buffer named gpsdata
      if(RMC_Pos!=7)
      {
          if( GPSData== '$' && (RMC_Pos == 0) )   //checks for $ in first character of the string
          RMC_Pos=1;
          else if( GPSData == 'G' && (RMC_Pos == 1) ) //checks for G in second character of the string
          RMC_Pos=2;
          else if( GPSData == 'P' && (RMC_Pos == 2) ) //checks for P in third character of the string
          RMC_Pos=3;
          else if( GPSData == 'R' && (RMC_Pos == 3) ) //checks for R in fourth character of the string
          RMC_Pos=4;
          else if( GPSData == 'M' && (RMC_Pos == 4) ) //checks for M in fifth character of the string
          RMC_Pos=5;
          else if( GPSData == 'C' && (RMC_Pos == 5) )//checks for C in sixth character of the string
         // {
            RMC_Pos=6;
            //GPSReceiveCtr=0;
         // }
          else if( GPSData != '*' && (RMC_Pos == 6) )  //checks for the *
                  {
                         GPSReceiveCtr=0;
             GPSArray[GPSReceiveCtr++]=GPSData;            //
                }
          else if( GPSData == '*' && (RMC_Pos == 6) )  // found the whole $gprmc string
          RMC_Pos=7;
          else
          RMC_Pos=0;
          if(GPSReceiveCtr > 80)
                  {       GPSReceiveCtr=80;
                  mybit=0;              //making P1.1 low for debugging
                  }
        }

   }


//if(TI)TI=0;
}

Which is easier to read?

This:

SCON = 0x50; /* SCON: mode 1, 8-bit UART, enable rcvr */
TMOD |= 0x20; /* TMOD: timer 1, mode 2, 8-bit reload */
TH1 = 0xFa; /* TH1: reload value for 4800 baud @ 11.0592 MHz */
TR1 = 1; /* TR1: timer 1 run */
ES=1;  /* Serial interrupt Enable*/
EA=1;  /* global interrupt enable*/

Or this:

SCON  = 0x50; /* mode 1, 8-bit UART, enable rcvr */
TMOD |= 0x20; /* timer 1, mode 2, 8-bit reload   */
TH1   = 0xFa; /* reload value for 4800 baud @ 11.0592 MHz */
TR1   = 1;    /* timer 1 run */
ES    = 1;    /* Serial interrupt Enable */
EA    = 1;    /* global interrupt enable */

People are a lot more likely to want to look at your code if you make it "easy on the eye" - it will help you, too.

See: www.8052.com/.../165484