Schreibfunktion funktioniert nicht

[Markus (Gast)]

Hallo

Bei mir funktioniert leider die HAL Funktion nicht, deshalb bin ich 
dabei die Funktion liesDaten zu schreiben.

Ich habe sie soweit fertig und versuche damit die angekommenen Daten 
RDLR_ und RDHR_ Byteweise in Data[0-8] zu schreiben.
Aber irgendwie schreibt er mir nur die Data[0] und Data[4], welche 
zufällig so richtig sind.

Deshalb würde ich gerne wissen, was ich da nicht richtig mache.
Unten in der for versuche ich solange i kleiner 3 ist zu verunden und 
schiebe immer 8 bits nach links, um Data[0-3] zu erhalten.
Wenn msk = 0 ist stelle ich auf den Anfanswert zurück und beginne von 
neu, diesmal aber mit dem HB

ausführung im main (einmal)

main:
CAN_getData(&rxMessage.Data[0]);

can.c
CanTxMsgTypeDef txMessage;
CanRxMsgTypeDef rxMessage;

can.h
extern CanTxMsgTypeDef txMessage;
extern CanRxMsgTypeDef rxMessage;
void CAN_getData(uint8_t* msg);

im can.c
function:
void liesDaten(uint8_t* msg)  
{
  /* check if a message is filtered and received by FIFO 0 */
  if ((CAN->RF0R & CAN_RF0R_FMP0)!=0)
  {
   uint32_t LB = 0;
    uint32_t HB= 0;
    
    LB = CAN->sFIFOMailBox[0].RDLR;                     // read data low bits register
    HB= CAN->sFIFOMailBox[0].RDHR;                     // read data high bits register
    releaseFIFOx(FIFO_0);                                  // FIFO must be released   
  
    uint32_t msk = 0xFF;
    
    for(uint8_t i = 0; i<8; i++)
    {
      if(i<=3)
      {
        msg[i] = LB& msk;
        msk <<= 8;
      }else
       {
         msg[i] = HB & msk;
         msk <<= 8;
       }
       
       if(msk == 0)
       msk = 0xFF;
    }
  }
  
}

Ich wäre um Ratschläge sehr dankbar

[Jim M. (turboj)]

Das mit den Bitschiebereien üben wir aber nochmal.

Hint: Was passiert bei

msg[i] = LB& msk;

wenn msk = 0xFF00 ist (i==1)?

[Markus (Gast)]

Jim M. schrieb:
> Das mit den Bitschiebereien üben wir aber nochmal.
>
> Hint: Was passiert bei
> msg[i] = LB& msk;
>
> wenn msk = 0xFF00 ist (i==1)?

Ja

wenn LB zB 0x02020202 ist
i = 0;

start: i=0
msg[0] =
02020202 LB
000000FF msk
ergibt 02
msk = 0000FF00

i=1
msg[1] =
02020202 LB
0000FF00 msk
ergibt 02

ich sehe leider nicht wo das Problem ist

[Nico W. (nico_w)]

0x0202 & 0xff00 == 0x0200

[Markus (Gast)]

Nico W. schrieb:
> 0x0202 & 0xff00 == 0x0200


Autssch :(

ok, hier die Verbesserung:

void liesDaten(uint8_t* msg)  
{
  /* check if a message is filtered and received by FIFO 0 */
  if ((CAN->RF0R & CAN_RF0R_FMP0)!=0)
  {
   uint32_t LB = 0;
    uint32_t HB= 0;
    
    LB = CAN->sFIFOMailBox[0].RDLR;                     // read data low bits register
    HB= CAN->sFIFOMailBox[0].RDHR;                     // read data high bits register
    releaseFIFOx(FIFO_0);                                  // FIFO must be released   
  
    uint32_t msk = 0xFF;
    uint8_t x = 0;
    
    for(uint8_t i = 0; i<8; i++)
    {
      if(i<=3)
      {
        msg[i] = (LB& msk) >> x;
        msk <<= 8;
        x += 8;
      }else
       {
         msg[i] = (HB & msk) >> x;
         msk <<= 8;
         x += 8;
       }
       
       if(msk == 0)
{
       msk = 0xFF;
       x=0;
}
    }
  }
  
}

[Markus (Gast)]

Und ja vielen Dank übrigens.
Hätte ich doch glatt vergessen.