UART sendet+empfängt nicht / Atmega8, C++ Programm

[Matthias (Gast)]

Hallo,

Ich habe eine kleine Platine mit Atmega8, der Atmega wird mit 2 
Interrupts für den Empfang/Senden von Daten konfiguriert.

Mit dem Oszi kann ich sehen, dass Daten am RX-Pin eingehen. Ebenso 
überwache ich den TX-Pin.

Leider schaffe ich es nicht dass er etwas sendet oder Empfängt, beide 
Interrupts schneinen niemals auszulösen...

Da dieser Code mit einem Atmega1284 problemlos funktioniert und ich den 
entscheidenden Fehler/Unterschied in der Konfiguration im 
Atmega8-Datenblatt und auch beim Vergleich der Datenblätter nicht sehe 
bin ich überfragt...

Er hat auch ein LCD-Display auf welchem das "E" geschrieben wird.
Jedoch kommt er nicht zum F.

Vielleicht habt ihr eine Idee?

static char TxBuff[32];
static char RxBuff[32];

static volatile uint8_t RxRead = 0;  // Zeigt auf Zeichen, das gesendet werden soll
static volatile uint8_t RxWrite =0;  // Zeigt auf nächste Schreibposition im Puffer

static volatile uint8_t RxLen =0;  // Zeigt auf nächste Schreibposition im Puffer

static volatile uint8_t TxRead = 0;  // Zeigt auf Zeichen, das gesendet werden soll
static volatile uint8_t TxWrite =0;  // Zeigt auf nächste Schreibposition im Puffer

int main(void)
{
  uint8_t a=0x04;
  uint8_t b=0x05;
  char* addr;
  uint8_t len=0;
  
  _delay_ms(1000);
  
  InituC();
  LCD_init();

    while(1)
    {
    _delay_ms(10);
    UART_HMI_PutC('l');
        addr=UART_HMI_GetRec(len);
    LCD_Send('E');
        if (len>0)
        {
      LCD_Send_ADDR(a);
      LCD_Send('F');
          for(int i=0;i<len;i++)
          {
            UART_HMI_PutC(addr[i]);
        LCD_Send_ADDR(b);
            LCD_Send('Z');
          }
        }
    }
}

void InituC()
{
  DDRD = 0x02;   //TXD = O
  //PORTD |= 0x3E; // Pull-UP SW4--SW0
  DDRC = 0x3F;   //(D4 = O,D5 = O,D6 = O,D7 = O,GP5 = I,GP6 = I,RS = O,E = O)
  DDRB = 0x00;   //(BEL = I, GP3 = I ,GP4 = I,MOSI = I,MISO = I,SCK = I)
  sei();         //Global IRQ-Enable
  //Timer0_Init();
  _delay_ms(20);
}

void UART_HMI_Init()
{

  UCSRA&=0xFC;      //U2X=0, kein MCPM
  UCSRB=0x98;      //RX/TX-Enable, IRQ-Enable
  UBRRH=0x00;
  UCSRC=0x86;      //8Bit, Kein Parity, ein Stop
  UBRRL=0x33;      // Baud-Rate @8MHz = 1200Baud
}

void UART_HMI_PutC(char data)
{
  while(TxWrite != TxRead);        // Warte bis Platz im Puffer
  TxBuff[TxWrite++] = data;
  if(TxWrite >= 32) TxWrite = 0;
  UCSRB |= (1<<UDRIE);          // UDRE Interrupt ein
}

char* UART_HMI_GetRec(uint8_t& len)
{
  char* pointer;              //Rückgabe der Aktuellen Addresse
  UCSRB &= ~(1<<RXCIE);          //Disable RX-IRQ
  len = RxLen;              //return actual length of readable bytes
  RxLen=0;                //Length of readable Bytes = 0
  uint8_t RxRead_buff=RxRead;        //Get last Read-Addr, wich beginns with 1. Byte, that has not been Read last Time
  RxRead=RxWrite;              //Set actual rx-Addr to wr-Addr --> 0 Bytes in Buffer
  UCSRB |= (1<<RXCIE);          //Enable RX-IRQ
  return pointer = &RxBuff[RxRead_buff];  //Return Pointer to 1. Readable Byte
}

ISR(USART_RXC_vect)      //Wartet auf ein erstes Zeichen des PC's
{
  UART_HMI_PutC('L');
  UCSRB &= ~(1<<RXCIE);
  RxBuff[RxWrite++] = UDR;
  if(RxWrite >= 32 ) RxWrite = 0;
  RxLen=RxLen + 1;
  UCSRB |= (1<<RXCIE);
}

ISR(USART_UDRE_vect)
{
  if(TxRead == TxWrite) UCSRB &= ~(1<<UDRIE);      // UDRE Interrupt aus
  else {
    UDR = TxBuff[TxRead++];              // nächtes Zeichen aus Puffer senden
    if(TxRead >= 32) TxRead = 0;
  }
}

Wäre toll wenn jemand da eine Idee hätte ;)

[Oliver S. (oliverso)]

Schreib die ganzen magic numbers mal mit Bitnamen.
Ein wesentlicher Unterschied zwischen den Prozessoren ist das handling 
von UCRSB/C.

Oliver

[S. Landolt (Gast)]

Wo wird dieses UART_HMI_Init denn aufgerufen?

[OMG (Gast)]

Matthias schrieb:
> void UART_HMI_Init()
> {
>
>   UCSRA&=0xFC;      //U2X=0, kein MCPM
>   UCSRB=0x98;      //RX/TX-Enable, IRQ-Enable
>   UBRRH=0x00;
>   UCSRC=0x86;      //8Bit, Kein Parity, ein Stop
>   UBRRL=0x33;      // Baud-Rate @8MHz = 1200Baud
> }


Soll das ein Witz sein mit deinen Magic Numbers? Da darf sich
jeder der Lust hast sich seine Bits raussuchen um zu sehen ob
du es richtig machst? Weil du zu faul bist es dem Hilfeanbieter
leicht zu machen durchzublicken?

Dann noch dies hier:

---------------------------------------------
Wichtige Regeln - erst lesen, dann posten!

.................
    Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang
---------------------------------------------

[Stefan F. (Gast)]

Probiere erst mal ein einfacheres Programm ohne ISR aus, um 
Hardwarefehler und falsche Taktfrequenz/Bitrate auszuschließen.

[Matthias (Gast)]

S. Landolt schrieb:
> Wo wird dieses UART_HMI_Init denn aufgerufen?

Dass wird es wohl sein :)

Ja, das ist es :-)

Vielen Dank ;-)

OMG schrieb:
> Soll das ein Witz sein mit deinen Magic Numbers? Da darf sich
> jeder der Lust hast sich seine Bits raussuchen um zu sehen ob
> du es richtig machst? Weil du zu faul bist es dem Hilfeanbieter
> leicht zu machen durchzublicken?

Nun, ich habe den Aufwand betrieben alle Bits 2mal auf ihre Richtigkeit 
zu Pürfen. Wenn du, wie du es selbst schreibst, zu faul bist dir diesen 
Aufwand zu machen sehe ich es nicht als mein Problem an.

[Stefan F. (Gast)]

Matthias schrieb:
> Wenn du, wie du es selbst schreibst, zu faul bist dir diesen
> Aufwand zu machen sehe ich es nicht als mein Problem an.

Wen dir deswegen keiner hilft, wird es aber wiederum dein Problem.

Das ist hier ja schon eine Form von Teamarbeit. In Teams muss man sich 
auf einen gemeinsamen Stil einigen, um erfolgreich zusammen arbeiten zu 
können. Wenn hier jetzt einige Leute wegen deinem Stil blockieren, kann 
es Dir egal sein. Im Beruf mit Kollegen allerdings nicht.

[Matthias (Gast)]

Stefan ⛄ F. schrieb:
> Wen dir deswegen keiner hilft, wird es aber wiederum dein Problem.
>
> Das ist hier ja schon eine Form von Teamarbeit. In Teams muss man sich
> auf einen gemeinsamen Stil einigen, um erfolgreich zusammen arbeiten zu
> können. Wenn hier jetzt einige leute wegen deinem Stil blockieren, kann
> es Dir egal sein. Im Beruf mit Kollegen allerdings nicht.

Stimmt aber da würde keiner diesen Tonfall benutzen...
https://www.mikrocontroller.net/articles/Netiquette

Und ja, da würde ich es wohl auch besser Dokumetieren...

[Stefan F. (Gast)]

Matthias schrieb:
> aber da (im Beruf) würde keiner diesen Tonfall benutzen...

Jedenfalls wäre ich dann schnell dort weg. Meine Frau hat zwei Firmen 
erlebt, wo sich nur die Frauen permanent angezickt hatten und manchmal 
sogar ihre Arbeiten gegenseitig durcheinander brachten, um die andere zu 
behindern.

Wer weiß, vielleicht sind hier lauter Frauen unterwegs, die sich 
Männernamen geben. Oder echte Weicheier, die sich wie Mimosen benehmen. 
Ich glaube ja, dass diese allgemeine Aggression hier lediglich ein 
Ausdruck von Schwäche ist. Ich will mich selbst dabei nicht einmal 
ausschließen. Ich werde auch laut, wenn ich mich bedrängt fühle.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Matthias schrieb:

  while(TxWrite != TxRead);        // Warte bis Platz im Puffer

Dein Code kann wegen dieser Abfrage übrigens immer nur genau 1 Zeichen 
senden. Denn wenn ein Zeichen in den Puffer geschrieben wurde, dann wird 
mit dem Schreiben des nächsten Zeichens so lange gewartet, bis das 
vorige gesendet wurde. Der Puffer von 32 Bytes ist so samt Verwaltung 
unnütz.

len = RxLen;              //return actual length of readable bytes
RxLen=0;                //Length of readable Bytes = 0
uint8_t RxRead_buff=RxRead;        //Get last Read-Addr, wich beginns with 1. Byte, that has not been Read last

Es ist nicht verboten, Kommentare auf Deutsch hinzuschreiben.
Besonders wenn man der Fremdsprache nicht so richtig mächtig ist. Denn 
"actual" heißt nicht direkt "aktuell". "current" träfe es hier besser.

Und wenn die Kommentare dann noch eher für Verwirrung sorgen, als den 
Quelltext zu erläutern, dann ist die Latte völlig gerissen.

Ich musste da ehrlich gesagt den Quelltext lesen um die Kommentare zu 
verstehen... :-/

Matthias schrieb:
> sehe ich es nicht als mein Problem an.
Ja, kann man so sehen.