ATmega48 - IRS funktioniert nicht richtig sei()

[Johannes L. (johannes_l37)]

Guten Tag, ich habe zur Zeit zwei Programme, eins das Daten über UART 
sendet, und eines welches einfach einen Counter hat und hochzählt, heute 
habe ich versucht das Counter-Programm in das UART-Sende-Programm 
einzufügen (im Endeffekt etwas zählen und dann etwas senden. ) 
Allerdings hängt er sich immer einen Befehl nach meinem sei(); d.h. er 
führt noch einen Befehl aus und macht dann gar nichts mehr. Ich 
programmiere noch nicht lange nehmt rücksicht :)
PS: Immer wenn ich etwas an PORTC setze leuchtet eine entsprechende LED, 
(PC3 ^= Rot, PC4 ^= Grün )
Interessant ist auch zu bemerken, dass die LED die ich in der IRS 
Routine aufleuchten lasse (um zu sehen ob ich da ankomme,) nicht 
leuchtet!
Ich hab wahrscheinlich etwas innerhalb der Setups (initialisierung) 
falsch gemacht.

Kann man eigentlich auch ne Variabel über Uart ausgeben? Also eine mit 
der ich z.b. den gezählten wert ausgeben :)

Code:

#include <avr/io.h>
#include <avr/interrupt.h>
#include <string.h>
#include "globals.h" //momentan nur Deff. von Counter1
//Fuer Baudrate
#define FOSC 8000000UL // Clock Speed
#define BAUD 9600L
#define UBRR ((FOSC / (BAUD * 16L)) - 1)
//Fuer Baudrate - ende


void setup()
{
  cli(); // Interrupts deaktiviert - sicherheitshalber

    DDRC &= ~(0x01);
    PORTC &= ~(0x01);

//Hier will ich noch die Baudrate einstellen
    UBRR0H = (unsigned char)(UBRR>>8);
    UBRR0L = (unsigned char)UBRR;
    /*aktiviere Sender und empfänger (uART)*/
    UCSR0B = (1<<RXCIE0)|(1<<RXEN0)|(1<<TXEN0) | (1 << UDRIE0);
    // frame format: 8data, 1stop bit
    UCSR0C = (1 << UCSZ01) | (1 << UCSZ00);
//Achtung, nicht vergessen bei den Fuses
//CKDIV8 raus zu nehmen
////////////////////////////////////////////


    ADMUX = 0xC0;
    ADCSRA = 0x9A;

    //INT0 + INT1
    EICRA = 0x0F;// Worauf reagiert der jeweilige Interrupt?
    EIFR = 0x03; //???? pc?
    EIMSK = 0x02; // Interrupts aktivieren

    TCCR1A = 0x00; // Normal Mode
    TCCR0A = 0x00;  // Normal Mode
    TCCR0B = 0x03;//0x00 // Prescaller bei 0x00 hier Timer/counter gestoppt
    //TCCR1B = 0x02; //Damit Prescaller auf 8 und timer Counter wieder aktiv
    TCNT0 = 0x00;   // 0..255
    OCR0A = 0x00;   // unwichtig da Normal Mode
    OCR0B = 0x00;   // unwichtig da Normal Mode
    TIMSK0 = 0x01;  // 0x01 = 1<<TOIE0)
    DDRC = 0xFF ; // Für LEDs, zum überprüfen wo ich grade bin.
    TCCR1C = 0x00; // nur für non PWM-mode
    TCNT1 = 0;    //Zähl register


}//ende von Setup (Initalisierung)

int uart_putc(unsigned char str)
{
    while (!(UCSR0A & (1<<UDRE0)))  /* warten bis Senden moeglich */
    {
    }

    UDR0 = str;                      /* sende Zeichen */
    return 0;
}

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

int main(void)
{
  PORTC |= (1<<PC4) ;
setup() ; // Initialisierung
sei() ; //Hier ist das erwähnte sei() ;
  PORTC |= (1<<PC3) ;// wird noch ausgeführt ...
  PORTC = 0x00 ; // wird nichmehr ausgeführt





//Beginne zu senden, UART
uint8_t i=0;
char cstr[300]; // größe der Sende Variable
strcpy(cstr, "123");//Das was gesendet wird
while( cstr[i] != 0){ // Solange bis String Ende Zeichen
   uart_putc(cstr[i]); // senden
   i++;//zeiger hoch.
}
//Ende von senden Uart.
while (1) {
   //Unendlichkeitsschleife ... (warten auf Interrupts? Andere Befehle?)
}

return 0; // never reached

}//ende von Main

ISR (TIMER0_OVF_vect)
{PORTC = 0xFF ; // wird AUCH nicht ausgeführt
  // F_CPU in MHz  Aufruf ca. alle                          (EDIT)
  // 1             2 ms                 1/(1000000 / 8 / 256)  Sekunden
  // 8             0,256 ms             1/(8000000 / 8 / 256)  Sekunden
  // 16            0,128 ms             1/(16000000 / 8 / 256) Sekunden
  Counter1++;
}//ISR EndeISR (TIMER0_OVF_vect)

fg - Johannes

[H.Joachim S. (crazyhorse)]

UCSR0B = (1<<RXCIE0)....

Du gibts den Receiver-Interupt frei, hast aber keine ISR dafür.

[Johannes L. (johannes_l37)]

Jo, hab das Register entsprechend überarbeitet,

UCSR0B = (1<<TXEN0) | (1 << UDRIE0);

funktioniert allerdings immernoch nicht.
danke

[spess53 (Gast)]

Hi

>funktioniert allerdings immernoch nicht. danke

Immer noch eine Interruptfreigabe (UDRIE) ohne Interruptroutine.

MfG Spess

[Karl H. (kbuchegg)]

Johannes L. schrieb:

Und wenn du dann deine ISR richtig gestellt hast
(Man gibt niemals, niemals, einen Interrupt frei, für den man keine ISR 
hat. So etwas wird mit einem Prozessorreset bei jedem Interrupt 
bestraft)

> Kann man eigentlich auch ne Variabel über Uart ausgeben? Also eine mit
> der ich z.b. den gezählten wert ausgeben :)

Klar kann man.
http://www.mikrocontroller.net/articles/FAQ


(Aber stell erst mal deine uart_puts Funktion wieder richtig. Die hast 
du nämlich sauber mit deiner Änderung zerstört. Und kauf dir ein C-Buch. 
Da gibt es neben dem, was in der FAQ zum Thema Stringverarbeitung steht 
noch ein großes Kapitel für C-Einsteiger zum Thema "Strings")

[Johannes L. (johannes_l37)]

Vielen dank, das Programm funktioniert jetzt soweit. Ja ich denke, dass 
mir ein C-Buch wirklich weiter helfen kann, vielen dank für eure Hilfe.
fg - Johannes

[Uwe (de0508)]

Hallo

Johannes L. schrieb:
> void uart_puts (char *s)
> {
>     while (*s)
>   s= "t" ;
>     {   /* so lange *s != '\0' also ungleich dem 
"String-Endezeichen(Terminator)" */
>         uart_putc(*s);
>         s++;
>     }
> }

hier ist noch ein Fehler !

s="t";

das ergibt eine Endlosschleife !