Empfangsprobleme mit DMX ( attiny2313A )

[Steve (Gast)]

Hallo
ich habe eine ganz einfach platine mit nem attiny2313a und zwei leds 
aufgebaut um dmx zu empfangen. das passende dmx treiber IC ist auch 
vorhanden. Ich habe meinen standart uart empfangs interrupt verwendet 
der auf anderen controllern ( atmega8,atmega32 ) immer fehlerfrei lief. 
mein problem ist jetzt dass es so scheint als ob der controller an einer 
bestimmten stelle aufhört das zu tun was er soll.
zu anfang ist die led aus.
wenn der empfangene wert größer als 127 wird soll die led angemacht 
werden. das funktioniert zuverlässig.
aber wenn der wert wieder kleiner wird soll auch die led wieder 
ausgehen. das läuft nicht.

CPU takt 12Mhz externer Quarz
CKDIV8 Bit nicht gesetzt
UBRR = 2

Danke für eure antworten

[Dödel (Gast)]

In Zeile 42 muss du den Wert der Variable anpassen.

[Steve (Gast)]

daraus schließe ich dass du den code haben willst ?

[Dödel (Gast)]

Wie soll man den sonst helfen können?

[Steve (Gast)]

/*
 * strobecontrol_beta.c
 *
 * Created: 30.07.2013 14:18:08
 *  Author: Steve
 */

#define F_CPU 1200000
#include <avr/io.h>

#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdlib.h>
#include <string.h>
#include <util/delay.h>
#include <util/twi.h>
#include "E:\AVR\ATtiny2313\Includes\BitUtilities.h"


    uint16_t dmx_pos = 0x0FFF;
    uint16_t dmx_addr;
    volatile uint8_t dmx_byte[6];
      

int main(void)
{
  DDRD = 0x0C;    // D2=gruen, D3=rot 


  
  // DMX UART: 8-N-2, 250kBAUD
  UCSRA = 0;
  UCSRB = (1<<RXEN) | (1<<RXCIE);
  UCSRC = (1<<USBS) | (1<<UCSZ1) | (1<<UCSZ0);    
    UBRRL = 2;          // 8Mhz : 1       12Mhz : 2
    UBRRH = 0;
  
  
  sei();
  
  while(1)
  {
    if (dmx_byte[1]>127)
    {
      sbi(PORTD,2);
    }
    else
    {
      cbi(PORTD,2);
    }

  }
  return 0;
}


ISR(USART_RX_vect)
{
  uint8_t uart_st = UCSRA;    
  
  uint8_t uart_da = UDR;    
  if (uart_st & (1<<FE))    
  {
    dmx_pos = 0;      
  }
  else                        
  {
    if (dmx_pos == 0)    
    {
      if (uart_da != 0)  
      {
        dmx_pos = 0x0FFF;  
      }
    }
    else if (dmx_pos >= dmx_addr && dmx_pos <= 0x0200)  
    {
      uint16_t ofs = dmx_pos-dmx_addr;
      if (ofs > sizeof(dmx_byte))
      {
        dmx_pos = 0x0FFF;          
        } else {
        dmx_byte[ofs] = uart_da;          

      }
    }
  }
  if (dmx_pos <= 0x0200) dmx_pos++;        
  dmx_addr = 1;
}

[oszi40 (Gast)]

Steve schrieb:
> aber wenn der Wert wieder kleiner wird soll auch die led wieder
> ausgehen. Das läuft nicht.

Dann kommt Dein Programm wohl nie an dem Schritt vorbei wo das geprüft 
wird?

[Steve (Gast)]

oszi40 schrieb:
> Dann kommt Dein Programm wohl nie an dem Schritt vorbei wo das geprüft
> wird?

doch sollte es aber da es in der while(1) schleife steht

[R. M. (rmax)]

Auf den ersten, zweiten und dritten Blick konnte ich keinen Fehler 
finden. Allerdings finde ich die ISR etwas unübersichtlich, deshalb hier 
ein (ungetesteter) Vorschlag für eine weniger verschachtelte Variante:

#define DMX_START 1
#define DMX_SIZE 6
#define DMX_END (DMX_START + DMX_SIZE)
#define DMX_MAX 512
#define DMX_SKIP (DMX_MAX + 1)
#if DMX_END > DMX_MAX
#   error "DMX only has 512 channels."
#endif

uint16_t dmx_pos = DMX_SKIP;
volatile uint8_t dmx_byte[DMX_SIZE];

ISR(USART_RX_vect)
{
    uint8_t         uart_st = UCSRA;
    uint8_t         uart_da = UDR;

    if (uart_st & (1 << FE))
    {
        // BREAK, start a new frame.
        dmx_pos = 0;
    }
    else if (dmx_pos == 0 && uart_da != 0)
    {
        // Invalid start byte, skip this frame.
        dmx_pos = DMX_SKIP;
    }
    else if (dmx_pos >= DMX_START && dmx_pos < DMX_END)
    {
        // Store the values of the channels we are interested in.
        dmx_byte[dmx_pos - DMX_START] = uart_da;
    }
    if (dmx_pos <= DMX_MAX) dmx_pos++;
}