Berechnungen in C fehlerhaft

[H. D. (lactobacillus)]

Hallo,
ich versuche ein BMP180 an einem MSP430 mit CCS6 zum Laufen zu bekommen. 
Leider sind die Ergebnisse der Berechnungen völliger Quatsch:

B5 ist definiert als long und abhängig von der Temperatur zur Zeit 3446.

long B6;
B6 = B5-4000;

B6 = -272302634

[Test (Gast)]

Womit gestestet ? Womit und wie ausgegeben ? Womit kompiliert ? Womit 
den Wert von b5 geprüft ?

Wie in 99% der fälle wird es anwendungsfehler sein...mit mehr details 
lässt der sich auch finden ;)

[??? (Gast)]

Was hat B5 für einen Datentyp ?

[R. F. (rfr)]

Eine Prise meer source, bitte!

Danke

Robert

[Thomas (Gast)]

du musst dein B6 als signed deklarieren und ggf. je nach compiler vorher 
b5 auch auf signed umcasten, wenns nicht automatisch passiert

[Rufus Τ. F. (rufus)]

Thomas schrieb:
> du musst dein B6 als signed deklarieren

long ist signed.

[foobar (Gast)]

Der Code

unsigned a = 3446;
unsigned b = 4000;
long c = a - b;

würde doch a-b unsigned berechnen und anschließend auf signed long 
casten, oder?

[(prx) A. K. (prx)]

foobar schrieb:
> würde doch a-b unsigned berechnen und anschließend auf signed long
> casten, oder?

Beim MSP430 ja, aber nicht casten, sondern konvertieren. Nur kommt dabei 
kein -272302634 raus.

[foobar (Gast)]

auch wieder wahr...

Dann wären vielleicht doch ein etwas umfangreicherer Codeauszug und ein 
paar Infos über die Umgebung hilfreich.

[Dirk B. (dirkb2)]

-272302634 sind 0xEFC4FDD6
                    0xFDD6 sind -554 Das Ergebnis

Wo kommt der Müll im HighWord her?

[Karl H. (kbuchegg)]

A. K. schrieb:
> foobar schrieb:
>> würde doch a-b unsigned berechnen und anschließend auf signed long
>> casten, oder?
>
> Beim MSP430 ja, aber nicht casten, sondern konvertieren. Nur kommt dabei
> kein -272302634 raus.

Allrdings ist dieses Ergebnis in Hex 0xEFC4FDD6

und das unsigned Ergebnis von 3446 - 4000 macht 0xFDD6

Aufffallend ist, dass das Lower-Word übereinstimmt. Da steckt irgendwo 
ein Datentypfehler im nicht gezeigten Code.
Fazt, und man möchte hinzufügen: eigentlich so wie fast immer, zeig 
deinen ganzen Code und nicht nur die 3 Zeilen, die du für relevant 
hältst Denn wenn dort der Fehler wäre, hättest du ihn (normalerweise) 
längst schon selbst gesehen. Hast du aber nicht. Ergo sizt das Problem 
woanders (ausser das es eigentlich vor der Tastatur sitzt)

[(prx) A. K. (prx)]

Der erzeugte Code wär möglicherweise auch interessant.

[Mark B. (markbrandis)]

H. D. schrieb:
> B5 ist definiert als long

Mach Dir das Leben leichter (und Deinen Code portabler) mit:

#include <stdint.h>

[Olaf (Gast)]

Hm... welcher Datentyp ist den 4000? .-)

Olaf

[(prx) A. K. (prx)]

Olaf schrieb:
> Hm... welcher Datentyp ist den 4000? .-)

Wenns oben nicht gelogen ist, dann steht da
   long = long - int
Und das ist völlig harmlos.

[H. D. (lactobacillus)]

Hier die entsprechenden Funktionen. Bei p kommt mist raus, weshalb ich 
mich auf die Suche gemacht habe.

int AC1;
int AC2;
int AC3;
unsigned int AC4;
unsigned int AC5;
unsigned int AC6;
int B1;
int B2;
int MB;
int MC;
int MD;


long B5;
unsigned long p;



void Temp180(void){
  unsigned int UT;
  long X1;
  long X2;

  UCB0CTL1 = UCSWRST;
  UCB0CTL0 = UCMODE_3 + UCMST + UCSYNC;      // I2C master mode
  UCB0CTL1 = UCSSEL_2 + UCSWRST; // Use SMCLK, keep SW reset
  UCB0BR0 = 0x29;                          // < 400 kHz
  UCB0I2CSA = 0x77;                       // address
  UCB0CTL1 &= ~UCSWRST;


  //write start temperature
  UCB0CTL1 |= UCTR + UCTXSTT; // I2C TX, start condition
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0xF4;
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0x2E;
  while (!(IFG2 & UCB0TXIFG));
  UCB0CTL1 = UCSSEL_2 + UCTXSTP;
  while (UCB0CTL1 & UCTXSTP);

  _delay_cycles(80000);//5 ms


  //go to register
  UCB0CTL1 = UCSSEL_2 + UCTR + UCTXSTT; // I2C RX, start condition
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0xF6;
  while (!(IFG2 & UCB0TXIFG));

  //read register
  UCB0CTL1 = UCSSEL_2 + UCTXSTT; // I2C RX, start condition
  while (!(IFG2 & UCB0RXIFG));
  UT = UCB0RXBUF<<8;
  while (!(IFG2 & UCB0RXIFG));
  UT += UCB0RXBUF;


  UCB0CTL1 = UCSSEL_2 + UCTXSTP;
  while (UCB0CTL1 & UCTXSTP);

  //Temp
  X1 = (((long)UT-AC6)*AC5)>>15;

  //X3 = MC;
  X2 = ((long)MC<<11)/(X1+MD);

  B5 = X1 + X2;

}

void Pres180(){
  unsigned long UP;
  long X1;
  long X2;
  long X3;
  unsigned long X4;
  long B3;
  unsigned long B4;
  long B6;
  unsigned long B7;


  UCB0CTL1 = UCSWRST;
  UCB0CTL0 = UCMODE_3 + UCMST + UCSYNC;      // I2C master mode
  UCB0CTL1 = UCSSEL_2 + UCSWRST; // Use SMCLK, keep SW reset
  UCB0BR0 = 0x29;                          // < 400 kHz
  UCB0I2CSA = 0x77;                       // address
  UCB0CTL1 &= ~UCSWRST;


  //write start pres
  UCB0CTL1 = UCSSEL_2 + UCTR + UCTXSTT; // I2C TX, start condition
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0xF4;
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0xF4;
  while (!(IFG2 & UCB0TXIFG));
  UCB0CTL1 = UCSSEL_2 + UCTXSTP;
  while (UCB0CTL1 & UCTXSTP);
  _delay_cycles(416000);//26 ms

  //go to register
  UCB0CTL1 = UCSSEL_2 + UCTR + UCTXSTT; // I2C RX, start condition
  while (!(IFG2 & UCB0TXIFG));
  UCB0TXBUF = 0xF6;
  while (!(IFG2 & UCB0TXIFG));

  //read register
  UCB0CTL1 = UCSSEL_2 + UCTXSTT; // I2C RX, start condition
  while (!(IFG2 & UCB0RXIFG));
  UP = UCB0RXBUF;
  //blaa[0] = UCB0RXBUF;
  while (!(IFG2 & UCB0RXIFG));
  UP = (UP<<8) + UCB0RXBUF;
  //blaa[1] = UCB0RXBUF;
  while (!(IFG2 & UCB0RXIFG));
  UP = (UP<<8) + UCB0RXBUF;
  //blaa[2] = UCB0RXBUF;

  UCB0CTL1 = UCSSEL_2 + UCTXSTP;
  while (UCB0CTL1 & UCTXSTP);

  UP = UP>>5;




  //Pres
  B6 = (signed long)(B5)-4000;
  X1 = (((B6*B6)>>12)*B2)>>11;
  X2 = (AC2*B6)>>11;
  X3 = X1 + X2;
  B3 = (((((long)AC1<<2)+X3)<<3)+2)>>2;
  X1 = (AC3*B6)>>13;
  X2 = (((B6*B6)>>12)*B1)>>16;
  X3 = (X1 + X2 + 2)>>4;

  if ((X3 + 32768) < 0){X4 = ~(X3 + 32767);}
  else{X4 = X3 + 32768;}
  B4 = (AC4*X4)>>15;
  B7 = (UP-B3)*(50000>>3);
  if (B7 < 0x80000000){p = (B7<<1)/B4;}
  else{p = (B7/B4)<<1;}
  X1 = (p>>8)*(p>>8);
  X1 = (X1*3038)>>16;
  X2 = (7357*p)>>16;
  p = p + (X1 - X2 + 3791)<<4;




}

[B. S. (bestucki)]

A. K. schrieb:
> long = long - int
> Und das ist völlig harmlos.

Nur wenn man keinen Unterlauf produziert, denn dieser erzeugt 
undefiniertes Verhalten.

[Mark B. (markbrandis)]

H. D. schrieb:
> Hier die entsprechenden Funktionen. Bei p kommt mist raus, weshalb ich
> mich auf die Suche gemacht habe.

Die Variablennamen sind wirklich zum Abgewöhnen :-(

[Joachim B. (jar)]

B6 ist long B5 ist long also sollte doch das hier reichen:

B6 = B5-4000L;

ich muss doch nix casten ausser die 4000 als L ong zu benennen.

aber ich kenne nur gcc

[Ludger (Gast)]

Hallo,

b5 sollte bei 25 Grad einen Wert von 4000 haben. B6 ist mit Vorzeichen, 
und ädert das Vorzeichen bei 25 Grad.

Ludger

[H. D. (lactobacillus)]

Mark Brandis schrieb:
> Die Variablennamen sind wirklich zum Abgewöhnen :-(

Übernomen aus dem Datasheet ...

Joachim B. schrieb:
> B6 ist long B5 ist long also sollte doch das hier reichen:
>
> B6 = B5-4000L;

keine Wirkung:

B5 = 3324;
--> B6 = -1392181924

[(prx) A. K. (prx)]

Joachim B. schrieb:
> B6 = B5-4000L;
>
> ich muss doch nix casten ausser die 4000 als L ong zu benennen.

Unnötig. B5 ist schon long, also auch 4000.

[Karl H. (kbuchegg)]

H. D. schrieb:
> Mark Brandis schrieb:
>> Die Variablennamen sind wirklich zum Abgewöhnen :-(
>
> Übernomen aus dem Datasheet ...
>
> Joachim B. schrieb:
>> B6 ist long B5 ist long also sollte doch das hier reichen:
>>
>> B6 = B5-4000L;
>
> keine Wirkung:
>
> B5 = 3324;
> --> B6 = -1392181924

wo hast du den Wert her?
Machst du eine Ausgabe oder bist du in irgendeinem Debugger?
(Hintergrund: Mit diesen Zahlen sieht das so aus, als ob im High-Word 
Schmutz wäre. Das soll es bei Debuggern schon gegeben haben, dass sie 
mit manchen Datentypen nicht zurecht gekommen sind. Letzten Endes sind 
die tatsächlich in B6 vorhandenen Bytes wichtig.)

[H. D. (lactobacillus)]

Ich benutze Code Composer Studio 6 und die Werte guck ich mir bei einem 
Breakpoint an.

[Karl H. (kbuchegg)]

Optimizer eingeschaltet?

[H. D. (lactobacillus)]

Optimization level: 0 (Register Optimizations)

Speed vs. Size trade-offs: None.

[Karl H. (kbuchegg)]

Nur um sicher zu gehen.
Kannst du mal ein neues Projekt aufsetzen, in dem tatsächlich nur diese 
Berechnung

long B6;
B6 = B5-4000;

drinn ist (natürlich mit den entsprechenden Variablen und Datentypen wie 
in deinem Projekt) uns sonst nichts anderes?
Hast du dort dasselbe fehlerhafte Ergebnis?

[Ludger (Gast)]

Hallo,

ich benutze auch die Funktionen aus dem Datenblatt, auf einem AVR.
Funktioniert tadelos.
Wie sehen denn deine Rohwerte für druck und Temperatur aus ?

Ludger

[Bernd K. (prof7bit)]

Bist Du sicher dass B5 vorher auch wirklich 3446 war? Wie stellst Du das 
fest? Was ist mit dem left shift bei X2 weiter oben, kommt an der Stelle 
vielleicht der Müll in das high word von B5? Ist B5 überhaupt 
initialisiert, wird Temp180() wirklich vor Pres180() aufgerufen? Und 
zwar auch beim ersten Mal? Fragen über Fragen...

[H. D. (lactobacillus)]

Karl Heinz schrieb:
> Nur um sicher zu gehen.
> Kannst du mal ein neues Projekt aufsetzen, in dem tatsächlich nur diese
> Berechnung
>

> long B6;
> B6 = B5-4000;
>

> drinn ist (natürlich mit den entsprechenden Variablen und Datentypen wie
> in deinem Projekt) uns sonst nichts anderes?
> Hast du dort dasselbe fehlerhafte Ergebnis?

Geht nicht, weil er das wegoptimiert.


Ergänze ich die Funktion von oben durch:

  long T5 = 3333;
  long T6;

  T6 = T5 - 4000;

  B1 = T6;

T5 = -667;
T6 = 1014431077;

Das Ding will mich doch verarschen!