Filtering $GPRMC string

On one hand - what happens if you try to send 70 characters to the transmit register of your serial port more or less instantly? Do you think the serial port can send out one character every microsecond?

But are you really sure about your 70?

What do you think happens here?

          else if( GPSData == 'C' && (RMC_Pos == 5) )
          {
                        RMC_Pos=6;
            GPSReceiveCtr=0;
          }

To get RMC_Pos=7, it first have to be 6.

Think again about the meaning of the GPSReceiveCtr=0 line.

On one hand - what happens if you try to send 70 characters to the transmit register of your serial port more or less instantly? Do you think the serial port can send out one character every microsecond?

But are you really sure about your 70?

What do you think happens here?

          else if( GPSData == 'C' && (RMC_Pos == 5) )
          {
                        RMC_Pos=6;
            GPSReceiveCtr=0;
          }

To get RMC_Pos=7, it first have to be 6.

Think again about the meaning of the GPSReceiveCtr=0 line.

Hi Per Westermark,

Please gothrough this code,

while(1);
}
void serial (void) interrupt 4
{
if(RI)
{
  RI=0;
  uc=SBUF; // receives the data from rx into the buffer
  SBUF=uc; // transmits the data to tx from the buffer
}
if(TI)TI=0;
}

This code is to receive the gps data and transmitts the gps data from the controller. IN 1hertz gps will be sending $GPRMC, $GPGGA, $GPGTA, $GPRMA datas so if the microcontroller can able to receive and transmitt those data's in one second

whats strong with the second code that i wrote for filtering the $GPRMC string, it is going to send only the Gprmc string. In previous code it was sending all the strings, there was no problem occured while sending all the strings from the buffer. In one second more than 300 characters was send to the hyperterminal from 89c51 TX pin.

Please correct me if am wrong.

I don't think there's anything particularly "strong" about it - but it certainly looks wrong!

Have you actually looked at the FAQs list that I suggested earlier about 8051 serial comms?

In particular, have you actually looked at the Tutorial that I suggested earlier about 8051 serial comms?

Your ISR tests both RI and TI, and yet you do nothing in the TI section - do you think that's right...?

Your GPS sends data serially. If you are able to retransmit with at least the same baudrate as the GPS, then you don't gain anything by filtering away all lines but the $GPRMC - but you do lose the information about the number of satellites, and any signal strengths etc. That is a significant loss, so I wouldn't call your optimization a "strong" solution. With only the $GPRMC, you will not even know if the GPS is in 2D or 3D mode.

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