Problem mit Funktion bei GSM-Modul über USART

[Bene J. (terfagter)]

Hallo zusammen,

Ich habe eine Funktion geschrieben, die die Signalstärke eines 
GSM-Moduls auswertet. Das ganze funktioniert über AT-Befehle.
Ich sende dem Modul AT+CSQ<ENTER> und das Modul antwortet mit (z.B.) 
+CSQ: 13,99 wobei 13 die endscheidende Zahl ist, die ich benötige. Meine 
Funktion funktioniert beim ersten Durchlauf wunderbar, beim zweiten 
Durchlauf bleibt die Funktion bei while(!(0 == ukbhit2())); am Anfang 
der Funktion hängen.
Also sendet das Modul nichts mehr zurück beim zweiten Durchlauf! Wenn 
ich die AT-Befehle aber über ein Terminal an das GSM-Modul sende, kriege 
ich immer eine Antwort, auch wenn ich jede Sekunde sende. In meiner 
main() rufe ich die Funktion nur alle 3 sec auf. Also Zeit genug sollte 
das Modul haben, also liegt es nicht am Modul.

/*************************************************************************
Function: GsmSignalAbrufen()
Purpose:  Fragt die Signalstärke des GSM-Moduls ab.
Input:    none
Returns:  Status 0...1 schlechter Empfang, 11...31 guter Empfang, 99 nicht feststellbar
**************************************************************************/
char gsmtemp2[3];
char *GsmSignalAbrufen(void)
{
  uint8_t i=0;
  char gsmzeichen;
  char gsmsatz[20];
  char gsmtemp[9];
    

  _delay_ms(100);
  
  uputs2("AT+CSQ\r\n");              //Eingabe AT-Befehl
  uputs0("3. Schritt\n");
  while(!(0 == ukbhit2()));
  uputs0("4. Schritt\n");
  while (ugetchar2() != 10);            //Erste Zeile bis Linefeed (\r\n) einlesen und verwerfen
  do                        // Kompletten Datensatz empfangen
  {
    gsmzeichen = ugetchar2();          // Zeichen von USART einlesen
    gsmsatz[i] = gsmzeichen;          // Zeichenkette erweitern
    i++;
  } while (gsmzeichen != 10);          // Zeilende v. Datensatz erkennen
    
  strncpy(gsmtemp, gsmsatz, 8);          //Die ersten 8 Zeichen des Strings kopieren
  gsmtemp[8] = '\0';  
  strrev(gsmtemp);                //String umdrehen
  strncpy(gsmtemp2, gsmtemp, 2);          //Die ersten 2 Zeichen des Strings kopieren
  gsmtemp2[2] = '\0';                
  strrev(gsmtemp2);                //String wieder umdrehen
    
  if(!(0 == ukbhit2()))              //Dritte Zeile anholen und verwerfen
    while (ugetchar2() != 10);
  
  return gsmtemp2;
}

Woran könnte das liegen? Schonmal vielen Dank für eure Hilfe.

[Bene J. (terfagter)]

Noch zusätzlich:
Ich habe keine Datenflußkontrolle aktiv.
Auch delay-Schleifen an verschiedensten Stellen in der Funktion, ergaben 
keine Besserung.
Wieso sendet das Modul nichts zurück, aber mit einem Terminal schon?!?

[Stefan E. (sternst)]

Unabhängig von deinem eigentlichen Problem (dazu müsste man nämlich mal 
wissen, welchen USART-Code du überhaupt verwendest, damit man mal weiß, 
was ukbhit2 überhaupt macht, oder machen soll):

  strncpy(gsmtemp, gsmsatz, 8);          //Die ersten 8 Zeichen des Strings kopieren
  gsmtemp[8] = '\0';  
  strrev(gsmtemp);                //String umdrehen
  strncpy(gsmtemp2, gsmtemp, 2);          //Die ersten 2 Zeichen des Strings kopieren
  gsmtemp2[2] = '\0';                
  strrev(gsmtemp2);

Wozu einfach, wenn es auch kompliziert geht, gell? ;-)

strncpy(gsmtmp2, gsmsatz+6, 2);
gsmtmp2[2] = '/0';

[Bene J. (terfagter)]

Ah ok. Ist natürlich einfacher. Was dazu gelernt...

[Bene J. (terfagter)]

Problem gelöst:
Ich habe bevor ich den AT-Befehl übergebe noch zwei kleine 
while-Schleifen hinzugefügt, die überprüfen ob Zeichen vorhanden sind 
und diese dann bis zum Zeilenende verwerfen.

Damit sieht meine Funktion so aus:

char gsmtmp[3];
char *GsmSignalAbrufen(void)
{
  uint8_t i=0;
  char gsmzeichen;
  char gsmsatz[20];
  
  while (!(0 == ukbhit2()))
    while (ugetchar2() != 10);

  uputs2("AT+CSQ\r\n");              //Eingabe AT-Befehl
  
  while (ugetchar2() != 10);            //Erste Zeile bis Linefeed (\r\n) einlesen und verwerfen
  do                        // Kompletten Datensatz empfangen
  {
    gsmzeichen = ugetchar2();          // Zeichen vo USART einlesen
    gsmsatz[i] = gsmzeichen;          // Zeichenkette erweitern
    i++;
  } while (gsmzeichen != 10);          // Zeilende v. Datensatz erkennen
    
  strncpy(gsmtmp, gsmsatz+6, 2);          //Benötigte Zahl in gsmtmp speichern
  gsmtmp[2] = '\0';                //Stringende manuell setzen

  return gsmtmp;
}

Kann ich die Funktion noch verbessern?

[gascht (Gast)]

im grunde kann man einiges anders machen ...

weil man das modul ja auch  für andere befehle brauch
kann man  einige sachen verallgemeinern

zB das wegsenden als funktion ..
entweder mit timeout und verwerfen oder neu senden

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Abgesehen von dem obigen Diskussionsgegenstand gibt es noch ein weiteres 
Problem mit dem aufgeführten Programmcode.

Die Startadresse des Arrays gsmtmp[] wird zurückgegeben. Leider handelt 
es sich dabei jedoch um eine lokale Variable, deren Gültigkeit mit dem 
Verlassen von GsmSignalAbrufen() endet. Zwar ist der zurückgegebene 
Zeiger gültig, jedoch zeigt er auf einen freigegebenen Speicherbereich. 
Das kann zwar gutgehen, da der Speicher nicht sofort gelöscht wird, 
muss es aber nicht. Sobald der Stack z.B. durch weitere Funktionsaufrufe 
wieder neu
belegt wird, werden auch die Daten überschrieben.

Es gibt mehrere Möglichkeiten, die obigen Probleme zu vermeiden:

1. Übergabe eines vorab definierten Puffers an GsmSignalAbrufen()

2. Anforderung von dynamischem Speicher innerhalb von GsmSignalAbrufen()
   und Freigabe nach Verarbeitung des Inhaltes. ACHTUNG: SEHR SCHLECHTER
   PROGRAMMIERSTIL, da äußerst fehlerträchtig.

3. Deklaration von gsmtmp[] als static-Variable. ACHTUNG: Mäßig
   schlechter Programmierstil, da hierbei implizit angenommen wird, dass
   die Adresse einer lokalen Variable auch außerhalb der Funktion
   wohldefiniert und konstant ist.

[Stefan E. (sternst)]

Andreas Schweigstill schrieb:
> Die Startadresse des Arrays gsmtmp[] wird zurückgegeben. Leider handelt
> es sich dabei jedoch um eine lokale Variable

Schau nochmal genau hin.

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Stefan Ernst schrieb:
> Schau nochmal genau hin.

Upps, da habe ich mich in der Tat verguckt. Ich war gedanklich noch bei 
gsmtemp2[] aus dem ersten Quelltext, wo jedoch auch alles richtig 
übergeben wird. :-(