ATMega16 LED Matrix

[Tomato (Gast)]

Hallöchen alle miteinander!

Ich habe das Ziel mit meinem ATMega16 ein kleines Tetris Spiel zu 
programmieren, welches auf zwei 7x5 LED Matrizen abgebildet werden soll.
Schritt für Schritt taste ich mich gerade ran, jedoch stehe ich gerade 
vor einem Problem, welches ich nocht nicht so lösen konnte.
Fallen direkt zwei Punkte hintereinander runter verschwindet beim 
Übergang von der oberen in die untere Matrix einer der Punkte (der 
obere).
Also die letzte Zeile der oberen Matrix wird nicht mehr angezeigt, 
sobald die obere Zeile der unteren Matrix angesprochen wird.

xo sind die Variablen für die obere Matrix und xu die Variablen für die 
untere Matrix.

#define F_CPU 16000000
#include <avr/io.h>
#include <avr/interrupt.h>

uint16_t matrix2d[5][2] = {{0,0},{0,0},{0,0},{0,0},{0,0}};

void Init();
void InitPorts();
void InitTimer();

volatile uint_8 var = 1;
volatile uint8_t c = 0;
volatile uint16_t slowy = 0;
volatile uint8_t xo1 = 3;
volatile uint8_t xo2 = 0;
volatile uint8_t xo3 = 0;
volatile uint8_t xo4 = 0;
volatile uint8_t xo5 = 0;
volatile uint8_t xu1 = 0;
volatile uint8_t xu2 = 0;
volatile uint8_t xu3 = 0;
volatile uint8_t xu4 = 0;
volatile uint8_t xu5 = 0;
int main(void)
{
  Init();
  while (1)
  {
   }
}

void Init()
{
  InitPorts();
  InitTimer();
}

void InitPorts()
{
  DDRA= 0xFF;
  DDRB= 0x1F;
  DDRC= 0xFF;
  DDRD= 0x00;
}

void InitTimer()
{
  //Timer1
  TCCR1B |= (1<<CS10) | (1<<CS12) | (1<<WGM12);
  TIMSK |= (1<<OCIE1A);
  OCR1A = 5;
  
  sei();
}

ISR(TIMER1_COMPA_vect)
{
//Multiplexen der Transistoren
  PORTA &= ~0xFF;
  PORTC &= ~0xFF;
   
  var=var*2;
  if (var==32)
  {
  var=1;
  }
  PORTB = ~var;

c++;
if (c == 5)
{
  c = 0;
}

//slowy für eine "angenehme" Geschwindigkeit
slowy++;
if (slowy == 1500)
{
  slowy=0;
//"fallende" Punkte
  if (xu1 <= 64)
  {
    xu1=xu1*2;
  }
  if (xo1 < 127)
  {
    xo1=xo1*2;
  }
  if (xo1 > 127)
  {
    xo1=xo1*2;
    
    if (xu1<0x15)
    {
      xu1=xu1+1;      
    }      
  }  
}

//Übergabe der Werte ans Array
matrix2d[0][0]=xo1;
matrix2d[1][0]=xo2;
matrix2d[2][0]=xo3;
matrix2d[3][0]=xo4;
matrix2d[4][0]=xo5;
matrix2d[0][1]=xu1;
matrix2d[1][1]=xu2;
matrix2d[2][1]=xu3;
matrix2d[3][1]=xu4;
matrix2d[4][1]=xu5;

//Ausgabe des Arrays
PORTA |= matrix2d[c][0];
PORTC |= matrix2d[c][1];

//Debug LED
PORTA ^= (1<<PA7);
}

[Michael B. (laberkopp)]

Tomato schrieb:
>   if (xo1 < 127)
>   {
>     xo1=xo1*2;
>   }
>   if (xo1 > 127)

Wenn man bits abfragt, würde ich eher xo&0x80 auswerten, als ein grösser 
oder kleiner, sonst passiert so ein Mist wie <127 und >127 und keiner 
weiss was genau bei 127 passieren soll.

>     if (xu1<0x15)

?!?

[Falk B. (falk)]

Tomato schrieb:

> Fallen direkt zwei Punkte hintereinander runter verschwindet beim
> Übergang von der oberen in die untere Matrix einer der Punkte (der
> obere).
> Also die letzte Zeile der oberen Matrix wird nicht mehr angezeigt,
> sobald die obere Zeile der unteren Matrix angesprochen wird.
>
> xo sind die Variablen für die obere Matrix und xu die Variablen für die
> untere Matrix.

Naja, bei Namen wie x01, xo2 etc. schreit das gerade nach einem Array. 
Auch deine Platzierung der Matrixfülllogik in der ISR ist nicht 
sinnvoll. Für einen Test mag das OK sein, für weitere 
Softwareentwicklung eher nicht. Deine ganzen volatile Definitionen der 
Variablen sind sinnlos, denn diese werden hier nur in der ISR benutzt, 
nicht im Hauptprogramm. Deine "fallenden Punkte" kann man einfacher und 
übersichtlicher prgrammieren. Wenn man eine Bitversciebung erreichen 
will, schreibt man das meist auch so hin, auch wenn ein *2 logisch das 
Gleiche bewirkt. Deine Logik deiner fallenden Punkte über 2 verteilte 
Bytes hab ich jetzt nicht im Detail geprüft, wo da der Fehler liegt. Das 
Problem kann man aber vereinfachen, indem man einfach 16 Bit für deine 
LED-Matrix benutzt. Upps, das hast du ja schon, aber die oberen 16 Bit 
sind immer ungenutzt. Warum? Siehe Anhang.

Man kann es mit der Modularität und den Unterfunktionen auch 
übertreiben. So einfache Initialisierungen schreibt man einfach so hin 
und gut, da braucht es keine Funktion.

Man sollte sinnvolle, aussagekräftige Variablennamen benutzen, "var" 
sagt gar nix.

Beim multiplexen einer LED-Matrix sollte man folgende Reihenfolge 
einhalten, um Geisterbilder zu vermeinden.

aktuelle Spalte ausschalten
neues Spaltenmuster berechnen und ausgeben
neue Spalte einschalten

Durch diese kleine Pause von wenigen us vermeidet man Nachleuchten des 
Musters einer Spalte in der nächsten.