Xmega: USART legt Timer lahm? Kennt jemand das Problem?

[Stefan E. (stefanosch)]

Hallo,

nach mehreren Stunden der Portierung von einem ATmega168 auf einen 
ATxmega128A4U, des Debuggen, Lesens des Manuals und Ausprobierens 
stehe ich immer noch auf dem Schlauch.

Der aufs wesentliche gekürzte Code am Ende hat in der main drei 
verschiedene Implementierungen, die sich durch das entsprechende 
define umschalten lassen.

Kurz zur Funktionsweise :

Zuerst wird das System initialisiert und der Timer gestartet. Der Timer 
lässt ganz profan mehrere LEDs nacheinander kurz aufleuchten. Das ganze 
erfolgt solange bis er wieder gestoppt wird. Danach wird er erneut 
gestartet.

Das Starten/Stoppen erfolgt in den ersten beiden Varianten regelmässig 
durch umschalten/warten.

In der dritten Variante soll es durch Befehle von aussen erfolgen.

Hier das Problem :

Variante 1 und 2 funktionieren einwandfrei, die 3. nur solange 
kein Zeichen von der seriellen Schnittstelle empfangen wurde. Sobald 
irgend ein Zeichen empfangen wurde wird der Timer abgschaltet und lässt 
sich auch nicht mehr aktivieren. Der Sprung in die beiden Funktionen 
enable_timer() und disable_timer() bei Empfang der Zeichen d oder e 
erfolgt auch korrekt, nur das sich dann aber nichts tut.

Hier meine Fragen :

1. Hat schon mal jemand ein ähnliches Verhalten bei den Xmegas 
festgestellt?

2. Wo gibt es die Errata-Files für die Xmegas, bei den Megas waren sie 
einfach zu finden?

3. Gibt es eine Abhängigkeit zwischen Timer und Usart, die man erst 
wegkonfigurieren muss?

4. Wäre jemand so nett den Code bei sich zu testen, um zu sehen ob er da 
genauso versagt?

5. Ist es schon dunkel draußen? ;-)

Hier der Code :

#define F_CPU 2000000

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>


#define LED_PORT  PORTD
#define LED0_PIN_bm  PIN0_bm
#define LED1_PIN_bm  PIN1_bm
#define LED2_PIN_bm  PIN2_bm
#define LED3_PIN_bm  PIN3_bm
#define LED4_PIN_bm  PIN4_bm

#define LED_bm    LED0_PIN_bm | LED1_PIN_bm | LED2_PIN_bm | LED3_PIN_bm | LED4_PIN_bm

#define BAUDRATE 38400
#define RS232 USARTD1
#define RS232_PORT PORTD
#define RS232_RX_PIN_bm PIN6_bm
#define RS232_TX_PIN_bm PIN7_bm

void init_clock()
{
   OSC.CTRL = OSC_RC2MEN_bm;
   OSC.PLLCTRL = OSC_PLLSRC_RC2M_gc;
   while(!(OSC.STATUS & OSC_RC2MRDY_bm));
   CCP = CCP_IOREG_gc;
   CLK.CTRL = CLK_SCLKSEL_RC2M_gc;
}

void init_leds()
{
   PORTCFG.MPCMASK = LED_bm;
   LED_PORT.PIN0CTRL = PORT_OPC_PULLDOWN_gc;
   LED_PORT.DIR  |=  LED_bm;
   LED_PORT.OUT  &= ~LED_bm;
}

void init_usart(void)
{
   RS232.BAUDCTRLB = (unsigned char)((F_CPU/16/BAUDRATE-1)>>8);
   RS232.BAUDCTRLA = (unsigned char)((F_CPU/16/BAUDRATE-1));
   RS232.CTRLA = USART_RXCINTLVL_HI_gc;
   RS232.CTRLB = USART_TXEN_bm | USART_RXEN_bm;
   RS232.CTRLC = USART_CMODE_ASYNCHRONOUS_gc | USART_PMODE_DISABLED_gc | USART_CHSIZE_8BIT_gc;
   PORTD.DIR &= ~PIN6_bm;
   PORTD.DIR |=  PIN7_bm;
}

int rxChar()
{
   while(!(RS232.STATUS & USART_RXCIF_bm));
   return (int)RS232.DATA;
}

void init_timer()
{
   TCC0.CTRLA = (TCC0.CTRLA & (~TC0_CLKSEL_gm)) | TC_CLKSEL_DIV8_gc;
   TCC0.CTRLB |= TC_WGMODE_NORMAL_gc;
   TCC0.PER = 0xffff;
   TCC0.CCA = 0x2000;
   TCC0.CCB = 0x4000;
   TCC0.CCC = 0x8000;
   TCC0.CCD = 0xC000;
}

void enable_timer()
{
   TCC0.CTRLA = (TCC0.CTRLA & (~TC0_CLKSEL_gm)) | TC_CLKSEL_DIV8_gc;
   TCC0.INTCTRLA = TC_OVFINTLVL_HI_gc;
   TCC0.INTCTRLB = (TC_CCAINTLVL_MED_gc | TC_CCBINTLVL_MED_gc | TC_CCCINTLVL_MED_gc | TC_CCDINTLVL_MED_gc);
   TCC0.CTRLFSET = TC_CMD_RESET_gc;
   TCC0.INTFLAGS = TCC0.INTFLAGS;
}

void disable_timer()
{
   TCC0.CTRLA = (TCC0.CTRLA & (~TC0_CLKSEL_gm)) | TC_CLKSEL_OFF_gc;
   TCC0.INTCTRLA = 0;
   TCC0.INTCTRLB = 0;
}

void reset()
{
   LED_PORT.OUT &= ~(LED_bm);
   PMIC.CTRL = PMIC_HILVLEN_bm | PMIC_LOLVLEN_bm | PMIC_MEDLVLEN_bm;
   sei();
}

ISR(TCC0_CCA_vect)
{
   LED_PORT.OUT &= ~(LED_bm);
   LED_PORT.OUT |= LED1_PIN_bm;
}

ISR(TCC0_CCB_vect)
{
   LED_PORT.OUT &= ~(LED_bm);
   LED_PORT.OUT |= LED2_PIN_bm;
}

ISR(TCC0_CCC_vect)
{
   LED_PORT.OUT &= ~(LED_bm);
   LED_PORT.OUT |= LED3_PIN_bm;
}

ISR(TCC0_CCD_vect)
{
   LED_PORT.OUT &= ~(LED_bm);
   LED_PORT.OUT |= LED4_PIN_bm;
}

ISR(TCC0_OVF_vect)
{
   LED_PORT.OUT &= ~(LED_bm);
}

int main(void)
{
   init_clock();
   init_leds();
   init_usart();
   init_timer();
   enable_timer(0);
   reset(0);

   while(1)
   {

#define TEST_CODE 3
#if TEST_CODE == 1

      /*
         Timer aktivieren und deaktivieren funktioniert
         bei Verwendung nur dieses Codes einwandfrei
      */
      enable_timer(0);
      _delay_ms(500);
      LED_PORT.OUT &= ~(LED_bm);
      disable_timer();
      _delay_ms(2000);

#elif TEST_CODE == 2

      /*
         Timer aktivieren und deaktivieren funktioniert
         bei Verwendung nur dieses Codes einwandfrei
      */
      static int ch = 'e';
      switch(ch)
      {
         case 'e':
            enable_timer();
            _delay_ms(500);
            break;

         case 'd':
            disable_timer();
            LED_PORT.OUT &= ~(LED_bm);
            _delay_ms(2000);
            break;

         default:
            ch = 0;
            break;
      }
      ch = (ch == 'e') ? 'd' : 'e';

#elif TEST_CODE == 3

      int ch = rxChar();
      switch(ch)
      {
         case 'e':
            enable_timer();
            break;

         case 'd':
            disable_timer();
            LED_PORT.OUT &= ~(LED_bm);
            break;

         default:
            ch = 0;
            break;
      }

#endif
   }
   return 0;
}

[Konrad S. (maybee)]

Stefan E. schrieb:
> 2. Wo gibt es die Errata-Files für die Xmegas, bei den Megas waren sie
> einfach zu finden?

Wie bei allen ATxmega-Serien im Baustein-spezifischen Datenblatt, nicht 
in der Beschreibung für die ganze Serie. Also z.B. hier:
http://www.atmel.com/Images/Atmel-8387-8-and16-bit-AVR-Microcontroller-XMEGA-A4U_Datasheet.pdf

> 5. Ist es schon dunkel draußen? ;-)

Nein. ;-)

[Gerhard G. (g_g)]

Hallo,

ich habe deinen Code getestet. Fehler liegt hier:

int ch = rxChar();

Es wird kein 'e' oder 'd' empfangen, sondern hier passiert irgend etwas 
fehlerhaftes!! Das Programm hängt sich hier auf.


Mit meiner USART-Routine ist dein Code ok!

USART-Lib hier rs232.h/rs232.c
https://github.com/BodyTrack/BodyTrack_CustomLoggers

Gruß G.G.

[Stefan E. (stefanosch)]

ich habe den Code jetzt mal zu Hause auf einem XPlain-Board getestet. Da 
ging erst auch nichts.

Nachdem ich die Antwort von G.G. gelesen hatte bin ich noch mal die 
Usart-Teile durchgegangen und siehe da es gab noch eine 
Interrupt-Zuweisung

RS232.CTRLA = USART_RXCINTLVL_HI_gc;

ohne zugehörige ISR. Nachdem ich diese entfernt hatte, funktioniert der 
Code bei mir auch wie gewollt. Jetzt muss es nur noch auf dem 
ATxmega128A4U laufen.

Irgendwie ist man nach 8 Stunden betriebsblind.

Danke für Eure Hilfe

Stefan

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Stefan E. schrieb:
> void enable_timer()
> {
Aber dann in mehreren Aufrufen so was:

> enable_timer(0);
Da die Funktion keine Parameter entgegen nimmt, solltest du sie auch 
nicht mit welchen aufrufen. Eigentlich sollten da auch Fehler beim 
Kompilieren erscheinen, a la : "Too many parameters when calling 
enable_timer"
Wenn keine Fehler erscheinen, schau dir mal deine Prototypen an...