Arduino / Atmel UART Fehler

[Max M. (maxmicr)]

Hallo,

ich hab nach diesem Tutorial versucht, über UART etwas zu senden:

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Der_UART

jetzt sagt mir Atmel-Studio aber:

Error  1  'UCSRA' was not declared in this scope
Error  2  'UDRE' was not declared in this scope
Error  3  'UDR' was not declared in this scope

#define F_CPU 16000000L //16 MHz

#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)

#include <avr/io.h>
#include <stdlib.h>
#include <util/setbaud.h>

int uart_putc(unsigned char c)
{
  while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
  {
  }
  UDR = c;                      /* sende Zeichen */
  return 0;
}

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

void uart_init(void)
{
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<TXEN0);
  // Frame Format: Asynchron 8N1
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

int main(void)
{
  char s[7];
  int16_t i = -12345;
  
  uart_init();
  
  itoa( i, s, 10 );
  uart_puts( s );
  
  uart_puts( itoa( i, s, 10 ) );
  
  while (1) {
  }
  return 0;
}

[(prx) A. K. (prx)]

Bei einigen AVRs gibts mehrere UARTs, weshalb die Register mittlerweile 
anders heissen. Mit Nummer drin. Bei uart_init passt es, bei uart_putc 
nicht.

[Max M. (maxmicr)]

Ich verwende einen Arduino mit ATMega328P. Ist das bei dem auch so?

[spess53 (Gast)]

Hi

>Ich verwende einen Arduino mit ATMega328P. Ist das bei dem auch so?

Was sagt das Datenblatt?

MfG Spess

[Max M. (maxmicr)]

spess53 schrieb:
> Was sagt das Datenblatt?

Danke, ich habs!

#define F_CPU 16000000L //16 MHz

#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)

#include <avr/io.h>
#include <stdlib.h>
#include <util/setbaud.h>

int uart_putc(unsigned char c)
{
  while (!(UCSR0A & (1<<UDRE0)))  /* warten bis Senden moeglich */
  {
  }
  UDR0 = c;                      /* sende Zeichen */
  return 0;
}

void uart_puts (char *s)
{
  while (*s)
  {
    uart_putc(*s);
    s++;
  }
}

void uart_init(void)
{
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

int main(void)
{
  char s[7];
  int16_t i = 1337;
  
  uart_init();
  
  itoa( i, s, 10 );
  uart_puts( s );
  
  uart_puts( itoa( i, s, 10 ) );
  
  while (1) {
  }
  return 0;
}

[Joachim B. (jar)]

Max M. schrieb:
> Ich verwende einen Arduino mit ATMega328P

dann schau halt nach:
http://www.atmel.com/images/doc8161.pdf

und mache dir deine eigenen #defines

#ifndef _PINS_H_
#define _PINS_H_


#if defined(__AVR_ATmega328P__)
  #define   TIMSKx              TIMSK1
  #define   OCIExA              OCIE1A
  #define   TIMERx_COMPA_vect   TIMER1_COMPA_vect  // ATmega
  #define   TCCRxA              TCCR1A
  #define   COMxA0              COM1A0
  #define   OCRxA               OCR1A
  #define   TCCRxB              TCCR1B
  #define   WGMx2               WGM12
  #define   CSx0                CS10


// D13(SCK)       PB5 (LED)
  #define LED_ARDUINO      5                          // D13 LED Arduino
  #define LED_ARDUINO_DDR  DDRB
  #define LED_ARDUINO_PORT PORTB
  #define LED_ARDUINO_ON   LED_ARDUINO_PORT|=(1<<LED_ARDUINO)
  #define LED_ARDUINO_OFF  LED_ARDUINO_PORT&=~(1<<LED_ARDUINO)

#elif defined(__AVR_ATmega1284P__) // -------------------------------------- m1284p -------------------------------------------------------
  #define   TIMSKx              TIMSK3
  #define   OCIExA              OCIE3A
  #define   TIMERx_COMPA_vect   TIMER3_COMPA_vect  // ATmega
  #define   TCCRxA              TCCR3A
  #define   COMxA0              COM3A0
  #define   OCRxA               OCR3A
  #define   TCCRxB              TCCR3B
  #define   WGMx2               WGM32
  #define   CSx0                CS30

//#elif defined(__AVR_ATmega1284P__)
  #define LED_ARDUINO       7                          // D7 LED Arduino
  #define LED_ARDUINO_PORT  PORTB
  #define LED_ARDUINO_DDR   DDRB
  #define LED_ARDUINO_ON       LED_ARDUINO_PORT  |= (1<<LED_ARDUINO)
  #define LED_ARDUINO_OFF      LED_ARDUINO_PORT &= ~(1<<LED_ARDUINO)

#else
 #error "board_wahl_falsch.h"
#endif // defined(__AVR_ATmega1284P__)

#endif // #ifndef _PINS_H_

[Thomas W. (Gast)]

spess53 schrieb:
> Was sagt das Datenblatt?

Was hat das Datenblatt damit zu tun? Die Bezeichnungen für die Register 
werden einzig und alleine in den devicespezifischen Headerfiles des 
Compilers festgelegt. Wenn die - mehr oder weniger zufällig - mit den 
Bezeichnungen im Datenblatt übereinstimmen, ist das glückliche Fügung 
und dient einzig der Arbeitserleichterung ;-)

[STK500-Besitzer (Gast)]

Thomas W. schrieb:
> Was hat das Datenblatt damit zu tun? Die Bezeichnungen für die Register
> werden einzig und alleine in den devicespezifischen Headerfiles des
> Compilers festgelegt. Wenn die - mehr oder weniger zufällig - mit den
> Bezeichnungen im Datenblatt übereinstimmen, ist das glückliche Fügung
> und dient einzig der Arbeitserleichterung ;-)

Und der Ersteller der Library wäre ein Idiot.

[Max M. (maxmicr)]

Warum funktioniert das hier nicht?

void uart_puts (char *s){
  while(!(UCSR0A & (1<<UDRE0))){
    
  }
  while(*s){
    UDR0 = *s++;
  }
}

wenn

char *s

 gleich:

char s[13] = {"HelloWorld"};

Es wird nur "He" gesendet.

[(prx) A. K. (prx)]

Weil in den Sendepuffer nur 2 Bytes reinpassen. Der Rest geht in den 
Ofen. Würde empfehlen, vor jedem einzelnen Byte zu fragen, ob noch Platz 
ist.

[spess53 (Gast)]

Hi

>  while(*s){
>    UDR0 = *s++;

Was wird das wohl machen?

MfG Spess

[Max M. (maxmicr)]

Danke für eure Antworten.

A. K. schrieb:
> Würde empfehlen, vor jedem einzelnen Byte zu fragen, ob noch Platz
> ist.

Danke!

void uart_puts (char *s){
  while(*s){
    if(UCSR0A & (1<<UDRE0))
      UDR0 = *s++;
  }
}

[(prx) A. K. (prx)]

Etwas unorthodoxe Methode, aber korrekt.

Faule Leute hätten in der Schleife einfach nur uart_putc aufgerufen.

[Max M. (maxmicr)]

A. K. schrieb:
> Faule Leute hätten in der Schleife einfach nur uart_putc aufgerufen.

Ich denke, wenn man zwei Funktionen in einer vereinen kann, sollte man 
es tun :)

[(prx) A. K. (prx)]

Max M. schrieb:
> Ich denke, wenn man zwei Funktionen in einer vereinen kann, sollte man
> es tun :)

Erstens: Nein, auf vorhandenem aufbauen ist im Prinzip besser.
Zweitens hätte der Compiler das auch selbst getan (inlining).
Drittens: Es bringt nix, Wartezeiten auf Zeit zu optimieren. ;-)

[Thomas E. (thomase)]

A. K. schrieb:
> Es bringt nix, Wartezeiten auf Zeit zu optimieren.

Gewagte These. Das solltest du erstmal mit Kurt diskutieren.

mfg.

[Max M. (maxmicr)]

Ein weiteres Problem. Ich versuche über UART auszulesen und dann das 
empfangene Ergebnis wieder zu schreiben aber es passiert nichts:

#define F_CPU 16000000L //16MHz
#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8) / (BAUD*16) - 1)
#define sbi(var,mask) ((var) |=(uint8_t)(1<<mask))
#define cbi(var,mask) ((var)&=(uint8_t)~(1<<mask))

#include <avr/io.h>
#include <util/setbaud.h>
#include <util/delay.h>
#include <stdlib.h>
#include <string.h>

uint8_t uart_getc(void){
  while(!(UCSR0A & (1<<RXC0))){
  }
  return UDR0;
}

void uart_puts (char *s){
  while(*s){ //stop sign of string is '\0'
    if(UCSR0A & (1<<UDRE0))
    UDR0 = *s++; //increment RAM adress
  }
}

void uart_init (void){
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01) | (1<<UCSZ00);
}

int main(void){
  uart_init();
  while(1){
    uint8_t val = uart_getc();
    char str[16];
    itoa(val,str,10);
    uart_puts(str);
    _delay_ms(100);
  }
}

Kann mir da jemand auf die Sprünge helfen?

[BirgerT (Gast)]

Musst halt auch den "Empfänger" einschalten..

    UCSR0B |= (1<<RXEN);     // UART RX einschalten

[Max M. (maxmicr)]

Hm, funktioniert irgendwie trotzdem nicht:

#define F_CPU 16000000L //16MHz
#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8) / (BAUD*16) - 1)

#include <avr/io.h>
#include <util/setbaud.h>
#include <util/delay.h>
#include <stdlib.h>
#include <string.h>

uint8_t uart_getc(void){
  while(!(UCSR0A & (1<<RXC0))){
  }
  return UDR0;
}

void uart_puts (char *s){
  while(*s){ //stop sign of string is '\0'
    if(UCSR0A & (1<<UDRE0))
    UDR0 = *s++; //increment RAM adress
  }
}

void uart_init (void){
  UBRR0 = UBRR_VAL >> 8;
  UBRR0L = UBRR_VAL & 0xFF;

  UCSR0B |= (1<<RXEN0);
  UCSR0A |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01) | (1<<UCSZ00);
}

int main(void){
  uart_init();
  while(1){
    if(UCSR0A & (1<<RXC0)){
      uint8_t val = uart_getc();
      char str[16];
      itoa(val,str,10);
      uart_puts(str);
      _delay_ms(100);
    }
  }
}

Die Daten werden von einem C#-Programm an COM5 gesendet, an dem der µC 
hängt:

            SerialPort sp = new SerialPort("COM5");
            sp.Open();
            while (true)
            {
                sp.Write(new byte[] { 1 }, 0, 1);
                Console.WriteLine(sp.ReadLine());
            }

[BirgerT (Gast)]

..PALMFACE..



Also hier funktioniert's nicht..

void uart_init (void){
  UBRR0 = UBRR_VAL >> 8;
  UBRR0L = UBRR_VAL & 0xFF;

  UCSR0B |= (1<<RXEN0);
  UCSR0A |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01) | (1<<UCSZ00);
}

..und hier hast's gehabt (das Senden)..

void uart_init(void)
{
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

Nun, wenn Du mit dem Tutorial hier nicht zurechtkommst, nimm halt die 
Beispiele aus dem ATmega48A..328P Datenblatt.

[Max M. (maxmicr)]

BirgerT schrieb:
> ..und hier hast's gehabt (das Senden)..

Naja gut, so funktionierts aber auch nicht:

void uart_init (void){
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<RXEN0);
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

[BirgerT (Gast)]

BirgerT schrieb:

> .. nimm halt die Beispiele aus dem ATmega48A..328P Datenblatt.

http://www.atmel.com/images/atmel-8271-8-bit-avr-microcontroller-atmega48a-48pa-88a-88pa-168a-168pa-328-328p_datasheet_complete.pdf

Runterladen, lesen, Beispiele (richtig) abtippen..
- Baudratenteilerregister richtig beschreiben
- Sender und(!) Empfänger einschalten
- UCSR0C Zuweisung haste richtig für 8bit und no Parity

[Max M. (maxmicr)]

BirgerT schrieb:
> Baudratenteilerregister richtig beschreiben

#define UBRR_VAL F_CPU/16/BAUD-1

BirgerT schrieb:
> - Sender und(!) Empfänger einschalten
> - UCSR0C Zuweisung haste richtig für 8bit und no Parity

void uart_init(void)
{
  UBRR0H = (unsigned char)(UBRR_VAL>>8);
  UBRR0L = (unsigned char)UBRR_VAL;
  UCSR0A = (1<<TXEN0);
  UCSR0A = (1<<RXEN0);
  UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}

[BirgerT (Gast)]

Max M. schrieb:
> BirgerT schrieb:
>> Baudratenteilerregister richtig beschreiben
> #define UBRR_VAL F_CPU/16/BAUD-1
> BirgerT schrieb:
>> - Sender und(!) Empfänger einschalten
>> - UCSR0C Zuweisung haste richtig für 8bit und no Parity
 void uart_init(void)
 {
   UBRR0H = (unsigned char)(UBRR_VAL>>8);  // oK
   UBRR0L = (unsigned char)UBRR_VAL;       // oK
   UCSR0A = (1<<TXEN0);   // falsches Register
   UCSR0A = (1<<RXEN0);   // 2x falsch: Register, Sender wieder aus
          // 2 Stopbits   8 Bit/Zeichen
   UCSR0C = (1<<USBS0)  |(3<<UCSZ00);        // oK
 }

Und wie ist die Serielle am PC eingestellt?
9600 Baud, 8 Bit, no Parity, 2 Stopbits (9600,8n2)?!

[Max M. (maxmicr)]

BirgerT schrieb:
> 9600 Baud, 8 Bit, no Parity, 2 Stopbits (9600,8n2)?!

Ja, ist so eingestellt

weder so:

void uart_init(void)
{
  UBRR0H = (unsigned char)(UBRR_VAL>>8);
  UBRR0L = (unsigned char)UBRR_VAL;
  UCSR0B = (1<<TXEN0);
  UCSR0B = (1<<RXEN0);
  UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}

noch so:

void uart_init(void)
{
  UBRR0H = (unsigned char)(UBRR_VAL>>8);
  UBRR0L = (unsigned char)UBRR_VAL;
  UCSR0B = (1<<TXEN0);
  UCSR0A = (1<<RXEN0);
  UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}

funktionierts :(

[Joachim B. (jar)]

STK500-Besitzer schrieb:
> Und der Ersteller der Library wäre ein Idiot.

ähm, so würde ich das nicht sagen, weil sich die Namen öfter mal 
geändert haben ist es oft nicht möglich irgendeine Lib zu nehmen die 
genau nicht für alle AVR passt.

Ich habe in meinem Beispiel zwar 2 -Punkte gesammelt ändert aber nix.

Ich benutze heute lieber Pseudo Defines in "StandardLIBs" und setze die 
echten Namen aus dem Datenblatt ein, zumal es ja wie schon geschieben 
heute UART0 und UART1 gibt, die Interrupt- Bits -register auch gerne mal 
wechseln.

[BirgerT (Gast)]

Geduldsfaden Ende erreicht..

..mach's halt richtig, so wie's im Datenblatt (für Schreibfaule) steht:

void USART_Init( unsigned int ubrr)
{
/*Set baud rate */
UBRR0H = (unsigned char)(ubrr>>8);
UBRR0L = (unsigned char)ubrr;

Enable receiver and transmitter */
// *****************************************************
UCSR0B = (1<<RXEN0)|(1<<TXEN0); // <<-- DING DING DING *
// *****************************************************

/* Set frame format: 8data, 2stop bit */
UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}

aber so nicht
  UCSR0B = (1<<TXEN0); // Im Controlregister wir nur Bit TXEN0 gesetzt
  UCSR0B = (1<<RXEN0); // Im Controlregister wir nur Bit RXEN0 gesetzt,
                       // TXEN0 und alle anderen Bits werden auf 0 
gesetzt

  UCSR0B |= (1<<RXEN0); // so werden einzelne Bits gesetzt, ohne die 
anderen wieder auf 0 zu löschen..

Jetzt aber..

[Max M. (maxmicr)]

BirgerT schrieb:
> Jetzt aber..

Ne, leider nicht:

#define F_CPU 16000000L //16 MHz

#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)

#include <avr/io.h>
#include <stdlib.h>
#include <util/setbaud.h>
#include <util/delay.h>

void uart_putc(unsigned char c)
{
  while (!(UCSR0A & (1<<UDRE0)))  /* warten bis Senden moeglich */
  {
  }
  UDR0 = c;                      /* sende Zeichen */
}

void uart_puts (char *s)
{
  while (*s)
  {
    uart_putc(*s);
    s++;
  }
}

uint8_t uart_getc(void){
  while(!(UCSR0A & (1<<RXC0))){
  }
  return UDR0;
}

void uart_init()
{
  UBRR0H = (unsigned char)(UBRR_VAL>>8);
  UBRR0L = (unsigned char)UBRR_VAL;
  UCSR0B = (1<<RXEN0)|(1<<TXEN0);
  UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}

int main(void){
  uart_init();
  while(1){
    uint8_t val = uart_getc();
    char str[16];
    itoa(val,str,10);
    uart_puts(str);
    _delay_ms(100);
  }
}

[Kai M. (kai_mauer)]

BirgerT schrieb:
> Geduldsfaden Ende erreicht..
>
> ..mach's halt richtig, so wie's im Datenblatt (für Schreibfaule) steht

BirgerT schrieb:
> ..PALMFACE..


Nur mal als kurzer Hinweis: Wenn Du weder LUST noch GEDULD aufbringst, 
etwas zu erklären, dann LASS ES EINFACH.
SCNR

[BirgerT (Gast)]

Kai M. schrieb:
> BirgerT schrieb:

> Nur mal als kurzer Hinweis: Wenn Du weder LUST noch GEDULD aufbringst,
> etwas zu erklären, dann LASS ES EINFACH.

Mach ich auch..

Meine Philosophie ist "Hilfe zur Selbsthilfe" und "Beratungsresistenz 
ignorieren".

Bei der Problematik hier handelt es sich um eine der vielen gut 
dokumentierten Aufgabenstellungen. Also habe ich erst versucht, den OT 
selber eine Lösung finden und das Erfolgserlebnis haben zu lassen..

..um dann doch die Katze aus dem Sack zu lassen.

Also Max_MMM probier bitte nochmal folgendes:

void uart_putc(unsigned char c)
{
  while (!(UCSR0A & (1<<UDRE0)))  /* warten bis Senden moeglich */
  { ; // hier noch ein Semikolon einfügen
  }
  UDR0 = c;                      /* sende Zeichen */
}

uint8_t uart_getc(void){
  while(!(UCSR0A & (1<<RXC0))){
    ; // und hier auch eine "leere Anweisung" 
  }
  return UDR0;
}

Wenn das auch noch nicht geht, probier nochmal das Senden:

char str[] = "Hallo Max_MMM\n-";

int main(void){
  uart_init();

  _delay_ms(100);
  uart_puts(str);

  while(1){
    uint8_t val = uart_getc();
   itoa(val,str,10);
    uart_puts(str);
    _delay_ms(100);
  }
}

und wenn's dann noch immer nicht klappt - bin ich zugegebenermaßen 
erstmal ratlos. Liegt es vielleicht an dem C# Programm?
Versuch doch mal ein normales Terminalprogramm (puTTYtel, Hterm..)

[Max M. (maxmicr)]

Yeah, es geht :)

#define F_CPU 16000000L //16 MHz

#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)

#include <avr/io.h>
#include <stdlib.h>
#include <util/setbaud.h>
#include <util/delay.h>

uint8_t uart_getc(){
  while(!(UCSR0A & (1<<RXC0))){
    ;
  }
  return UDR0;
}

void uart_puts (char *s)
{
  while (*s)
    if(UCSR0A & (1<<UDRE0))
      UDR0 = *s++;
}

void uart_init(void)
{
  UBRR0 = UBRR_VAL;
  UCSR0B |= (1<<TXEN0);
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

int main(void){
  uart_init();
  while(1){
    uint8_t val = uart_getc();
    char str[2];
    itoa(val,str,10);
    uart_puts(str);
    _delay_ms(100);
  }
}

Danke!

P.S. Warum 49 als Ergebnis ausgegeben wird obwohl ich "1" sende, 
verstehe ich aber nicht...

Edit: okay, anscheinend braucht man C#-Programm ein Ende-Zeichen, so 
gehts:

int main(void){
  uart_init();
  while(1){
    uint8_t val = uart_getc();
    char str[3];
    itoa(val,str,10);
    strcat(str,"\n");
    uart_puts(str);
    _delay_ms(100);
  }
}

[BirgerT (Gast)]

BirgerT schrieb:

> und wenn's dann noch immer nicht klappt - bin ich zugegebenermaßen
> erstmal ratlos. Liegt es vielleicht an dem C# Programm?

Augenscheinlich liegt es am C# Programm..
                Console.WriteLine(sp.ReadLine());
..heisst bestimmt, dass der empfangene String ebenfalls mit einem 
"return" abgeschlossen sein muss.

> Versuch doch mal ein normales Terminalprogramm (puTTYtel, Hterm..)

char str[] = "Hallo Max_MMM\r\n";

int main(void){
  uart_init();

  _delay_ms(100);
  uart_puts(str);

  while(1){
    uint8_t val = uart_getc();
   itoa(val,str,10);
    uart_puts(str);
    uart_putc('\r');
    uart_putc('\n');
    _delay_ms(100);
  }
}

[BirgerT (Gast)]

Max M. schrieb:
> Yeah, es geht :)
das ist schön..

> P.S. Warum 49 als Ergebnis ausgegeben wird obwohl ich "1" sende,
> verstehe ich aber nicht...
Weil Du das Zeichen mit itoa in einen Zahlstring konvtertierst, der dem 
Zeichencode in dez entspricht. Nimmste als Radix 16, wird hex 31 
zurückgegeben.. 'A' ist 65 dez und 41 hex
>
> Edit: okay, anscheinend braucht man C#-Programm ein Ende-Zeichen, so
> gehts:
Da haben sich unsere Posts überschnitten..

[Max M. (maxmicr)]

Ist das eigentlich normal, dass der Arduino den Code verliert, wenn man 
ihn vom Strom trennt?

Edit: Hm, ein anderes Programm funktioniert nach dem Strom trennen 
einwandfrei...