Maskierungsprobleme

[Josh (Gast)]

Hallo miteinander

Ich lese momentan mit einem Mikrokontroller (ATM644) Einen 8X Dipswitch 
aus.
Ich möchte in den Variabeln GainSen1-4 nur immer 2 Bit von dem Switch 
haben. Dafür habe ich dies nach rechts geschoben und mit 0x03 verundet. 
Nachdem noch mit 0x10 verodert, da ich immer dieses eine Bit noch dazu 
benötige. Somit sollten keine anderen Schalter einen Einfluss auf die 
Einzelnen Gain's besitzen. Jedoch wenn ich z.B. nur Bit 6 und 1 
einschalte am Switch, wird mir schon ein Fehler ausgegben. So besitzt 
einer der Variabeln GainSen1-4 0x13. Jedoch sollte dies nie passieren 
und ist für mich unlogisch. Sieht jemand den Fehler, wo dies auslösen 
könnte. Ich Versuchte schon das Ganze konstrukt anderst zu schreiben 
half aber jedoch nicht.

#include <avr/io.h>

int main(void)
{

  unsigned char GainSen1 = 0;
  unsigned char GainSen2 = 0;
  unsigned char GainSen3 = 0;
  unsigned char GainSen4 = 0;

  
    while(1)
    {

        //8x Dipswitch auslesen
        GainSen1 = ((PIND >> 6)&0x03)|0x10;    //Nur Bit 6 und 7
        GainSen2 = ((PIND >> 4)&0x03)|0x10;    //Nur Bit 4 und 5
        GainSen3 = ((PIND >> 2)&0x03)|0x10;    //Nur Bit 2 und 3
        GainSen4 = (PIND & 0x03)|0x10;      //Nur Bit 0 und 1
    
        //Auf ("0x03" + 0x10) überprüfen Verboten laut Datenblatt
        if ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13) 
        {
          PORTA = PORTA & ~0x80;
        }
    }
}

[Rufus Τ. F. (rufus)]

Josh schrieb:
> Jedoch wenn ich z.B. nur Bit 6 und 1 einschalte am Switch, wird mir schon
> ein Fehler ausgegben.

Wer gibt Dir einen Fehler aus?

[Josh (Gast)]

Der Fehler wird ausgegeben an der LED auf Port A.

[Nils P. (torus)]

Dein Code sieht gut aus. Schuss ins Blaue: vielleicht ist bei dir PIND 
als Signed deklariert. Dann wäre der Fehler logisch:

Probier mal das hier:

        //8x Dipswitch auslesen
        unsigned char p = PIND;
        GainSen1 = ((p >> 6)&0x03)|0x10;    //Nur Bit 6 und 7
        GainSen2 = ((p >> 4)&0x03)|0x10;    //Nur Bit 4 und 5
        GainSen3 = ((p >> 2)&0x03)|0x10;    //Nur Bit 2 und 3
        GainSen4 = (p & 0x03)|0x10;      //Nur Bit 0 und 1

[Mario M. (thelonging)]

Josh schrieb:
> if ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13)

Das macht nicht das, was Du willst.

[Sebastian R. (sebastian_r569)]

Mario M. schrieb:
> Josh schrieb:
>> if ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13)
>
> Das macht nicht das, was Du willst.


if ((GainSen1 | GainSen2 << 1 | GainSen3 << 2 | GainSen4 << 3) == 0x13)

Oder so ähnlich.

[Theor (Gast)]

Naja. Leider bedeutet der Code nicht das, was Du verbal beschreibst.


Das hier:

if ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13)

bedeutet nicht, dass die Bedingung war wird, falls einer der 
GainSenX-Werte == 0x13 ist.

Vielmehr bedeutet es, dass die Bedingung wahr wird, falls _die 
Bitweise-Oder Verknüpfung aller_ GenSenX Werte == 0x13 ist.

Dem Fehlerbeispiel entsprechend ergibt das Folgendes:

Seien Bit 6 und Bit 1 des Schalter gesetzt.


        GainSen1 = ((PIND >> 6)&0x03)|0x10;
        GainSen4 = (PIND & 0x03)|0x10;

mit Eingabe ergibt sich:

        GainSen1 = ((0x82 >> 6)&0x03)|0x10;
        GainSen4 = (0x82 & 0x03)|0x10;

nach dem schieben:

        GainSen1 = (0x02 & 0x03)|0x10;
        GainSen4 = (0x81 & 0x03)|0x10;

nach der Und-Verknüpfung

        GainSen1 = (0x02 |0x10;
        GainSen4 = (0x01)|0x10;

nach der Oder Verknüpfung

        GainSen1 = 0x12;
        GainSen4 = 0x11;

In der IF-Bedingung werden diese Werte nochmal Oder-Verknüpft.

        GainSen1 | ... | GainSen4;

        0x12 | ... | 0x11;

was eben 0x13 ergibt.

Erst danach vergleichst Du ja mit 0x13.

[Josh (Gast)]

Habe ich gleich ausprobiert. Funktioniert Leider nicht.
Folgende Bitmuster habe ich ausprobiert und gehen nicht:

10010000
10000100
10000001
01100000
01001000
01000010

Es besitzt auch eine Regelmäsigkeit somit muss ein fehler forliegen.

[Felix (Gast)]

Gensens1: 0x02 | 0x10 ist 0x12.
Gensens4: 0x01 | 0x10 ist 0x11.

Dann bei deiner If-Prüfung 0x12 | 0x11 ergibt 0x13.

[Tassilo H. (tassilo_h)]

Deine Beschreibung klingt, als meintest du in der if-Bedingung:

//Auf ("0x03" + 0x10) überprüfen Verboten laut Datenblatt
if ((GainSen1 == 0x13) || (GainSen2 == 0x13) || (GainSen3 == 0x13) || (GainSen4 == 0x13)) {

(true wenn einer der GainSen-Werte 0x13 ist)

Deine Bit-Veroderung ist etwas anderes: Wenn Bit 1 und 6 an sind, dann 
ist GainSen1 = 0x12, GainSen2= GainSen3 = 0x10 und GainSen4 = 0x11.
0x12 | 0x11 = 0x13

[Josh (Gast)]

Dan wird er leider nie in das IF reinkommen auch wen 2 Bit 11 besitzen 
und 0x13 währen

[Theor (Gast)]

Au Mist. Da hat sich ein Fehler eingeschlichen:

nach dem schieben:

        GainSen1 = (0x02 & 0x03)|0x10;
        GainSen4 = (0x82 & 0x03)|0x10;

nach der Und-Verknüpfung

        GainSen1 = (0x02 |0x10;
        GainSen4 = (0x02)|0x10;

nach der Oder Verknüpfung

        GainSen1 = 0x12;
        GainSen4 = 0x12;

In der IF-Bedingung werden diese Werte nochmal Oder-Verknüpft.

        GainSen1 | ... | GainSen4;

        0x12 | ... | 0x12;

was eben 0x12 ergibt.


Dann tritt der Fehler so nicht ein. Sorry.

[Stefan K. (stefan64)]

Du möchtest das:

if ( (GainSen1 == 0x13) || 
     (GainSen2 == 0x13) || 
     (GainSen3 == 0x13) || 
     (GainSen4 == 0x13)
   )

Gruß, Stefan

[Theor (Gast)]

D.h. Dein Beispiel

> Jedoch wenn ich z.B. nur Bit 6 und 1 einschalte am Switch

dürfte den Fehler gar nicht erzeugen. Der Code gibt das nicht her.

[Josh (Gast)]

Stefan K. schrieb:
> Du möchtest das:if ( (GainSen1 == 0x13) ||
>      (GainSen2 == 0x13) ||
>      (GainSen3 == 0x13) ||
>      (GainSen4 == 0x13)
>    )
> Gruß, Stefan

Mit dem Funktionier es. Aber wiso nicht, wenn ich über das gesammte nach 
0x13 abfrage?

[Theor (Gast)]

Au Mann. Tut mir leid. Ist heute nicht mein Tag für Bitschubsereien.

Bit 6 gesetzt und Bit 1 ergibt ja nicht 0x82 sondern 0x42.

Ich halte mich jetzt raus.

[oder (Gast)]

Josh schrieb:
> So besitzt einer der Variabeln GainSen1-4 0x13.
das wäre
(GainSen1== 0x13 || GainSen2 == 0x13|| GainSen3== 0x13 || GainSen4== 
0x13) == 0x13)
>         if ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13)
bit0 bit1 bit7 ist in mind.1 einer Variable gesetzt.

[Stefan K. (stefan64)]

Josh schrieb:
> Mit dem Funktionier es. Aber wiso nicht, wenn ich über das gesammte nach
> 0x13 abfrage?

Du verwechselst | und ||.
| liefert eine bitweise VerODERung, || ist ein logisches ODER.
Mit ((GainSen1 | GainSen2 | GainSen3 | GainSen4) == 0x13) veroderst Du 
zuerst alle Bits von GainSen1..4 und vergleichst dann das Ergebnis mit 
0x13.

Du willst aber wissen, ob mindestens eine der 4 GainSen Variablen beide 
Bits 0 bzw. 1 gesetzt haben, also 0x13 als Wert haben. Und dazu musst Du 
eben jede der 4 Variablen mit 0x13 verghleichen und das jeweilige 
TRU/FALSE Ergebnis miteinander logisch verODERn.

Gruß, Stefan

[Josh (Gast)]

Danke viel mals für die Aufklärung.