printf bei Handy-SMS Code aus Artikel geht nicht?

[Y. T. (moritzz)]

Hallo,
Ich bin gerade dabei, aus der Artikelsammlung das "mikrocontroller 
verschickt SMS an Siemens-Handy"-Projekt nachzubauen. Hab auch alles 
soweit, Handy geht, etc. jetzt schaue ich, dass ich den Code auf einen 
ATmega8 abgespeckt bekomme.

Leider geht die printf-Funktion nicht, der µC sendet dabei nix über das 
UART raus...Woran kann das liegen? Ich habe im Makefile schon 
printf-options auf "minimalistic" eingestellt. Warum geht diese 
printf-Variante nicht?

In meiner Hauptroutine steht jetzt zum Test:

uart_sends("hallo"); //das funktioniert
printf("at+cclk?\r\n"); //da kommt nix an
_delay_ms(2500);
sms_send("015781581259","Die Krümmung ist eine Größe der inneren Geometrie");
//da kommt auch nix an

Die wichtigsten Code-Schnipsel aus dem Artikel für die SMS-Funktionen 
lauten so:

void sms_init(void){
  // Initialisierung der seriellen Schnittstelle Asynchron
  uart_init();
  // öffnet Kanal für printf
  //stdout = &mystdout;
  fdevopen (uart_sendc, uart_receive);

  UCSRB &= ~(1 << RXCIE);    // UART Receive ISR ausstellen

  printf("ATE0\r\n");    // Echo off, reduce traffic
  _delay_ms(200);
  printf("AT+CPMS=\"ME\",\"MT\",\"MT\"\r\n");    // Speicher von gelesenen, geschriebenen und empfangenen SMS auf Telefonspeicher stellen
  _delay_ms(200);
  printf("AT+CNMI=1,1,0,0,1\r\n");        // new message indication --> "+CMTI:<mem>,<index>"
  _delay_ms(200);

  // Auslesen der ersten 5 Telefonnummern der SIM-Karte
  char telnr[SMS_TELNR_MAX_LEN+1];
  uint8_t i=0;
  int dummy;
  printf("AT+CPBS=\"SM\"\r\n");          // Telefonbuch auf SIM Karte umstellen
  _delay_ms(200);
  uart_clear();
  for (i=0;i<SMS_TELNR_LIST_MAX;i++) {
    uart_clear();
    printf("AT+CPBR=%i\r\n",i+1);
    gets(telnr);
    scanf("+CPBR: %i,\"%[^\"]",&dummy,telnr);
    _delay_ms(400);                // Rest der Antwort abwarten und verwerfen
    if (strlen(telnr)>3) {
      strcpy(sms_telnr_list[i].telnr,telnr);  // ausgelesene Telefonnummer in Struktur speichern  
    }
    else { strcpy(sms_telnr_list[i].telnr, "noNR");}
  }
  uart_clear();
  if (UART_ISR!=0) {UCSRB |= (1 << RXCIE);}    // UART Receive ISR wieder einstellen
}

und

void sms_send(char* zielnr, char* nachricht)
{
  uint8_t cnt_shiftzero=0;
  uint8_t cnt_nr=0;          // Zähler für Zielnummer, nach Umwandlung = Länge der Zielnummer
  uint8_t cnt_txt=0;          // Zähler für Nachricht, nach Umwandlung = Länge der Nachricht
  uint8_t shift=0;
  uint8_t i=0;
  char nachricht_neu[SMS_MESSAGE_MAX_LEN+1]={"\0"};
  char zielnr_neu[SMS_TELNR_MAX_LEN+1]={" "};

  UCSRB &= ~(1 << RXCIE);    // UART Receive ISR ausstellen

  // Nachricht umwandeln in PDU-Format
  while (nachricht[cnt_txt]!=0)
  {
    if (shift==7) {
      shift=0;
      cnt_txt++;
      cnt_shiftzero++;
    }
    else {                
      nachricht_neu[cnt_txt-cnt_shiftzero] = (((nachricht[cnt_txt]&0x7F)>>shift) | ((nachricht[cnt_txt+1])<<(7-shift)));
      shift++;
      cnt_txt++;
    }
  }
  nachricht_neu[cnt_txt-cnt_shiftzero]=nachricht[cnt_txt];  // '0' anfügen , Endestring
  
  // Zielnummer neu zusammensetzen
  // Bsp. 01 62 64 94 30 0 --> 10 26 46 49 03 F0
  // wenn 2 Bytes nicht voll ausgenutzt werden, 'F' einfügen
  while (zielnr[cnt_nr]!=0) {
    if (zielnr[cnt_nr+1]==0) {
      zielnr_neu[cnt_nr]='F';  
      zielnr_neu[cnt_nr+1]=zielnr[cnt_nr];
      cnt_nr++;
      cnt_nr++;
      zielnr_neu[cnt_nr]=zielnr[cnt_nr-1];  // '0' anfügen , Endestring
      break;
    }
    else {
      zielnr_neu[cnt_nr]=zielnr[cnt_nr+1];
      zielnr_neu[cnt_nr+1]=zielnr[cnt_nr];
      cnt_nr=cnt_nr+2;
    }
  }

  //Anzahl der Zeichen im PDU-String/2-->Hex Paare -2 (ersten 2 Hex Zeichen weglassen)
  //-->'Anzahl der Nachrichtenzeichen' + 'Anzahl der Nummern /2' (/2 da nur Hex Paare gezählt werden) + '7' (zusätzliche Header)
  if (zielnr_neu[cnt_nr-2]=='F') {
    printf("AT+CMGS=%i\r\n",cnt_txt-cnt_shiftzero+(cnt_nr/2)+7);
    cnt_nr--;    //Länge der Zielnummer -1, da 'F' angefügt
  }
  else {
    printf("AT+CMGS=%i\r\n",cnt_txt-cnt_shiftzero+(cnt_nr/2)+7);
  }

  //while(uart_receive()!=' ');    // auf antwort von Handy warten '> '
  _delay_ms(1000);
  printf("000100%02X81",cnt_nr);
  printf(zielnr_neu);
  printf("0000%02X",cnt_txt);
  for (i = 0;i<(cnt_txt-cnt_shiftzero);i++){
    printf("%02X", nachricht_neu[i]);
  };
  printf("%c",26);
  if (UART_ISR!=0) {UCSRB |= (1 << RXCIE);}    // UART Receive ISR wieder einstellen)
}

Danke!

[SMS (Gast)]

M. G. schrieb:
> In meiner Hauptroutine steht jetzt zum Test:
> uart_sends("hallo"); //das funktioniert
> printf("at+cclk?\r\n"); //da kommt nix an
> _delay_ms(2500);
> sms_send("015781581259","Die Krümmung ist eine Größe der inneren Geometrie");
> //da kommt auch nix an

Kann ´ja auch nicht ohne vorher sms_init aufzurufen.

[SMS (Gast)]

M. G. schrieb:
> Hab auch alles
> soweit, Handy geht, etc. jetzt schaue ich, dass ich den Code auf einen
> ATmega8 abgespeckt bekomme.

Das geht eher nicht, selbst wenn du alle lcd routinen etc. rausnimmst 
wird das kaum passen. Steht auch so im Artikel.

[Y. T. (moritzz)]

Na, sms_init ist natürlich aufgerufen, und Platz ist auch genug auf dem 
µC (siehe unten), ich habe halt in der sms.c alle Funktionen 
rausgenommen ausser sms_send und sms_init. Ich will auch nur diese zwei 
Sachen erstmal machen!
Also die ganze Hauptroutine in main.c sieht halt so aus:

#include "main.h"
#include "uart.c"
#include "sms.c"

volatile uint8_t timer_flag;

int main(void) {

DDRD = (1<<PD1) | ( 1 << PD2 );
DDRB=0xFF;
PORTB=0;

  sei();
  sms_init();
  uart_clear();

uart_sends("hallo"); //geht
printf("at+cclk?\r\n"); //geht nicht


  
  
  
while(1) //Ende
{
PORTB = ( 1 << PB0 );
_delay_ms(250);
PORTB = ( 0 << PB0 );
_delay_ms(250);
}
}

Ach ja, und der AVR ist halt bis zum Rand gefüllt:

Program:    5112 bytes (62.4% Full)
(.text + .data + .bootloader)

Data:       1009 bytes (98.5% Full)
(.data + .bss + .noinit)

[SMS (Gast)]

M. G. schrieb:
> #include "uart.c"
> #include "sms.c"

Das sollte eher *.h heißen.
Am original Code brauchst du sicher keine Fehler suchen, der 
funktioniert so wie er ist. Wurde schon dutzende Male erfolgreich 
verwendet. Wahrscheinlicher ist, dass du etwas auskommentiert hast, was 
du nicht hättest auskommentieren sollen (wie z.B. die header -Dateien)

[Stefan E. (sternst)]

M. G. schrieb:
> Data:       1009 bytes (98.5% Full)
> (.data + .bss + .noinit)

Also praktisch Null Platz für den Stack. Das funktioniert so sicher 
nicht. Als erste Maßnahme solltest du mal die ganzen 
printf-Formatstrings ins Flash verbannen und entsprechend printf_P 
benutzen.

[Y. T. (moritzz)]

Ah,
das klingt nach der Lösung...
Also du meinst, man definiert für die AT-Commandostrings 
Flash-Variablen?
etwa so:

char StringImFlash[] PROGMEM = "at+cclk?\r\n"; // im "Flash"

printf_P(StringImFlash);

oder etwa sowas wie

const char Ichbinimflash[] PROGMEM = "%s";

printf_P(Ichbinimflash); 
??????

(Kennt jemand eigentlich mal so eine Art Dokumentation zu diesen 
merkwürdigen printf-Funktionen? Ich weiß eigentlich gar nicht genau, was 
die machen...?!)

[Stefan E. (sternst)]

M. G. schrieb:
> Also du meinst, man definiert für die AT-Commandostrings
> Flash-Variablen?
> etwa so:

char StringImFlash[] PROGMEM = "at+cclk?\r\n"; // im "Flash"

printf_P(StringImFlash);

>
> oder etwa sowas wie

const char Ichbinimflash[] PROGMEM = "%s";

printf_P(Ichbinimflash);
??????

Oder einfach so:

printf_P(PSTR("at+cclk?\r\n"));

> (Kennt jemand eigentlich mal so eine Art Dokumentation zu diesen
> merkwürdigen printf-Funktionen? Ich weiß eigentlich gar nicht genau, was
> die machen...?!)
http://www.nongnu.org/avr-libc/user-manual/

[Y. T. (moritzz)]

Vielen Dank!
Das werd ich jetzt  erstmal probieren!!!
Dabei werd ich mir auch die Doku vorknüpfen!

[Läubi .. (laeubi)]

Was ich mich frage ist: Wozu benötigt man hier überhaupt printf? Nur für 
die Formatierte Ausgabe von ein paar Zahlen? Da könnte man durch eine 
eigene Routine sicherlich auch noch etwas Platz sparen.

[SMS (Gast)]

Läubi .. schrieb:
> Was ich mich frage ist: Wozu benötigt man hier überhaupt printf? Nur für
> die Formatierte Ausgabe von ein paar Zahlen? Da könnte man durch eine
> eigene Routine sicherlich auch noch etwas Platz sparen.

Zitat aus Artikel:
"Die einzelnen Algorithmen sind dabei sehr einfach gehalten. Das 
bedeutet, dass es sicherlich sehr viel kürzere Varianten gibt, um z. B. 
einen Text in das PDU Format zu transformieren. Ich habe mich aber 
lieber auf mehr und damit für andere auch verständlicheren Code 
beschränkt. Das verbraucht zwar mehr Speicher, trägt aber meines 
Erachtens zum Verständnis bei."

Platz sparen war hier nicht der Ansporn, sondern vielmehr das 
Verständniß und printf kennt nun mal jeder. Der Anfänger muss siche eben 
nicht mit dem UART rumschlagen.

[Läubi .. (laeubi)]

SMS schrieb:
> Platz sparen war hier nicht der Ansporn, sondern vielmehr
> das Verständniß
Schon klar nur der TE scheint ja
 a) Platz sparen zu wollen
 b) mit printf auf dem Kriegsfuß zu stehen ;)

[SMS (Gast)]

Läubi .. schrieb:
> Schon klar nur der TE scheint ja
>  a) Platz sparen zu wollen
>  b) mit printf auf dem Kriegsfuß zu stehen ;)

Selten aber wahr^^