UART von Mega32 auf Mega168 anpassen

[Martin (Gast)]

Hallo,

ich bin gerade dabei einen C-Code für den Atmega32 auf einen Atmega168A 
anzupassen.

Ich habe die Register und Bitnamen dem Datenblatt angepasst, der 
Compiler gibt keine Warnung und keinen Fehler zurück. Ich verwende genau 
wie beim Mega32 einen externen 8MHz Quarz.

Nun funktioniert meine UART Übertragung nicht. Mit dem Mega32 
funktioniert es prima. Ich nutze das Pollin-Evaluationsboard Version 
2.0.1 und gehe davon aus, dass die Beschaltung stimmt.

Gibt es irgendwelche Stolperfallen bzw. Tricks die bei der Portierung 
der UART-Routine vom Mega32 zum Mega168A zu beachten sind?

Hatte jemand von euch bereits das Problem?

MFG Martin

[Stefan (Gast)]

Hast Du eventuell die beiden Ports vertauscht?

Ansonsten habe ich neben den anderen register-Namen noch diese 
Unterscheidung in meinen Programmen, damit sie auf beiden Chips laufen:

        // framing format 8N1
        #ifdef URSEL
            UCSRC = (1<<URSEL) | (1<<UCSZ1) | (1<<UCSZ0);
        #else
            UCSRC = (1<<UCSZ1) | (1<<UCSZ0);
        #endif

[Martin (Gast)]

@Stefan: Habe jetzt die Änderungen ebenfalls so angelegt, dass es mit 
beiden µCs laufen sollte.

Mit dem Atmega32 läufts super - ich empfange die richtigen Daten im 
Hterm. Aber mit dem 168 wills nich laufen :-(

Könntest du mir eventuell deinen Code bzw. einen UART-Auszug zur 
Verfügung stellen, damit ich meinen Fehler suchen kann ?

[Thomas E. (thomase)]

Stefan schrieb:
> #else
>
>             UCSRC = (1<<UCSZ1) | (1<<UCSZ0);
>
>         #endif

Der Atmega168 hat kein UCSRC-Register.

mfg.

[Martin (Gast)]

Hallo Thomas,

ja das hab ich auch in meinem Code berücksichtigt. Bei mir ist es:

 UCSR0C = (1<<UCSZ01) | (1<<UCSZ00);

Trotzdem will es einfach nicht laufen..

[Oliver J. (skriptkiddy)]

Zeig bitte den kompletten Code.

[g457 (Gast)]

> Gibt es irgendwelche Stolperfallen [..] die [..] zu beachten sind?

Größte Stolperfalle ist wie üblich, den Code nicht oder nicht 
vollständig zu zeigen. Passiert leider immer wieder. Die zweitgrößte 
Stolperfalle ist eine ungenaue Fehlerbeschreibung wie

> Nun funktioniert meine UART Übertragung nicht.

HTH und nix für ungut

[Thomas E. (thomase)]

Martin schrieb:
> ja das hab ich auch in meinem Code berücksichtigt.
Das würde auch gar nicht kompilieren und stammt ja auch nicht von dir. 
Aber es ist einfach nur schlampig und der Compiler verzeiht keine 
Schlamperei.

Martin schrieb:
> UCSR0C = (1<<UCSZ01) | (1<<UCSZ00);
Da liegt dein Fehler auch nicht. Das ist die mit Abstand unwichtigste 
Einstellung. Denn der Registerwert ist der gleiche wie nach Reset.

Für eine normale UART-Übertragung müssen beim 168er nur das 
UCSR0B-Register und die Baudrate eingestellt werden. Evtl. noch das U2X0 
in UCSR0A zur Baudratenverdoppelung.

mfg.

[Martin (Gast)]

Hallo,

danke an alle die sich bisher die Mühe gemacht haben, zu antworten.

Die UART-Übertragung ist mittlerweile kein Problem mehr. Es war vielmehr 
das Problem mit der Umleitung von printf.

Anbei habe ich den angepassten Quellcode (stammt ursprünglich aus dem 
Artikel Versenden von SMS mittels Mobiltelefon).

Ich habe in den Code noch den Beschleunigungssensor BMA020 eingebunden, 
was aber nichts mit dem Problem zutun haben sollte, dieser funktioniert 
einwandfrei.

Ich programmiere im AVRStudio 5 und habe "optimize -O1" für die delay.h 
eingestellt.

Der Code wird, je nach Device-Einstellung (ATmega32 oder ATmega168A) im 
AVRStudio, fehler- und warnungsfrei übersetzt.

Wenn ich mir das ganze im Hterm betrachte, dann sehe ich mit dem 
ATmega168A gar nichts. Mit dem Mega32 werden die Daten richtig 
übertragen (siehe Bild: Original_fdevopen_M32).

Wie der Bildname vermuten lässt, habe ich dann die Stdio-Umleitung mit 
fdevopen gegen die stdout-Variante aus dem AVR-GCC-Tutorial ersetzt.

 //in sms.c
    uart_init();
    fdevopen (uart_sendc, uart_receive);

 // in uart.c
    int uart_sendc(char c, FILE *dummy)
    {
    while (!(UCSRA & (1<<UDRE))) {}  // warten bis Senden möglich 
    UDR = c;                         // Zeichen senden   
    return (0);
    }
    ...
    int uart_receive(FILE *dummy)
    {
        ...
    }

Neue Variante:

 // in main.c
    int uart_sendc(char c, FILE *stream);   

    static FILE mystdout = FDEV_SETUP_STREAM( uart_sendc, NULL,     
    _FDEV_SETUP_WRITE );

    ...
    
    uart_init();
    stdout = &mystdout;

 //in uart.c
    int uart_sendc(char c, FILE *stream)
    {
    if( c == '\n' )
        uart_sendc( '\r', stream );
 
    loop_until_bit_is_set( UCSRA, UDRE );
    UDR = c;
    return 0;
    }
    ...
    int uart_receive(FILE *stream)
    {
        ...
    }

Das Ergebnis der neuen Variante ist, dass es tadellos mit dem Mega32 
funktioniert (siehe Bild SMSAlarm_stdout_M32), jedoch nichts mit dem 
Mega168 zu sehen ist.

Schiebe ich jetzt den folgenden Teil in die sms.c -> sms_init()

  //sms.c
     int uart_sendc(char c, FILE *stream);
     static FILE mystdout = FDEV_SETUP_STREAM( uart_sendc, NULL,    
     _FDEV_SETUP_WRITE );

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

     ...

dann erhalte ich mit dem Mega168 erste Lebenszeichen in Hterm (siehe 
Bild SMSAlarm_stdout_M168A). Die Daten werden aber immer noch nicht 
korrekt übertragen.

Jetzt meine Fragen :-)

Kann das sein, dass in dem Original-Code veraltete Stellen sind, die auf 
dem Mega168 Probleme bereiten?

Oder kommt mein Compiler nicht mit dem Code klar?

Wenn ich die Funktionen:

uint8_t sms_check_nr(char* telnr_in)
SMS_MSG sms_state_machine(void)
uint8_t sms_cmdset(uint8_t dat)

auskommentiere, dann sehen die Daten im Hterm schon ganz in Ordnung aus. 
Das Problem ist jetzt, dass die Nachrichten, die den Funktionen 
sms_send() und sms_display_text() übergeben werden, nicht berücksichtigt 
werden :-(

Ich bin bald am verzweifeln und hoffe jemand von euch kann mir da 
weiterhelfen.

In der SMSALARM_angepasst_M168.zip ist der aktuelle C-Code der von mir 
bis hier hin geschrieben bzw. abgeändert wurde.

Gruß Martin

[Martin (Gast)]

Hallo,

ich glaube ich konnte die Probleme nun noch weiter eingrenzen:

Kann es sein, dass ich mit strlen(nachricht) und printf Probleme beim 
compilieren kriege?

Ich vermute, dass durch das Strlen im folgenden Code-Stück Probleme 
auftauchen:

   ...

   void sms_display_text(char* nachricht)
   {  
      char tmp[20]="";
      uint8_t i;
      uint8_t cnt_txt=0;

      UCSRB &= ~(1 << RXCIE);    // UART Receive ISR ausstellen
      uart_clear();
    
      cnt_txt = strlen(nachricht);
      printf("AT^SSTK=%02i,0\r\n",cnt_txt+13+1);  // 13 Header, 1 't' 
                                                  // vor nachricht
      gets(tmp);
      printf("D0");  // Text String Tag
      printf("%02X",cnt_txt+13+1-2);  // Länge des PDU String - 1

   ...

Kann sich das bitte mal jemand anschauen?

Gruß Martin

[Karl H. (kbuchegg)]

Martin schrieb:

> Kann es sein, dass ich mit strlen(nachricht) und printf Probleme beim
> compilieren kriege?

Ich denke eher, dass du noch Fehler im Progamm hast, die dir vorher 
nicht aufgefallen sind, bzw. die sich nicht bemerkbar gemacht haben.
'Funktioniert auf einem Mega8' ist KEIN Beweis für ein fehlerfreies 
Programm.


Warum gehst du nicht schrittweise vor?

Teste doch erst mal nur die UART allein. Verkabel sie mit deinem PC und 
schreib dir ein Testprogramm welches die UART für sich alleine testet, 
indem sie endlos zb ein Zeichen an den PC schickt.


int main()
{
  uart_init();

  while( 1 )
    uart_putc( 'x', NULL );
}

und dann geht es weiter, indem du sukzessive Teile aus deinem 
vorhandenen Programm übernimmst. WObei es nicht schaden kann, wenn man 
über den jeweiligen Teil noch mal drüber liest und gezielt nach 
Problemstellen sucht.

Gegen 'ich bin bald am verzweifeln' hilft nur systematisches Vorgehen. 
Was nicht funktioniert: planlos herumprobieren weil man denkt: dort hat 
es ja auch funktioniert. Immer daran denken: Durch testen kann man nicht 
die Abwesenheit von Fehlern feststellen sondern immer nur die 
Anwesenheit. Nur weil ein Programm scheinbar funktioniert, heißt das 
nicht, dass es fehlerfrei wäre.

[Martin (Gast)]

Die UART und printf-Umleitung funktionieren jetzt.

Ich habe dasselbe Programm auf dem Mega168 und dem Mega32. Nur der 
Inhalt der strings, die von sms_display_text() und sms_send() 
(Mega168)gesendet werden stimmen nicht mit denen vom Mega32 überein.

[Martin (Gast)]

Ich krieg's einfach nicht hin :-(

Meine letzte Aussage muss ich leider zurückziehen, die Übertragung eines 
einzelnen Zeichens über UART funktioniert, nur die Umleitung will nicht 
so wie ich will.

Mit

fdevopen(uart_sendc,uart_receive)

funktioniert es auf dem Mega32 und das Handy empfängt die Daten, aber 
auf dem Mega168 gehts nicht. Sobald ich die Variante aus dem 
AVR-GCC-Tutorial nehme, tut sich etwas am Mega168, aber nicht so wie es 
sein soll.

// in main.c
    int uart_sendc(char c, FILE *stream);   

    static FILE mystdout = FDEV_SETUP_STREAM( uart_sendc, NULL,     
    _FDEV_SETUP_WRITE );

    ...
    
    uart_init();
    stdout = &mystdout;

 //in uart.c
    int uart_sendc(char c, FILE *stream)
    {
    if( c == '\n' )
        uart_sendc( '\r', stream );
 
    loop_until_bit_is_set( UCSRA, UDRE );
    UDR = c;
    return 0;
    }
    ...
    int uart_receive(FILE *stream)
    {
        ...
    }

Ich habe auch folgendes probiert:

FILE *fdevopen(int (*uart_sendc)(char c, FILE* stream), int (*uart_receive)(FILE* stream));

Der Compiler macht alles fehlerfrei, aber im Terminal ist nichts zu 
sehen.

Was habe ich nicht beachtet oder was mache ich grundsätzlich falsch?

MfG Martin