16 Bit Multiplikation

[Martin (Gast)]

Hi,

ich hab folgenden Code:

void SetLightBrightness(uint8_t kanal, uint8_t brightness)
{
  uint16_t data = 0;

  [...]//Hier wird data aus einem Array ausgelesen

  uart_puts("data: ");
  uart_putuint16(data);
  uart_puts("\n");

  data = data * brightness / 100;

  uart_puts("result: ");
  uart_putuint16(data);
  uart_puts("\n");
}

Mein Problem war dann folgendes:
brightness = 50, data = 1307, result = 653.
brightness = 50, data = 1336, result = 12.
Mit 1336 * 50 bin ich über 16bit im Zwischenergebnis. Daher hab ich 
folgendes gemacht:

void SetLightBrightness(uint8_t kanal, uint8_t brightness)
{
  uint16_t data = 0;

  [...]//Hier wird data aus einem Array ausgelesen

  uart_puts("data: ");
  uart_putuint16(data);
  uart_puts("\n");

  uint32 Math32 = data * brightness / 100;

  char buffer[12];
  itoa(Math32, buffer, 10);
  uart_puts("result: ");
  uart_puts(buffer);
  uart_puts("\n");
}

Trotzdem kommen komische Ergebnisse raus, sobald ein Zwischenergebnis 
über 16bit ist - Warum?
Ich hab schon verschiedene Schreibweisen probiert (z.B. alle 
Berechnungen nacheinander), selbes Problem oder falsche Ergebnisse.
Weiß jemand Rat?

Vielen dank schon mal!


Noch ein paar Infos:
Controller ist ein Mega32L8, läuft mit 3,3V @ 8MHz.
AtmelStudio 6.1, Optimization ist auf -Os.

[chris (Gast)]

Der Compiler wertet erst den rechten Ausdruck aus und schreibt ihn dann 
in die Variable. Dabei ist die Größe der Variable irrelevant. D.h. die 
rechte Seite wird mit 16bit berechnet (weil data 16bit breit ist). Um 
das zu verhindern, musst du dem Compiler explizit sagen, dass er mit 
32bit rechnen soll:

uint32 Math32 = (uint32)data * brightness / 100;

So wandelt der Compiler data in eine 32bit Zahl und führt dann die 
Berechnung in 32bit durch.

lg
Chris

[Kaj (Gast)]

Oder einfach die Variable Data direkt vom Typ uint32 machen.

[Martin (Gast)]

Super, danke, das funktioniert! :)

[safety first (Gast)]

.. oder gleich als uint64 deklarieren.