Ratlosigkeit obwohl nur ein Bit getestet wird

[Michael D. (etzen_michi)]

Irgendwie komme ich nichtmehr weiter ...
Es geht um ein Unterprogramm, welches einen Wert übergeben bekommt und 
die Bits 11 - 0 ausließt, je nachdem ob 0 oder 1 einen PortPin setzt und 
dann einmal SCLK toogled.


Wenn ich

PORTD|= 1<<SIN;
for(int i=0; i<=11; i++) {
   tSCLK();
}

mache geht alles.

Wenn ich

//PORTD|= 1<<SIN;
int i;
for(i=0; i<=11; i++) {
   if((GS&0x800)) {
      PORTD|= 1<<SIN;
   } else {
      //PORTD&= ~(1<<SIN);
      PORTD|= 1<<SIN;
   }
   tSCLK();
   GS<= 1;
   //GS*= 2;
}

sollte eig. nichts anderes passieren können, aber ich bekomme nur Mist 
raus (der SIN wird zeitweise auf 0 geschaltet.

Aus dem erstellten Code werde ich auch nicht ganz schlau weil da 
irgendwie ganze Teile fehlen.
Das Programm läuft weder mit wie auch ohne Optimierung.

0000012c <GSData>:

void GSData(volatile int GS) {
 12c:  df 93         push  r29
 12e:  cf 93         push  r28
 130:  00 d0         rcall  .+0        ; 0x132 <GSData+0x6>
 132:  cd b7         in  r28, 0x3d  ; 61
 134:  de b7         in  r29, 0x3e  ; 62
 136:  9a 83         std  Y+2, r25  ; 0x02
 138:  89 83         std  Y+1, r24  ; 0x01
 13a:  20 e0         ldi  r18, 0x00  ; 0
 13c:  30 e0         ldi  r19, 0x00  ; 0
          }
     }
     */
     int i;
     for(i=0; i<=11; i++) {
          if((GS&0x800)) {
 13e:  89 81         ldd  r24, Y+1  ; 0x01
 140:  9a 81         ldd  r25, Y+2  ; 0x02
 142:  93 ff         sbrs  r25, 3
 144:  02 c0         rjmp  .+4        ; 0x14a <GSData+0x1e>
               PORTD|= 1<<SIN;
 146:  91 9a         sbi  0x12, 1  ; 18
 148:  01 c0         rjmp  .+2        ; 0x14c <GSData+0x20>
          } else {
               PORTD&= ~(1<<SIN);
 14a:  91 98         cbi  0x12, 1  ; 18
     }
     tXLAT();
}

Vielleicht sieht hier ja irgendwer direkt irgendeinen groben Fehler... 
Google habe ich bemüht und die Forensuche auch.

*Info: Bei GS<= 1; meldet er das der berechnete Wert nicht verwendet 
wird, obwohl er doch beim nächsten Schleifendurchlauf wieder verglichen 
werden sollte.
volatile bringt keine besserungen.
Desweiteren verstehe ich auch nicht wie er auf die Idee kommt obwohl 
nirgends im Code steht er solle den Pin auf Low setzen dieser auf Low 
gesetzt wird.

Die Info das der Teil nicht funktioniert habe ich nicht nachgemessen, 
sehe ich auf der angeschlossenen Hardware da nicht alle LED`s 
gleichmäßig aufleuchten, sondern teilweise gedimmt sind (Wert < 4095).

[Noname (Gast)]

>Bei GS<= 1; meldet er das der berechnete Wert nicht verwendet
>wird, obwohl er doch beim nächsten Schleifendurchlauf wieder verglichen
>werden sollte.

Kein Wunder. Was soll  GS<= 1; - ein Vergleich ohne Entscheidung oder 
Zuweisung - auch bewirken?

Das
   GS<= 1;

ist nicht das selbe wie das

   //GS*= 2;


Kann es sein, das es

GS <<= 1

heissen sollte?

[Stefan E. (sternst)]

Michael D. schrieb:
> Vielleicht sieht hier ja irgendwer direkt irgendeinen groben Fehler...

GS<= 1;

->

GS <<= 1;

[Michael D. (etzen_michi)]

Mit

GS<<= 1;

das gleiche Problem

0000012c <GSData>:

void GSData(volatile int GS) {
 12c:  df 93         push  r29
 12e:  cf 93         push  r28
 130:  00 d0         rcall  .+0        ; 0x132 <GSData+0x6>
 132:  cd b7         in  r28, 0x3d  ; 61
 134:  de b7         in  r29, 0x3e  ; 62
 136:  9a 83         std  Y+2, r25  ; 0x02
 138:  89 83         std  Y+1, r24  ; 0x01
          for(j=0; j<=11; j++) {
               tSCLK();
          }
     }
     */
     PORTD|= 1<<SIN;
 13a:  91 9a         sbi  0x12, 1  ; 18
 13c:  20 e0         ldi  r18, 0x00  ; 0
 13e:  30 e0         ldi  r19, 0x00  ; 0
     int i;
     for(i=0; i<=11; i++) {
          if((GS&0x800)) {
 140:  89 81         ldd  r24, Y+1  ; 0x01
 142:  9a 81         ldd  r25, Y+2  ; 0x02
               PORTD|= 1<<SIN;
          } else {
               //PORTD&= ~(1<<SIN);
               PORTD|= 1<<SIN;
 144:  91 9a         sbi  0x12, 1  ; 18
     }
     tXLAT();
}

Die ganzen *Alternativen, Extras welche als Kommentar vorhanden sind 
habe ich bereits alle schon ausgetestet ..

Aber zumindest eine Frage ist nun beantwortet, bleibt noch: Warum setzt 
der den Pin Low?

[Noname (Gast)]

Hm. Es scheint mir das der Code genau das macht, was Du schreibst.

Du rufst doch GSData mit einem konstanten Argument auf. Da kann sich der 
Compiler an fünf Fingern abzählen, welches Bit er setzt soll. Und die 
Anzahl der Schleifendurchläufe ändert daran auch nichts.

[Michael D. (etzen_michi)]

Ha ... Fehler gefunden ...

Seid zwei Tagen am Suchen und rumwursteln, nun gehts.

In der .c steht nicht:

for(i=0; i<=11; i++)..

sondern

for(i=0; i<= 47; i++) {
   for(j=0; j<= 11; j++) ...

Habe das "for48" anfangs drinne gehabt um die zusammensetzung des Codes 
zu testen, dann aber vergessen raus zu nehmen.

Problem war also das er nach jeder LED (12Bit) tXLAT(); ausgeführt hat, 
was dazu geführt hat das die TLC die Daten im Schieberegister 
durcheinander gewürfelt haben.

Habe nun das überflüssige tXLAT(); herausgenommen und es läuft.

Frage mich warum man die Lösung erst findet wenn man sich öffentlich 
blamiert hat.


Nochmals vielen Dank.

[R. F. (rfr)]

Michael D. schrieb:
> Frage mich warum man die Lösung erst findet wenn man sich öffentlich
>
> blamiert hat.

Wegen des gesteigerten Lerneffektes.


:-))


Gruss

Robert