SMS senden mit ATMEGA32

[Wolfi (Gast)]

hallo!
ich würde gerne mit meinem atmega32 sms versenden. leider treten noch 
einige fehler im programm auf.

hier die fehler:

Warning: C:\...\main\main.c(89): local variable 'c' was set, but not 
used
Warning: C:\...\main\main.c(105): function parameter #1 name: 's' 
doesn't match it's previous declaration: 'szBuf' from line: 38
Warning: C:\...\main\main.c(94): unused function 'uart_putc' was removed 
by the linker
Warning: C:\...\main\main.c(105): unused function 'uart_puts' was 
removed by the linker
Warning: C:\...\main\main.c(70): unused function 'sende_SMS' was removed 
by the linker

ich schreibe die software mit codevisionavr.
hier ist der code:

#include <mega32.h>         //Bibliothek ATMEGA32 einbinden
#include <stdio.h>              //Bibliothek Standard I/O einbinden
#include <delay.h>              //Bibliothek Delay einbinden   
     
int uart_putc(unsigned char c);
void uart_puts (char *szBuf);



#define UDRE 5                   // UART Data Register Empty
    

/*Unterprogramm SMS senden*/
/*******************/       

void sende_SMS()
{
  int c=0;


  if(c==0)
  {
  uart_puts("at+cpin=0000\r\n"); //AT-Befehl zur Pineingabe
  delay_ms(2000);       //2 sec warten   //2 Sekunden Warteschleife
  uart_puts("at+cmgf=1\r\n");   //SMS-Ein und Ausgabe erfolgt
                                       //    im 7Bit Klartext
  delay_ms(2000);
  uart_puts("at+cmgs=+xxxxxxxxxxxxxxx\r\n"); //Zieladressnummer                                                   //nach Österreich A1
  delay_ms(2000);
  uart_puts("Hello World"); //SMS-Text
  delay_ms(2000);
  uart_putc(0x1A);  //Hex Code fuer Strg+Z wird gesendet
  delay_ms(5000);
  c=1;  // Überprüfung das SMS nur 1 mal gesendet wurde
  }
  return;

}
int uart_putc(unsigned char c)
{
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }

    UDR = c;                      /* sende Zeichen c */
    return 0;
}


void uart_puts (char *s)
{
    while (*s)
    {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen" */
        uart_putc(*s);
        s++;
    }
}

ich hoffe es kann mir jemand weiterhelfen.

[Frank S. (franksanderdo)]

Hallo Wolfi,

hast Du keine Funktion main angelegt?


Warning: C:\...\main\main.c(94): unused function 'uart_putc' was removed
by the linker
Warning: C:\...\main\main.c(105): unused function 'uart_puts' was
removed by the linker
Warning: C:\...\main\main.c(70): unused function 'sende_SMS' was removed
by the linker

Der Linker wirft deine ganze Arbeit weg weil die Funktionen nirgends 
aufgerufen werden.

Warning: C:\...\main\main.c(89): local variable 'c' was set, but not
used

Wie er schon sagt. Da gibt es einen Variable die nirgendwo genutzt wird 
;-)

Grüße
Frank

[Wolfi (Gast)]

stimmt...hab die funktion nirgends aufgerufen.
leider bleibt folgender fehler:

Warning: C:\...\main\main.c(89): local variable 'c' was set, but not
used

und das senden funktioniert auch nicht.
benutze ein gsm-modul von siemens (tc35).

[Daniel F. (df311)]

da wir alle wisse, wo in deinem code-snippet die zeile 89 ist, würde ich 
einfach mal sagen, schreib // vor die zeile

[Der Bastler (Gast)]

https://www.mikrocontroller.net/attachment/106245/EPJ8_web.pdf

Lesen und staunen ...

[Wolfi (Gast)]

Der Bastler schrieb:
> https://www.mikrocontroller.net/attachment/106245/...
>
> Lesen und staunen ...

da hat doch das meiste etwas mit dem empfangen von sms zu tun. ich 
möchte nur senden und nicht empfangen.

[Der Bastler (Gast)]

Senden wird da dick und breit behandelt ...

lesen musst du schon selbst

[Dom (Gast)]

> Warning: C:\...\main\main.c(89): local variable 'c' was set, but not
> used

bedeutet das was da steht :)
Du hast an an besagter Stelle Speicher für eine Variable geholt, diese 
aber nie benutzt.
Weg mit der entsprechenden Zeile.

Für was hast du in der Fkt. "sende_SMS" die Abfrage mit dem c eingebaut?

> int c=0;
>  if(c==0)
>  {
>  ...
>  c=1;  // Überprüfung das SMS nur 1 mal gesendet wurde
>  }
>  return;

Die Abfrage tut an sich garnichts :D
Das return brauchst du auch nicht.

[Michelle K. (Firma: electronica@tdnet) (michellekonzack)]

Wolfi schrieb:
>   uart_puts("at+cpin=0000\r\n"); //AT-Befehl zur Pineingabe
>   delay_ms(2000);       //2 sec warten   //2 Sekunden Warteschleife
>   uart_puts("at+cmgf=1\r\n");   //SMS-Ein und Ausgabe erfolgt
>                                        //    im 7Bit Klartext
>   delay_ms(2000);
>   uart_puts("at+cmgs=+xxxxxxxxxxxxxxx\r\n"); //Zieladressnummer
> //nach Österreich A1
>   delay_ms(2000);
>   uart_puts("Hello World"); //SMS-Text
>   delay_ms(2000);
>   uart_putc(0x1A);  //Hex Code fuer Strg+Z wird gesendet
>   delay_ms(5000);

Bist Du sicher das diese operationen alle erfolgreich sind?

Anstatt eine millionen delay_ms() einzubauen, solltest Du
uart_get() verwenden und das OK auswerten.

Anm.:  Mein TC35 hatte teilweise länger als 2 Sekunden
       gebraucht, um die PIN zu akzepieren...

Grüße
Michelle

[Alex (Gast)]

Hallo Wolfi,

das mit den delays ist so ne Sache! Bspw. dauert die 
Netzwerkregistrierung nach der Pin-Eingabe je nach Standort und 
Sendestärke seine Zeit--> AT+CREG?
ich würde dir raten die GSM-Antwort bzw. die Quittierung unbedingt 
auszuwerten. Empfange die Daten schreib sie in ein Array und werte sie 
so

if( strcmp( buffer, "OK" ) == 0 )
do something;

bspw. aus.
Wenn die Antwort "OK" ist, gehst du zum nächsten Initialiserungsschritt 
und so weiter. Bei nem ERROR kannst du bspw. eine rote LED aufblinken 
lassen oder einen Reset ausführen.

Gruß