Port ansteuern / Bitmaske über Daten (C18)

[Richard K. (richardk)]

Hallo zusammen,

Folgende code-Zeilen machen nicht das wass sie sollen:

void sendToDataPort(unsigned char DATA){
  sLCD_D0 = DATA &(1 << 0);
  sLCD_D1 = DATA &(1 << 1);
  sLCD_D2 = DATA &(1 << 2);
  sLCD_D3 = DATA &(1 << 3);
  return;
}

Hinter sLCD_Dx steht:

#define sLCD_D0 LATBbits.LATB0
#define sLCD_D1 LATBbits.LATB3
#define sLCD_D2 LATBbits.LATB4
#define sLCD_D3 LATBbits.LATB5

Aufgerufen wird diese Funktion z.B. mit:

sendToDataPort(0x03);

Die Erste Ausgabe an sLCD_D0,
funktioniert in Abhängigkeit von DATA. Alle andern nicht.
Im Simmulator landen die Bit's ebenfalls nicht im LATB Register.

Getestet habe ich bereits alle Einzelkomponenten des Ausdrucks:

sLCD_D1 = 1;  // <- Funktioniert

unsigned char test;
test = DATA &(1 << 1);  // <- Funktioniert ebenfalls

sLCD_D1 = DATA &(1 << 1);  // <- Funktioniert nicht

Seht Ihr den Fehler, ich bin ratlos.

Danke,
Gruss Richard

[hmmm (Gast)]

PIC kenne ich nicht.

Variablennamen in Großbuchstaben sind keine gute Idee (ist normalerweise 
für Makros reserviert).

Frage: Ein Byte hat 8 Bit, LATBbits.LATBx kann aber offensichtlich nur 1 
Bit speichern. Welches wird da genommen?

Ich tippe mal auf LSB, von daher:

void sendToDataPort(unsigned char Data){
  sLCD_D0 = (Data >> 0 ) & 1;
  sLCD_D1 = (Data >> 1 ) & 1;
  sLCD_D2 = (Data >> 2 ) & 1;
  sLCD_D3 = (Data >> 3 ) & 1;
  /* explizites return unnötig */
}

oder ähnlich.

[hmmm (Gast)]

Das & 1 kann man sich wahrscheinlich auch sparen, probiers aus.

[Richard K. (richardk)]

Hallo hmmm (Gast),

Tja...  In der Zwischenzeit war ich einkaufen und
bin selbst auf die Lösung / Ursächlichkeit des Problems gekommen.

Mein Lösungsansatz:

void sendToDataPort(unsigned char DATA){
  sLCD_D0 = DATA &(1 << 0);
  sLCD_D1 = (DATA &(1 << 1)) >> 1;
  sLCD_D2 = (DATA &(1 << 2)) >> 2;
  sLCD_D3 = (DATA &(1 << 3)) >> 3;
  return;
}

So, dein Lösungsansatz, ink. dem & 1; was ich mir bei genau diesem
Procedere wirklich sparen kann, gefällt mir deutlich besser!

Danke,
Gruss Richard