Led mit UART ansteuern

[Kanaille (Gast)]

Hallo Leute, bin noch ein MC Greenhorn.

Ich versuche eine LED oder irgendwas durch eine über serielle 
Schnittstelle übermittelten Variable zu steuern. Wenn die Variable 1 ist 
dann soll dies gemacht werden und bei 2 jenes. Die Baudrate und alles 
ist richtig, also es ist schonmal was angekommen.
Könnt ihr mir vll helfen ?

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

#define F_CPU 16000000UL

/* 9600 baud */
#define BAUD  9600 
#include <util/setbaud.h>

char = c;

void setutil(){
UBRR0H = UBRRH_VALUE;
UBRR0L = UBRRL_VALUE;
UCSR0C = (1<<UCSZ01)|(1<<UCSZ00); // Asynchron 8N1
UCSR0B |= (1<<RXEN0)|(1<<TXEN0)|(1<<RXCIE0);     
}


ISR(USART_RXC_vect){
  c= UDR0;
}

int main(void) {
  setutil();
  DDRC |= (1<<PC5);
  sei();
  while (1) {
    if (c ==1)
    {
      PORTB &= (!(1<<PB1));
      _delay_ms(5000);
      PORTB |= (1<<PB1);
      _delay_ms(5000);
    
    } 
    else
    {
    }
    PORTB = (0<<PB1);  
  }
  return 1;
}

--

Bitte in Zukunft die Formatierungszeichen für Quelltext ([с][/с]) 
verwenden.

-rufus

[Uwe (de0508)]

Sind ein paar Frage erlaubt ?

was ist PC5 ?

was ist /char = c;/ - wird das überhaupt übersetzt ?

Ich würde in der ISR USART_RXC_vect ein Event-Flag setzen (neues 
Zeichen da!).

Eine weitere Möglichkeit ist, das empfange Zeichen in einem RX-Fifo zu 
speichern und über die RX-Fifo Methode new_char():

bool_t rx_fifo_new_char(void);

abfragen, ob neue Zeichen eingetroffen sind.

was soll den hier passieren ?

else {
}
PORTB = (0<<PB1);

Das was ist mir klar.

[Joachim B. (jar)]

Kanaille schrieb:
> else
>     {
>     }

auch sinnlos

Uwe S. schrieb:
> was ist PC5 ?

ein Portbit 5 im PortC eines Atmel, hier unbenutzt.

> was ist /char = c;/ - wird das überhaupt übersetzt ?

kann ich mir nicht vorstellen ohne Fehlermeldung.

[Uwe (de0508)]

@Joachim B.,

das sind doch teilweise rhetorische Fragen, Danke.
Ich kenne mich mit den AVR aus.

[Joachim B. (jar)]

OK sorry....

[Jochen S. (jochen_s)]

Hallo  Kanaille ,

was definitiv Fakt ist, eine Variable die im Interrupt und im "normalen" 
Programm genutzt werden soll, muss !volatile! sein.


Das Delay blockiert den kompletten Prozess.....

Gruß
Jochen

[Kanaille (Gast)]

Vielen Dank schonmal für die Beteiligung, ich bin echt beeindruckt von 
diesem Forum !

ja in dem Quellcode sind echt grobe Fehler drin, dafür muss ich mich 
entschuldigen und ein bisschen konzentrieren.

meine nächste Frage ist, kann ich diese ganze Umwandlung umgehen indem 
ich dem MC hexadezimalzahlen schicke? ich glaube für meine Anwendung 
würde das reichen ...

[Karl H. (kbuchegg)]

Kanaille schrieb:

> ja in dem Quellcode sind echt grobe Fehler drin, dafür muss ich mich
> entschuldigen und ein bisschen konzentrieren.

Das solltest du unbedingt.

> meine nächste Frage ist, kann ich diese ganze Umwandlung

welche Umwandlung?

Ob du im Programm schreibst

...
    if( c == 'E' )             // E wie 'ein'
      PORTB &= ~( 1 << PB1 );
    else if( c == 'A' )        // A wie 'aus'
      PORTB |= ( 1 << PB1 );     
...

und dann auf deinem Terminal die Taste E bzw. A drückst, ist doch völlig 
unabhängig davon, dass die Variable volatile sein muss (weil sie in 
einer ISR beschrieben und in main abgefragt wird) und dass du dein C 
richtig schreiben musst.

Viellcith solltest du deine ersten Versuche mal mit einer 
Nicht-Interrupt-getriebenen Lösung machen, wenn du mit Interrupts noch 
nicht so auf du-und_du stehst und noch mit recht banalen C Grundlagen 
kämpfst?

[Karl H. (kbuchegg)]

Und noch ein kleiner Tipp:
Du hast eine UART. DIe kann man auch wunderbar dazu benutzen, dass einem 
das Programm dort Ausgaben machen kann. Zb. Ausgaben, in denen einem das 
Programm selber mitteilt, was es aus welchem Grund gemacht hat.
Es gibt kein Gesetz, dass einem verbieten würde, während der 
Entwicklungszeit Ausgaben zu machen, die im endgültigen Programm dann 
wieder rausfliegen.

[Kanaille (Gast)]

irgendwie kommt hier nichts an..
muss ich die Variable noch irgendwo zwischenspeichern oder hat das vll. 
was mit dem Terminal zutun?
MfG

[с]
#include <stdlib.h>
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>

#define F_CPU 16000000UL

/* 9600 baud */
#define BAUD  9600
#include <util/setbaud.h>

volatile c = '2';

void setutil(){
  UBRR0H = UBRRH_VALUE;
  UBRR0L = UBRRL_VALUE;
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00); // Asynchron 8N1
  UCSR0B |= (1<<RXEN0)|(1<<TXEN0)|(1<<RXCIE0);
}


ISR(USART_RX_vect){

  c = UDR0;
}

int main(void) {
  setutil();
  DDRD |= (1<<4);
  sei();
  while (1) {
    if( c == '2' )
    PORTD = ( 0 << PD4);
    else if( c == '1' ) {
    PORTD = ( 1 << PD4 );
    }
  }
  return 1;
}
[/с]

[Karl H. (kbuchegg)]

Kanaille schrieb:
> irgendwie kommt hier nichts an..

Dann würde ich halt mal an deiner Stelle die Umkehrung probieren.
Nicht der PC sendet, sondern der µC sendet. Denn im Unterschied zum µC 
kannst du im Terminal-Programm vom PC, ob vom µC was kommt und wenn ja, 
ob das richtig ist.

> int main(void) {
>   setutil();
>   DDRD |= (1<<4);
>   sei();

    uart_putc( 'x' );


eventuell in einer Schleife mal 10 oder 20 mal ein x ausgeben.
Wenn du die x im Terminal nicht siehst, dann stimmt was nicht.
Und dem solltest du zuerst nachgehen, ehe du im Nebel stocherst, warum 
am µC nichts ankommt oder das falsche ankommt oder..

Stimmt die Ausgabe am Terminal, dann hindert dich nichts und niemand 
daran, mal den µC das was er gekriegt hat, zurückschicken zu lassen

  while (1) {
    uart_putc( c );

    if( c == '2' ) PORTD = ( 0 << PD4);
    else if( c == '1' ) { PORTD = ( 1 << PD4 );
  }


Aber erst muss die Datenrichtung µC->PC funktionieren. Wenn die nicht 
funktikoniert, brauchst du mit der Gegenrichtung PC->µC gar nicht erst 
anfangen.


(Handshake am PC hast du abgeschaltet?)

[Kanaille (Gast)]

soo , ich habe den Fehler gedunden !
ich möchte mich nochmal bei allen bedanken

[Marc V. (Firma: Vescomp) (logarithmus)]

Kanaille schrieb:
> soo , ich habe den Fehler gedunden !

 Möchtest du uns den gedundenen Fehler vielleicht mitteilen ?