C - Anfängerfrage

[Sonya W. (Gast)]

Einen wunderschönen guten Abend,

uint8_t crc8 ( uint8_t inCrc , uint8_t inData )      
{
  uint8_t   i;
  uint8_t   data;

  data = inCrc ^ inData;

  for ( i = 0; i < 8; i++ )
  {
    if (( data & 0x80 ) != 0 )
    {
      data <<= 1;
      data ^= 0x07;
    }
    else
    {
      data <<= 1;
    }
  }
  return data;
}


void buildCmd( uint8_t addr , uint32_t cmd , bool crcEn )    
{
  uint8_t frameLow  []  = { 0xFF , addr    , ( cmd & 0x000000FF ) , ( ( cmd & 0x0000FF00) >> 8 ) , 0x00 };
  uint8_t frameHigh  []  = { 0xFF , addr + 1 , ( ( cmd & 0x00FF0000 ) >> 16 ) , ( ( cmd & 0xFF000000) >> 24 ) , 0x00 };
  
  uint8_t x = 0;
  uint8_t result = 0;
  
  for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
  {
    result = crc8( frameLow[4] , frameLow[x] );
  }
  frameLow[4] = result;
  cmdOut( frameLow , ( 4 + crcEn ) );
  
  
  for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
  {
    result = crc8( frameHigh[4] , frameHigh[x] );
  }
  frameHigh[4] = result;
  cmdOut( frameHigh , ( 4 + crcEn ) );
}

Wieso klappt die Übergabe hier nicht? Kann es was damit zu tun haben das 
es das gleiche Array ist? Bekomme jedesmal "0" raus beim CRC

result = crc8( frameLow[4] , frameLow[x] );

Danke schon mal für die nette Erklärung (:

[Jim M. (turboj)]

Was tut denn die Anweisung

data <<= 1;

Deiner Meinung nach?

Das ist ein Linksshift, nach 8x shiften ist ein Byte leer...

[Sonya W. (Gast)]

Jim M. schrieb:
> Was tut denn die Anweisung
> data <<= 1;
>
> Deiner Meinung nach?
>
> Das ist ein Linksshift, nach 8x shiften ist ein Byte leer...

Das hat doch nichts mit der Übergabe zu tun?

[Felix (Gast)]

Du hast zweimal dieses Konstrukt:

for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
  {
    result = crc8( frameLow[4] , frameLow[x] );
  }

result ist kein Array, daher scheint diese Schleife völlig sinnlos.
Steht die Funktion crc8 noch im .lss-File oder wurde die wegoptimiert?

[Sonya W. (Gast)]

Felix schrieb:
> Du hast zweimal dieses Konstrukt:
>
> for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
>   {
>     result = crc8( frameLow[4] , frameLow[x] );
>   }
>
> result ist kein Array, daher scheint diese Schleife völlig sinnlos.
> Steht die Funktion crc8 noch im .lss-File oder wurde die wegoptimiert?

Das passt schon alles.
Wenn ich jetzt anstatt den oben erwähten Code nehmen..

> for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
>   {
>     frameLow[4] = crc8( frameLow[4] , frameLow[x] );
>   }

Kommt immer nur "0" raus.

[Frank M. (ukw) (Moderator)]

Sonya W. schrieb:
> Wieso klappt die Übergabe hier nicht? Kann es was damit zu tun haben das
> es das gleiche Array ist? Bekomme jedesmal "0" raus beim CRC

Ich nicht.

Damit ich das Programm überhaupt übersetzen konnte, habe ich die Aufrufe 
von cmdOut() auskommentiert, da diese Funktion von Dir nicht angegeben 
wurde. Anschließend habe ich unter jedem Aufruf von crc8() per printf() 
den Wert von result ausgeben lassen.

Ausserdem noch folgende includes hinzugefügt:

#include <stdio.h>
#include <stdint.h>
#include <stdbool.h>

Dann noch main() hinzugefügt:

int main ()
{
    buildCmd (10, 330, 1);
    return 0;
}

Nach Ausführen wird das folgende ausgegeben:

243
54
241
7
0
243
49
0
0
0

Von "jedesmal 0" kann also keine Rede sein.

Bitte gib ein vollständig kompilierbares Programm an, mit dem man das 
nachvollziehen kann. Bruchstücke helfen hier offenbar nicht.

[Sonya W. (Gast)]

> Nach Ausführen wird das folgende ausgegeben:243
> 54
> 241
> 7
> 0
> 243
> 49
> 0
> 0
> 0
>
> Von "jedesmal 0" kann also keine Rede sein.

Mein Fehler.
framex[4], dort sollte die CRC drin stehen. Das tut sie aber nicht wie 
man sieht..

[void (Gast)]

Sonya W. schrieb:
> Kommt immer nur "0" raus.

Die crc8 Funktion erwartet als Eingangs-Parameter inCrc, das Ergebnis 
der crc8 von der vorhergehenden Stelle. Sonst rechnest du jedes mal von 
vorne...

  result = frameLow[4];
  for ( x = 0 ; x < ( 4 + crcEn ) ; x++ )
  {
    result = crc8( result , frameLow[x] );
  }

[MaWin (Gast)]

Sonya W. schrieb:
> Bekomme jedesmal "0" raus beim CRC

Natürlich.
Die crc8 Funktion will ihr letztes Ergebnis erneut verwursten, also

result=crc8(result,frameLow[x]);

Nur beim ersten Aufruf wird frameLow[4] verwendet.

[Stromverdichter (Gast)]

Hallo Sonya,

da hast du ja einen super kommentierten Code gebastelt. Da erkennt man 
sofort, was er machen soll.

Sonya W. schrieb:
> Kommt immer nur "0" raus.

Vielleicht schiebst du ja immer 0 rein?

Was steht den vor Ausführung genau in framelow[]?
Hast du keinen Debugger?
Geh doch die Ausführung schrittweise durch.