Ausgabeprobleme LCD 4Bit

[Klaus H. (Firma: privat) (klaus777)]

Hallo in die Runde,
ich habe sicherlich ein Anfängerproblem.
Ich wollte einen Zähler rückwärts laufen lassen und den Wert auf einem 
LCD-Display darstellen.
Das initialisieren des Displays klappt,
die beiden Werte 'Timer' und '3 Min' erscheinen richtig,
nur die Ausgabe des Wertes leider nicht. Da stehen nur asiatische 
Zeichen.
Ich habe verschiedene Displays getestet HD44780 oder 1602.
Selbe Problematik.

Danke für eure Unterstützung.
aus dem Makefile:

MCU = attiny2313
F_CPU = 1000000


Meine Source:

#include <avr/io.h>
#include <stdlib.h>
#include "lcd-routines.h"
#include <avr/interrupt.h>
#include <util/delay.h>


volatile int zeit1 = 180;
volatile int zeit ;

void setup_global(void)
{
DDRD= 0xff;
}

void setup_timer(void)
{
// Timer1
    TCCR1A = (1<<COM1A0);  // CTC mode, toggle OC1A on compare match
    TCCR1B = (1<<CS12)         // Start Timer1 with prescaler 256
             |(1<<WGM12);
    OCR1A = 3906;  // F_Timer = F_clk/Prescaler
    // Compare Interrupt erlauben
    TIMSK |= (1<<OCIE1A);
}
void Displ_def(void)
{
  lcd_clear();
  lcd_string("Timer ");
  lcd_setcursor( 0, 2 );
  lcd_string("3 Min");
  return;
} 

void display_time(int x)
{
  lcd_setcursor( 15, 2 );
  char Buffer[5]; // in diesem {} lokal
  itoa( zeit, Buffer, 10 ); 
  lcd_string( Buffer );
}
 
int main(void)
{
  setup_global();
  lcd_init();
  Displ_def();
  setup_timer();
  sei();
  while(1)
  {
    display_time(zeit);
    _delay_ms(100);
  PORTD ^= ( 1 << PD1 );
  }

}
ISR (TIMER1_COMPA_vect)
{
  zeit1 --;
  zeit=zeit1;
}

--

Bitte die Formatierungstags verwenden, die das Forum bietet. So ist 
Dein Quelltext gleich lesbarer.

-rufus

[Mark B. (markbrandis)]

Das beschriebene Verhalten passt wohl zu einer nicht korrekt gesetzten 
Variablen.

Auffälligkeiten:

1.) Die Variable zeit wird zu Beginn auf keinen vernünftigen Wert 
initialisiert.

2.) In dieser Funktion hier:
void display_time(int x)
wird der Parameter x nirgendwo verwendet. Wozu existiert er dann?

3.) Bist Du sicher, dass die Interrupt Service Routine korrekt 
konfiguriert ist und auch ausgeführt wird? Das ist die einzige Stelle im 
Code, an der die Variable zeit beschrieben wird.

4.) Die Variable zeit1 wird irgendwann negativ (so die ISR immer wieder 
ausgeführt wird). Dann hätte auch die Variable zeit einen negativen 
Wert. Das ist vermutlich nicht was Du willst.

[holger (Gast)]

>Dann hätte auch die Variable zeit einen negativen Wert.

Und passt dann nicht mehr in den Buffer.
Der müsste mit abschliessender Null schon
mal 7 Bytes aufnehmen können.

[Mark B. (markbrandis)]

holger schrieb:
> Und passt dann nicht mehr in den Buffer.
> Der müsste mit abschliessender Null schon
> mal 7 Bytes aufnehmen können.

Das wäre dann Punkt 5, den ich vergessen habe. ;-)

[Klaus H. (Firma: privat) (klaus777)]

Hallo und Danke.

1. verständlich, ich hatte die Variable zu Testzwecken noch im Source, 
das entschuldigt es nicht. Sie wird richtig gefüllt und 
weiterverarbeitet.
2. stimmt auch, wurde korrigiert.
3. Ja, wurde mit >>PORTD ^= ( 1 << PD2 );<< in der ISR getestet
4. im Moment stehe ich vor dem Problem das die Ausgabe im Display nicht 
funktioniert, der Rest wurde noch nicht implementiert.
(5.) ist richtig, erklärt 4.
Anbei das Bild vom Display.

[Ralph S. (jjflash)]

Ich kenne deine lcd-rotuines.h nicht, aber ich gehe stark davon aus, 
dass

  lcd_string

als Parameter einen String aus dem Progmem erwartet, du aber mit

  lcd_string( Buffer );

einen Array übergibst. Hier wundert es mich, dass der Compiler nicht 
meckert, denn normalerweise hätte zumindest ein Zeiger übergeben werden 
müssen ...

Du brauchst also fuer die Ausgabe eines Strings im Ram eine 
entstprechende Funktion !

Wenn es in lcd-routines.h etwas in der Art wie:

void lcd_putchar(char ch)

gibt (so in der Art halt), dann lässt sich eine Ausgaberoutine leicht 
selbst implementieren.


void lcd_ramstring(char *c)
{
  while (*c)
  {
    lcd_putchar(*c++);
  }
}

Der Aufruf in deinem Programm würde dann erfolgen mit:

lcd_ramstring(&Buffer[0]);

Im übrigen:

Du reservierst füf <Buffer> 5 Bytes. Ein Integer kann maximal den Wert 
65535 annehmen (und ist somit 5 stellig). Ein abschließendes Zero Byte 
(als Endekennung des Strings) wird bei <itoa> automatisch mit angehängt, 
sodass ein Ergebnis im schlimmen Fall 6 Bytes Speicher benötigt !

Gruß am Morgen,

Ralph

[Ralph S. (jjflash)]

Ralph S. schrieb:
> im schlimmen Fall 6 Bytes Speicher benötigt !

Ah, wurde oben schon erwähnt, sorry: 6 Bytes natürlich für ein

unsigned integer (vorzeichenlos)

ansonsten 7.

[Michael U. (amiga)]

Hallo,

Ralph S. schrieb:
> Ich kenne deine lcd-rotuines.h nicht, aber ich gehe stark davon aus,
> dass
>
>   lcd_string
>
> als Parameter einen String aus dem Progmem erwartet, du aber mit
>
>   lcd_string( Buffer );
>
> einen Array übergibst. Hier wundert es mich, dass der Compiler nicht
> meckert, denn normalerweise hätte zumindest ein Zeiger übergeben werden
> müssen ...

ich bin nicht der sonderliche C-Spezi, aber wie ich es verstanden habe, 
wird in C in diesem Fall immer der Zeiger auf das Array übergeben.

Ich lasse mich da aber gern korrigieren.

Gruß aus Berlin
Michael

[Klaus H. (Firma: privat) (klaus777)]

Hallo und Danke,

ich habe erst hier im Forum gelesen wie so was angesteuert wird.
https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/LCD-Ansteuerung

dort findet sich auch folgendes Beispiel:

// Ausgabe der Variable als Text in dezimaler Schreibweise
  {
     // ... umwandeln siehe FAQ Artikel bei 
http://www.mikrocontroller.net/articles/FAQ
     // WinAVR hat eine itoa()-Funktion, das erfordert obiges #include 
<stdlib.h>
     char Buffer[20]; // in diesem {} lokal
     itoa( variable, Buffer, 10 );

     // ... ausgeben
     lcd_string( Buffer );
  }

Gruß Klaus.