Spielkonsole mit LED-Matrix und 2 Atmega644

[Julian B. (juli55)]

Hallo, ich habe mir eine Spielkonsole gebaut mit Tastern und zur Anzeige 
eine LED-Matrix ich steuere sie mit 2 Atmega644.
Ich habe mehrere Matrix pins an dem einen Controllern und mehrere an dem 
anderen.
Ich lass sie über UART miteinander kommunizieren und das funktioniert 
auch soweit.
Mein Problem ist,dass wenn ich erst eine Led, welcher der Pin der Reihe 
an dem einen controller hängt und die Zeile der Led an dem anderen 
Controller, oder eben anderstherum ist und ich dann zu einer led 
wechseln will bei dem reihe und zeile amersten controller hängen dann 
bleibt die erstgenannte led trotzdem an. Hat vlt. jemand eine Idee

[Dennis (Gast)]

Ist ja super wenn es soweit klappt, aber wieso für sowas zwei uC?

[Fehler (Gast)]

Es liegt am C16, bestimmt.

[Julian B. (juli55)]

c16?
PS: Wenn ich was wichtiges nicht genannt habe schreibt es einfach 
anstatt sich drüber aufzuren hilft allen mehr

[Fehler (Gast)]

Naja,

wir kennen weder deinen Schaltplan, noch deine Quellcode,
ich vermute dann immer, das C16 defekt ist und die Probleme
erzeugt.

Vielleicht fehlen auch nur Treiber-Transistoren, aber das wird
sich zeigen, wenn du mehr als sagen wie 10 LEDs anhast, und jede
etwa 2mA zieht..

[(Gast) (Gast)]

Muss ich meine Brillen putzen, oder seht ihr auch keinen Spannungsregler 
auf der Platine? Habe ich den Schaltplan auch übersehen?

[Julian B. (juli55)]

Also das ist noch ein altes  bild davon und ich hab jetzt n 
Spannungsregler und Bypasskondensatoren an den mcs und die kondensatoren 
für den regler. Das war noch mein altes Problem.

PS: Es handelt sich hier jedoch um ein Softwareproblem nicht um ein 
Hardwarefehler

[Max H. (hartl192)]

Julian Bertsch schrieb:
> PS: Es handelt sich hier jedoch um ein Softwareproblem nicht um ein
> Hardwarefehler
Jetzt sehe ich den Fehler. In der 32. Zeile der main hast du den 
Dereferenzierungsoperator falsch angewendet.


Poste deinen Code, und der Schaltplan könnte bei der Fehlersuche auch 
hilfreich sein.

[Julian B. (juli55)]

1. Kann man mich nicht eifach auf was fehlendes Hinweisen anstatt gleich 
sarkastisch reagiert aber egal jetz, ich poste einfach mal den quellcode

main mc1

#include <avr/io.h>
#define numberOfButtons  2
#include <util/delay.h>
#include "UART_read.h"
#include "UART_send.h"

#include "matrix_register.h"
#include "button_register.h"
#include "bild.h"


int  main(void){

DDRD |= (1<<7);
DDRD &= ~(1<< PIND5) | (1<< PIND6);

PORTD |= 1 << PIND5 | 1 << PIND6;



DDRC = (1<<DDC0) | (1<<DDC1) | (1<<DDC2) | (1<<DDC3) | (1<<DDC4)| (1<<DDC5) | (1<<DDC6) | (1<<DDC7);
PORTC |= (1<<DDC0) | (1<<DDC1) | (1<<DDC2) | (1<<DDC3) | (1<<DDC4)| (1<<DDC5) | (1<<DDC6) | (1<<DDC7);

DDRB = (1<<DDB0) | (1<<DDB1) | (1<<DDB2);
PORTB |= (1<<DDB0) | (1<<DDB1) | (1<<DDB2);
DDRA = (1<<DDA0) | (1<<DDA1) | (1<<DDA2) | (1<<DDA3) | (1<<DDA4) | (1<<DDA5) | (1<<DDA6) | (1<<DDA7);
PORTA |= (1<<DDA0) | (1<<DDA1) | (1<<DDA2) | (1<<DDA3) | (1<<DDA4) | (1<<DDA5) | (1<<DDA6) | (1<<DDA7);


PORTC |= (1<<PC0) | (1<<PC1) | (1<<PC2)  | (1<<PC3) | (1<<PC4) | (1<<PC5) | (1<<PC6) | (1<<PC7);


PORTD &= ~(1<<7);
  
PORTB &= ~(1<<0);

PORTB &= ~(1<<2);

PORTB &= ~(1<<1);

PORTA &= ~(1<<0);
  
PORTA &= ~(1<<1);

PORTA &= ~(1<<2);

PORTA &= ~(1<<3);

PORTA &= ~(1<<4);

PORTA &= ~(1<<5);

PORTA &= ~(1<<6);

PORTA &= ~(1<<7);



  
int Pressed =0;
int i = 0;
int pos_s1 = 1;
int pos_s2 = 1;
int pos_col_ball = 2;
int pos_row_ball = 2;

  int UBBRValue = 25;

//Put the upper part of the baud number here (bits 8 to 11)
UBRR0H = (unsigned char) (UBBRValue >> 8);

//Put the remaining part of the baud number here
UBRR0L = (unsigned char) UBBRValue;

//Enable the receiver and transmitter
UCSR0B = (1 << RXEN0) | (1 << TXEN0);

//Set 2 stop bits and data bit length is 8-bit
UCSR0C = (1 << USBS0) | (3 << UCSZ00);



  while(1)
  {
  
  
    
    
  
      
      

      if (ButtonPressed(0, PIND, 5, 0)){
      
      pos_s1++;
      if(pos_s1 >= 16 ){
      pos_s1 = 16;
      }
      }
      
      if (ButtonPressed(1, PIND, 6, 0)){
      
      pos_s1--;
      if(pos_s1 < 1 ){
      pos_s1 = 1;
      }
      }
      
      
    


  
//bild(i,pos_s1,pos_s2,pos_col_ball,pos_row_ball);

if(i<= 500){

  matrix(1,9);
  
  }
  else {
  if(i<= 1000){

  matrix(16,8);
  
  }
  else{
  if(i<= 1500){
  matrix(16,7);

  }
  else{
  i = 0;
  }
  }
  }
i++;
  
}
}

matrix_register.h

#include <avr/io.h>
void matrix(int col, int row){
float ms = 0.1;

if((col == 5 && row == 9)||(col == 5 && row == 10)|| 
(col == 5 && row == 11) ||(col == 5 && row == 12)|| 
(col == 5 && row == 13) ||(col == 5 && row == 14)|| 
(col == 5 && row == 15)|| (col == 5 && row == 16)){

switch(row){
case  9:
uart_send(1);
while(uart_read() != 0);
break;

case  10:
uart_send(2);
while(uart_read() != 0);
break;

case  11:
uart_send(34);
while(uart_read() != 0);
break;

case  12:
uart_send(4);
while(uart_read() != 0);
break;

case  13:
uart_send(5);
while(uart_read() != 0);
break;

case  14:
uart_send(6);
while(uart_read() != 0);
break;

case  15:
uart_send(20);
while(uart_read() != 0);
break;

case  16:
uart_send(8);
while(uart_read() != 0);
break;
}

}
else if((col == 6 && row == 9)||(col == 6 && row == 10)|| 
(col == 6 && row == 11) ||(col == 6 && row == 12)|| 
(col == 6 && row == 13) ||(col == 6 && row == 14)|| 
(col == 6 && row == 15)|| (col == 6 && row == 16)){
switch(row){
case  9:
uart_send(9);

break;

case  10:
uart_send(10);

break;

case  11:
uart_send(11);

break;

case  12:
uart_send(12);

break;

case  13:
uart_send(13);

break;

case  14:
uart_send(14);

break;

case  15:
uart_send(33);

break;

case  16:
uart_send(16);

break;
}
}
else if((col == 7 && row == 9)||(col == 7 && row == 10)|| 
(col == 7 && row == 11) ||(col == 7 && row == 12)|| 
(col == 7 && row == 13) ||(col == 7 && row == 14)|| 
(col == 7 && row == 15)|| (col == 7 && row == 16)){
switch(row){
case  9:
uart_send(17);
while(uart_read() != 0);
break;

case  10:
uart_send(18);
while(uart_read() != 0);
break;

case  11:
uart_send(19);
while(uart_read() != 0);
break;

case  12:
uart_send(35);
while(uart_read() != 0);
break;

case  13:
uart_send(21);
while(uart_read() != 0);
break;

case  14:
uart_send(22);
while(uart_read() != 0);
break;

case  15:
uart_send(23);
while(uart_read() != 0);
break;

case  16:
uart_send(24);
while(uart_read() != 0);
break;
}
}

else if((col == 8 && row == 9)||(col == 8 && row == 10)|| 
(col == 8 && row == 11) ||(col == 8 && row == 12)|| 
(col == 8 && row == 13) ||(col == 8 && row == 14)|| 
(col == 8 && row == 15)|| (col == 8 && row == 16)){
switch(row){
case  9:
uart_send(25);
while(uart_read() != 0);
break;

case  10:
uart_send(26);
while(uart_read() != 0);
break;

case  11:
uart_send(27);
while(uart_read() != 0);
break;

case  12:
uart_send(28);
while(uart_read() != 0);
break;

case  13:
uart_send(29);
while(uart_read() != 0);
break;

case  14:
uart_send(30);
while(uart_read() != 0);
break;

case  15:
uart_send(31);

break;

case  16:
uart_send(32);

break;
}
}
else{
switch(col){

case 1:
 

PORTD |= 1 << 7;

 break;

case 2:
  
  
  PORTB |= 1 << 0;
  
break;

case 3:


  
PORTB = 1 << 2;

break;

case 4:

  PORTB |= 1 << 1;

break;

case 5:

  uart_send(0b11111111);

break;

case 6:

  uart_send(0b00000011);

break;

case 7:

  uart_send(0b00000111);

break;
case 8:

  uart_send(0b00001111);

break;



case 9:

PORTA |= 1 << 0;
  
  

break;

case 10:


PORTA |= 1 << 1;

break;

case 11:


PORTA |= 1 << 2;

break;

case 12:


PORTA |= 1 << 3;

break;

case 13:

PORTA |= 1 << 4;

break;

case 14:


PORTA |= 1 << 5;

break;

case 15:


PORTA |= 1 << 6;

break;

case 16:


PORTA |= 1 << 7;

break;





}

switch(row){
case 1:
 


PORTC &= ~(1<<PC0);
_delay_ms(ms);
PORTC |= (1<<PC0);
 
break;

case 2: 

PORTC &= ~(1<<PC1);
_delay_ms(ms);
PORTC |= (1<<PC1);

break;

case 3:
PORTC &= ~(1<<PC2);
_delay_ms(ms);
PORTC |= (1<<PC2);
break;

case 4:
PORTC &= ~(1<<PC3);
_delay_ms(ms);
PORTC |= (1<<PC3);
break;

case 5:
PORTC &= ~(1<<PC7);
_delay_ms(ms);
PORTC |= (1<<PC7);
break;

case 6:
PORTC &= ~(1<<PC6);
_delay_ms(ms);
PORTC |= (1<<PC6);
break;

case 7:
PORTC &= ~(1<<PC5);
_delay_ms(ms);
PORTC |= (1<<PC5);
break;

case 8:
PORTC &= ~(1<<PC4);
_delay_ms(ms);
PORTC |= (1<<PC4);
break;



case 9:
uart_send(0b00000000);
while(uart_read() != 0);
break;

case 10:
uart_send(0b10000000);
while(uart_read() != 0);
break;

case 11:
uart_send(0b11000000);
while(uart_read() != 0);
break;

case 12:
uart_send(0b11100000);
while(uart_read() != 0);
break;

case 13:
uart_send(0b11110000);
while(uart_read() != 0);
break;

case 14:
uart_send(0b11111000);
while(uart_read() != 0);
break;

case 15:
uart_send(0b11111100);
while(uart_read() != 0);
break;

case 16:
uart_send(0b11111110);
while(uart_read() != 0);
break;
}




switch(col){

case 1:
 


PORTD &= ~(1<<7);
 break;

case 2:
  
  
  
  PORTB &= ~(1<<0);
break;

case 3:


  

PORTB &= ~(1<<2);
break;

case 4:
PORTB &= ~(1<<1);

break;

case 9:

  PORTA &= ~(1<<0);
  

break;

case 10:

PORTA &= ~(1<<1);

break;

case 11:

PORTA &= ~(1<<2);

break;

case 12:

PORTA &= ~(1<<3);
break;

case 13:

PORTA &= ~(1<<4);

break;

case 14:

PORTA &= ~(1<<5);

break;

case 15:

PORTA &= ~(1<<6);

break;

case 16:

PORTA &= ~(1<<7);

break;





}

}
  
  }

main.c mc2

#include <avr/io.h>
#define numberOfButtons  6
#include <util/delay.h>
#include "UART_send.h"
#include "UART_read.h"
#include "matrix_register.h"
#include "button_register.h"



int  main(void){
int b = 1;

DDRB &= ~(1<<b) ;

PORTB |= 1 << b ;





DDRB = (1<<2) ;
DDRC = (1<<0) | (1<<1) | (1<<2) | (1<<3) | (1<<4) | (1<<5) | (1<<6);
DDRA = (1<<0) | (1<<1) | (1<<2) | (1<<3);
PORTA |= (1<<0) | (1<<1) | (1<<2);
PORTC |= (1<<0) | (1<<1) | (1<<2) | (1<<3) | (1<<4) | (1<<5) | (1<<6) ;
PORTB |= (1<<2);



DDRD |= (1<<7);


PORTD &= ~(1<<7);
PORTD |= (1<<7);
PORTB &= ~(1<<2);
PORTA &= ~(1 << 2);
PORTA &= ~(1 << 0);
PORTA &= ~(1 << 1);




  int UBBRValue = 25;

//Put the upper part of the baud number here (bits 8 to 11)
UBRR0H = (unsigned char) (UBBRValue >> 8);

//Put the remaining part of the baud number here
UBRR0L = (unsigned char) UBBRValue;

//Enable the receiver and transmitter
UCSR0B = (1 << RXEN0) | (1 << TXEN0);

//Set 2 stop bits and data bit length is 8-bit
UCSR0C = (1 << USBS0) | (3 << UCSZ00);







  
  int i = 0;  
  int col = 9;
  int uart = 0;
  int matrix_led = 0;



  while(1)
  {
  
      
    
      


switch(uart_read()){



case 0b00000000:
matrix_led = 1;

break;

case 0b10000000:
matrix_led = 2;

break;

case 0b11000000:
matrix_led = 3;

break;

case 0b11100000:
matrix_led = 4;

break;

case 0b11110000:
matrix_led = 5;

break;

case 0b11111000:
matrix_led = 6;

break;

case 0b11111100:
matrix_led = 7;

break;

case 0b11111110:
matrix_led = 8;

break;

case 0b11111111:
matrix_led = 9;

break;

case 0b00000011:
matrix_led = 10;

break;

case 0b00000111:
matrix_led = 11;

break;

case 0b00001111:
matrix_led = 12;

break;

case 1:
matrix_led = 13;

break;

case 2:
matrix_led = 14;

break;

case 34:
matrix_led = 15;

break;

case 4:
matrix_led = 16;

break;

case 5:
matrix_led = 17;

break;

case 6:
matrix_led = 18;

break;

case 20:
matrix_led = 19;

break;

case 8:
matrix_led = 20;

break;

case 9:
matrix_led = 21;

break;

case 10:
matrix_led = 22;

break;

case 11:
matrix_led = 23;

break;

case 12:
matrix_led = 24;

break;

case 13:
matrix_led = 25;

break;

case 14:
matrix_led = 26;

break;

case 33:
matrix_led = 27;

break;

case 16:
matrix_led = 28;

break;

case 17:
matrix_led = 29;

break;

case 18:
matrix_led = 30;

break;

case 19:
matrix_led = 31;

break;

case 35:
matrix_led = 32;

break;

case 21:
matrix_led = 33;

break;

case 22:
matrix_led = 34;

break;

case 23:
matrix_led = 35;

break;

case 24:
matrix_led = 36;

break;

case 25:
matrix_led = 37;

break;

case 26:
matrix_led = 38;

break;

case 27:
matrix_led = 39;

break;

case 28:
matrix_led = 40;

break;

case 29:
matrix_led = 41;

break;

case 30:
matrix_led = 42;

break;
case 31:
matrix_led = 43;

break;
case 32:
matrix_led = 44;

break;

}

switch(matrix_led){

case 1:
matrix(0,9);
uart_send(0);
break;

case 2:
matrix(0,10);
uart_send(0);
break;

case 3:
matrix(0,11);
uart_send(0);
break;


case 4:
matrix(0,12);
uart_send(0);
break;

case 5:
matrix(0,13);
uart_send(0);
break;

case 6:
matrix(0,14);
uart_send(0);
break;


case 7:
matrix(0,15);
uart_send(0);
break;

case 8:
matrix(0,16);
uart_send(0);
break;

case 9:
matrix(5,0);
uart_send(0);
break;

case 10:
matrix(6,0);
uart_send(0);
break;

case 11:
matrix(7,0);
uart_send(0);
break;

case 12:
matrix(8,0);
uart_send(0);
break;

case 13:
matrix(5,9);
uart_send(0);
break;
case 14:
matrix(5,10);
uart_send(0);
break;

case 15:
matrix(5,11);
uart_send(0);
break;
case 16:
matrix(5,12);
uart_send(0);
break;

case 17:
matrix(5,13);
uart_send(0);
break;
case 18:
matrix(5,14);
uart_send(0);
break;

case 19:
matrix(5,15);
uart_send(0);
break;
case 20:
matrix(5,16);
uart_send(0);
break;


case 21:
matrix(6,9);
uart_send(0);
break;
case 22:
matrix(6,10);
uart_send(0);
break;

case 23:
matrix(6,11);
uart_send(0);
break;
case 24:
matrix(6,12);
uart_send(0);
break;

case 25:
matrix(6,13);
uart_send(0);
break;
case 26:
matrix(6,14);
uart_send(0);
break;

case 27:
matrix(6,15);
uart_send(0);
break;
case 28:
matrix(6,16);
uart_send(0);
break;

case 29:
matrix(7,9);
uart_send(0);
break;
case 30:
matrix(7,10);
uart_send(0);
break;

case 31:
matrix(7,11);
uart_send(0);
break;
case 32:
matrix(7,12);
uart_send(0);
break;

case 33:
matrix(7,13);
uart_send(0);
break;
case 34:
matrix(7,14);
uart_send(0);
break;

case 35:
matrix(7,15);
uart_send(0);
break;
case 36:
matrix(7,16);
uart_send(0);
break;

case 37:
matrix(8,9);
uart_send(0);
break;
case 38:
matrix(8,10);
uart_send(0);
break;

case 39:
matrix(8,11);
uart_send(0);
break;
case 40:
matrix(8,12);
uart_send(0);
break;

case 41:
matrix(8,13);
uart_send(0);
break;
case 42:
matrix(8,14);
uart_send(0);
break;

case 43:
matrix(8,15);
uart_send(0);
break;
case 44:
matrix(8,16);
uart_send(0);
break;
}



}
}

matrix_register.h mc2

#include <avr/io.h>
void matrix(int col, int row){
float ms = 0.1;
switch(col){


case 5:
PORTB |= 1 << 2;
break;

case 6:
PORTA |= 1 << 2;
break;

case 7:
PORTA |= 1 << 0;
break;

case 8:
PORTA |= 1 << 1;
break;








}

switch(row){
case 9:
PORTD &= ~(1<<7);
_delay_ms(ms);
PORTD |= (1<<7);
break;

case 10:
PORTC &= ~(1<<PC0);
_delay_ms(ms);
PORTC |= (1<<PC0);
break;

case 11:
PORTC &= ~(1<<PC1);
_delay_ms(ms);
PORTC |= (1<<PC1);
break;

case 12:
PORTC &= ~(1<<PC2);
_delay_ms(ms);
PORTC |= (1<<PC2);
break;

case 13:
PORTC &= ~(1<<PC3);
_delay_ms(ms);
PORTC |= (1<<PC3);
break;

case 14:
PORTC &= ~(1<<PC4);
_delay_ms(ms);
PORTC |= (1<<PC4);
break;

case 15:
PORTC &= ~(1<<PC5);
_delay_ms(ms);
PORTC |= (1<<PC5);
break;

case 16:
PORTC &= ~(1<<PC6);
_delay_ms(ms);
PORTC |= (1<<PC6);
break;
}

switch(col){


case 5:
PORTB &= ~(1<<2);
break;

case 6:
PORTA &= ~(1 << 2);
break;

case 7:
PORTA &= ~(1 << 0);
break;

case 8:
PORTA &= ~(1 << 1);
break;



}


  
  }

[Bernd M. (bernd_m)]

Wie soll man darauf reagieren, wenn nicht sarkastisch?
Hut ab an alle die sich in ihrer Freizeit damit auseinander setzen.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Julian Bertsch schrieb:
> aber egal jetz, ich poste einfach mal den quellcode

Um Himmels Willen - viel Fleissarbeit, wenig System. Auch die Kommentare 
sind vorbildlich...
Aber mal im Ernst, was glaubst du, wie wir dir hier helfen können, du 
rotzt hier den Code hin (anstatt einen Anhang draus zu machen), 
unkommentiert und ohne Kommentar - hunderte von Zeilen und wir sollen 
uns jetzt da durchwühlen?

[Thomas E. (thomase)]

Julian Bertsch schrieb:
> 1. Kann man mich nicht eifach auf was fehlendes Hinweisen anstatt gleich
> sarkastisch reagiert aber egal jetz, ich poste einfach mal den quellcode

Hübscher Kot. Vor allen Dingen schön lang. Das Scrollrad von meiner Maus 
ist kurz vor einem Lagerschaden.

Wer soll dir jetzt sagen, wo der Fehler ist?

Weisst du was Funktionen sind?
Tu dir selbst den Gefallen und packe die Programmteile, die jeweils 
bestimmte Sachen machen, in einzelne Funktionen. Da steigt so doch keine 
Sau durch! Erzähl jetzt nicht, daß du da durchsteigst!

Wenn du damit fertig bist, grenzt du dein Problem soweit ein, daß du 
sagen kannst, in welcher dieser Funktionen der Fehler liegt. Und vor 
allen Dingen beschreibst du genau, was der Fehler ist. Dann kann man dir 
auch helfen. Und du kannst dir selber helfen, weil du dann nämlich 
einzelne Programmteile, die besagten Funktionen eben, testen kannst.

Dann rückst du deinen Code noch schön ein und geizt ein wenig mit den 
Leerzeilen.

Und wenn du damit fertig, kommst du nochmal wieder und dir wird auch 
geholfen.

mfg.

und Frohe Ostern

[Max H. (hartl192)]

Am leichtesten würdest du uns die Arbeit machen, wenn du den Teil der 
die LEDs ansteuert postest bzw. markierst. Und dazu den Schaltplan dazu 
postest. Es ist unmöglich, den Fehler zu finden, wenn man nicht weiß an 
welchen Pins die LEDs hängen.