Bitmanipulation

[P. F. (pfuhsy)]

Hallo zusammen,

ich benutze zum RFM12 konfiguieren, als Beispiel, folgende Code-Zeile:

RFM_cmd_empf(0x8280);  //Power Management

Die dazugehörige Funktion sieht so aus:

unsigned int RFM_cmd_empf(uint16_t aCmd)//Software-Lösung für MOSI/MISO als Empänger
{
  unsigned char i;
  unsigned int temp;
  
  temp=0;
  Low_Sck();
  Low_Sel();
  
  for(i=0;i<16;i++)
  {
    if(aCmd&0x8000)
    {
      Hi_Sdi();
    }
    else
    {
      Low_Sdi();
    }
    
    Hi_Sck();
    temp<<=1;
    
    if(Sdo_ist_Hi())
    {
      temp|=0x0001;
    }
    
    Low_Sck();
    aCmd<<=1;
  };
  
  Hi_Sel();
  return(temp);
}

Soweit so gut.
Jetzt hab ich mir gedacht, da die o.g. Code-Zeile direkt das Ganze Byte 
anspricht, wäre es doch kleverer nur das Bit zu minpulieren, welches 
gerade auch wirklich notwenig ist. Also hab ich in der Header und 
C-Datei des RFM12 folgendes eingebaut.

rfm12.h:

//(Power Management 82xx) Einzelbit-Konfiguration / -Abfrage
extern uint16_t PWRM;  //Adressierbereich, in *.c deklaiert
#define ER      7  //enable receiver
#define EBB      6  //enable baseband
#define ET      5  //enable transmitter
#define ES      4  //enable synthesizer
#define EX      3  //enable xtal(Quarz)
#define EB      2  //enable brownout
#define EW      1  //enable wakeup
#define DC      0  //disable clock

rfm12.c:

//--------------------------Initialisierung-----------------------------
uint16_t PWRM     = 0x8200;    //Adresszuweisung (Power Management 82xx)

void RFM_Empf_Ein()          //Empfänger einschalten
{
  RFM_cmd_empf(PWRM |=  (1<<ER));  //Empfänger einschalten
  _delay_ms(1);          //Warten bis Empfänger an ist
  RFM_Fifo_Reset();        //Fifo reseten
}

Ersetzte ich jetzt die o.g. Zeile:

RFM_cmd_empf(0x8280);

durch den Funktionsaufruf:

RFM_Empf_Ein();

...funkioniert der ganze spass nicht mehr.
Wo ist mein Denkfehler, in der Simluation sieht das alles ganz gut aus ? 
Vorher "8280" und danach "8200" wie es sein soll.

[Tappo (Gast)]

Peter F. schrieb:
> RFM_cmd_empf(PWRM |=  (1<<ER));

lass das | weg

RFM_cmd_empf(PWRM |  (1<<ER));

[Tappo (Gast)]

natürlci = weg, sorry

[P. F. (pfuhsy)]

Tappo schrieb:
> natürlci = weg, sorry

Warum denn ?
Bei der Bitmanipulation der Ports funktioniert das doch auch so.
Z.B.:

DDRB |= (1<<PB1);

[Sascha W. (sascha-w)]

So wie du das geschieben hast wird die Funktion mit PWRM als Parameter 
aufgerufen, und anschließend das entsprechende Bit in PWRM gesetzt.
Vergleiche:
foo(var++)
ruft die Funktion mit var auf und erhöht selbige im Anschluss.

Sascha

[P. F. (pfuhsy)]

Ich verstehe das nicht so recht. PWRM ist doch keine Funktion. PWRM ist 
eien Variable 16 Bits, ich versuche beispielsweise NUR Bit 7 zu 
verändern.

[Sascha W. (sascha-w)]

ja, aber mit dem = weist du der Variable PWRM einen neuen Wert zu, das 
passiert aber erst nachdem deine Funktion RFM_cmd_empf zuvor mit dem 
alten Wert von PWRM aufgerufen wurde.

Sascha

[P. F. (pfuhsy)]

Sascha W. schrieb:
> ja, aber mit dem = weist du der Variable PWRM einen neuen Wert zu, das
> passiert aber erst nachdem deine Funktion RFM_cmd_empf zuvor mit dem
> alten Wert von PWRM aufgerufen wurde.
>
> Sascha

Ich kann das nciht so ganz nachvollziehen.
In der Simulation übernimmt er nur dann den Wert, wenn die Schreibweise 
die meinige ist. Er addiert genau den Bit dazu den ich manipulieren 
will. Steht dort vorher 8200, wird 8280 draus, Bild 1. Steht dort vorher 
8201, wird 8281 draus. Und den Wert übernimmt er beim Aufruf der 
Funktion auch korrekt, Bild2.

Wenn ich das "=" rauslasse passiert gar nichts, Bild3.

[vorher (Gast)]

Sascha W. schrieb:
> passiert aber erst nachdem deine Funktion RFM_cmd_empf zuvor mit dem
> alten Wert von PWRM aufgerufen wurde.

Nein. Das passiert vor dem Funktionsaufruf.

[Walter S. (avatar)]

warum nicht so:
PWRM |=  (1<<ER);
RFM_cmd_empf(PWRM);  //Empfänger einschalten

wenn du PWRM verändern willst oder wie empfohlen:
RFM_cmd_empf(PWRM |  (1<<ER));

[Tappo (Gast)]

Dann schreibe einfach so und gut jetzt.

//--------------------------Initialisierung-----------------------------
uint16_t PWRM     = 0x8200;    //Adresszuweisung (Power Management 82xx)

void RFM_Empf_Ein()          //Empfänger einschalten
{
  PWRM |=  (1<<ER);
  RFM_cmd_empf(PWRM);  //Empfänger einschalten
  _delay_ms(1);          //Warten bis Empfänger an ist
  RFM_Fifo_Reset();        //Fifo reseten
}

[Walter S. (avatar)]

Peter F. schrieb:
> Wenn ich das "=" rauslasse passiert gar nichts, Bild3.

steppe mal in die Funktion, dann  wirst du sehen dass der Parameter 
richtig ist

[P. F. (pfuhsy)]

Walter S. schrieb:
> Peter F. schrieb:
>> Wenn ich das "=" rauslasse passiert gar nichts, Bild3.
>
> steppe mal in die Funktion, dann  wirst du sehen dass der Parameter
> richtig ist

Stimmt.

Tappo schrieb:
> void RFM_Empf_Ein()          //Empfänger einschalten
> {
>   PWRM |=  (1<<ER);
>   RFM_cmd_empf(PWRM);  //Empfänger einschalten
>   _delay_ms(1);          //Warten bis Empfänger an ist
>   RFM_Fifo_Reset();        //Fifo reseten
> }

Diese Variante auch funktioniert nicht.