Uart Variable

[Michael H. (h_m)]

Guten Tag,

ich habe mich jetzt schon länger nicht mehr mit meinem Code beschäftig. 
Und jetzt bringe ich es nicht mehr zu stande die Variable

val

 mit dem Uart an mein Terminal zu senden. Vielleicht kann mir ja jemand 
zeigen was ich verkehrt mache.


anstatt val nur über Leds auszugeben würde ich gerne, den Wert von val 
auch gerne im Hterm ausgeben.

 for(;;){
     val += encode_read1();          // read a single step encoder
     LEDS = val;
     }

 

und ich glaube mein Fehler Liegt hier, aber ich weis nicht mehr warum.
[c] while (1)
   {
     
  itoa( val, buffer, 10);      // convert interger into string (decimal format)
  uart_puts(buffer);         // and transmit string to UART
  uart_puts("\r\n");         //neue Zeile verwenden CR+LF im Terminal
  _delay_ms(1);
  
   }

hier der Ganze Code

/***************************************************************************************************

Beispiel um den Inhalt einer Variable mit der "Interrupt controlled UART library"
von Peter Fleury an das terminal Hterm zu senden.



Um die Libary zu verwenden müssen die Dateien Soloutionexplorer mit eingebunden werden.
uart.c
uart.h   

 
Atmega88P
uartlibary       
****************************************************************************************************/

#define F_CPU 16000000

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


/* Define UART buad rate here */
#define UART_BAUD_RATE      9600  



/* Kofiguration Drehgeber */
#define XTAL        16e6         // 16MHz


#define PHASE_A     (PINB & 1<<PB0)
#define PHASE_B     (PINB & 1<<PB1)

#define LEDS_DDR    DDRD
#define LEDS        PORTD           // LEDs against VCC

volatile int8_t enc_delta;          // -128 ... 127
static int8_t last;

void encode_init( void ) {
  int8_t new;

  // init encoder
  new = 0;
  if( PHASE_A ) new  = 3;
  if( PHASE_B ) new ^= 1;       // convert gray to binary
  last = new;                         // power on state
  enc_delta = 0;

  // init timer 0
  TCCR1B = (1<<WGM12) | (1<<CS12) ;   // CTC, prescaler 256
  OCR1A = (uint8_t)(XTAL / 256 * 1e-3 - 0.5);   // 1ms
  TIMSK1 |= (1<<OCIE1A);              // interrupt bei Macht aktivieren
}

ISR( TIMER1_COMPA_vect ) {           // 1ms for manual movement / encoder mit Handbetätigung
  int8_t new, diff;

  new = 0;
  if ( PHASE_A ) new  = 3;
  if ( PHASE_B ) new ^= 1;   // convert gray to binary
  diff = last - new;               // difference last - new
  if( diff & 1 ) {                 // bit 0 = value (1)
    last = new;                    // store new as next last
    enc_delta += (diff & 2) - 1;   // bit 1 = direction (+/-)
  }
}

// read 1, 2, or 4 step encoders
int8_t encode_read1( void )
{
  int8_t val;

  // atomic access to enc_delta
  cli();
  val = enc_delta;
  enc_delta = 0;
  
  sei();
  return val;                   // counts since last call
}   

 
int main(void)
{
  volatile  int32_t val = 0;

   LEDS_DDR = 0xFF;
   encode_init();
   sei();

   for(;;){
     val += encode_read1();          // read a single step encoder
     LEDS = val;
     }
     
     
    uint16_t x = 1024;
    char buffer[7];
    
   
    uart_init(UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
    
  sei();
 
    
   while (1)
   {
     
  itoa( val, buffer, 10);      // convert interger into string (decimal format)
  uart_puts(buffer);         // and transmit string to UART
  uart_puts("\r\n");         //neue Zeile verwenden CR+LF im Terminal
  _delay_ms(1);
  
   }
   
    
    
}

[tralala (Gast)]

So schnell drübergeschaut: du kommst nie aus der for(;;){...} Schleife 
raus, also wird alles nachher in der main nie ausgeführt.

[Michael H. (h_m)]

wenn ich die For schleife ausklammere und in die While schleife unten 
einsetze geht es immer noch nicht. wo könnte ich denn noch einen Fehler 
haben ?

hier der Code ohne For[;;]

/***************************************************************************************************

Beispiel um den Inhalt einer Variable mit der "Interrupt controlled UART library"
von Peter Fleury an das terminal Hterm zu senden.



Um die Libary zu verwenden müssen die Dateien Soloutionexplorer mit eingebunden werden.
uart.c
uart.h   

 
Atmega88P
uartlibary       
****************************************************************************************************/

#define F_CPU 16000000

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


/* Define UART buad rate here */
#define UART_BAUD_RATE      9600  



/* Kofiguration Drehgeber */
#define XTAL        16e6         // 16MHz


#define PHASE_A     (PINB & 1<<PB0)
#define PHASE_B     (PINB & 1<<PB1)

#define LEDS_DDR    DDRD
#define LEDS        PORTD           // LEDs against VCC

volatile int8_t enc_delta;          // -128 ... 127
static int8_t last;

void encode_init( void ) {
  int8_t new;

  // init encoder
  new = 0;
  if( PHASE_A ) new  = 3;
  if( PHASE_B ) new ^= 1;       // convert gray to binary
  last = new;                         // power on state
  enc_delta = 0;

  // init timer 0
  TCCR1B = (1<<WGM12) | (1<<CS12) ;   // CTC, prescaler 256
  OCR1A = (uint8_t)(XTAL / 256 * 1e-3 - 0.5);   // 1ms
  TIMSK1 |= (1<<OCIE1A);              // interrupt bei Macht aktivieren
}

ISR( TIMER1_COMPA_vect ) {           // 1ms for manual movement / encoder mit Handbetätigung
  int8_t new, diff;

  new = 0;
  if ( PHASE_A ) new  = 3;
  if ( PHASE_B ) new ^= 1;   // convert gray to binary
  diff = last - new;               // difference last - new
  if( diff & 1 ) {                 // bit 0 = value (1)
    last = new;                    // store new as next last
    enc_delta += (diff & 2) - 1;   // bit 1 = direction (+/-)
  }
}

// read 1, 2, or 4 step encoders
int8_t encode_read1( void )
{
  int8_t val;

  // atomic access to enc_delta
  cli();
  val = enc_delta;
  enc_delta = 0;
  
  sei();
  return val;                   // counts since last call
}   

 
int main(void)
{
    int32_t val = 0;

   LEDS_DDR = 0xFF;
   encode_init();
   sei();

   //for(;;){
   //  val += encode_read1();          // read a single step encoder
   //  LEDS = val;
    // }
     
     
    uint16_t x = 1024;
    char buffer[7];
    
   
    uart_init(UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
    
  sei();
 
    
   while (1)
   {
     
    val += encode_read1();          // read a single step encoder
  LEDS = val;
   
  itoa( val, buffer, 10);      // convert interger into string (decimal format)
  uart_puts(buffer);         // and transmit string to UART
  uart_puts("\r\n");         //neue Zeile verwenden CR+LF im Terminal
  _delay_ms(1);
  
   }
   
    
    
}

[Schaltplanleser (Gast)]

Michael H. schrieb:
> hier der Code ohne For[;;]

Vielleicht doch mal einen Editor benutzen um ein paar Leerzeilen
rauszumachen? Oder bekommt man dafür Geld wenn man die Leserschaft
so mit Overhead überschüttet?

Ich schlage vor deine Source ist gross genug sie in einen Anhang
zu stecken, das macht die Suche einfacher und die Hilfsbereit-
schaft grösser.

[A. S. (Gast)]

Was funktioniert denn nicht?

Kannst Du ein Zeichen senden?

Kannst Du mehrere Zeichen senden?

Kannst Du einen String senden?

Kannst Du mehrere Strings senden?

Kannst Du einen festen wert senden?

Nur wenn alle Fragen mit ja zu beantworten sind, ist dein val irgendwie 
involviert.

Ansonsten beginne beim ersten nein.

Meine Frage (wenn die ersten Versuche ok sind) wäre, was genau uart-puts 
macht. Aber da bist Du noch nicht.

[Veit D. (devil-elec)]

Hallo,

das Einzigste was mir auffällt ist, dass dein char buffer für uint32 zu 
klein ist. Dezimal sind das 10 Ziffern + \0 muss der 11 Zeichen groß 
sein. Teste ob einzelne feste Zeichen/Zahlen überhaupt übertragen werden 
können.

[Michael H. (h_m)]

Danke, es hat sich wohl irgendetwas aufgehäng, alles neu getsartet und 
jetzt gehts, danke noch mal.

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

A. S. schrieb:
> Was funktioniert denn nicht?
Was soll denn passieren?
Und was passiert stattdessen?
Wie stellst du das fest?

Michael H. schrieb:
> es hat sich wohl irgendetwas aufgehäng
Ich würde dem auf jeden Fall nochmal nachgehen. Kann ja laufend wieder 
passieren.

> jetzt gehts
Läuft! Ausliefern!

BTW: mir wäre da mindestens ein sei() zu viel drin. Das sieht "schnell 
mal zusammenkopiert" aus...