Help: ADC code br0ken seit upgrade auf gcc 4.7.2

[●● pit ●. (Gast)]

Hi,

bin etwas ratlos und würde mich über einen Tipp freuen:

Habe von gcc 4.6.2 - mit dem mein Projekt einwandfrei läuft - auf 4.7.2 
aktualisiert, nun klappt mein ADC-Oversampling Code nicht mehr. Bevor 
ich den gcc gewechselt habe, hat der Code recht gut 12 Bit aus dem ADC 
interpoliert.

Kann mir netterweise jemand erklären, was nun mit dem neuen gcc in 
meinem Code falsch ist und - optional - warum das vorher geklappt hat?

Der Code soll einfach im Interrupt 128 samples aufaddieren und dann 
dividieren, so das 1023 dann (128 * 1023) >> 5 == 4092 wäre.

Nun aber kommt nur 0-31 raus, wenn ich den Wert von

adcOversampledValue[n]

 auslese. Die Hardware ist nicht verändert.

Ohne Oversampling (kommentierte Zeile) bekomme ich erwartungsgemäß 
0-1023.

Config & Code unten.

1000dank,

 - pit

System/Config: ATMega328p, 8MHz int. Oszillator.
ADC Prescaler auf 62.5kHz
Free Running Mode
ich lese abwechselnd 2 ADC-Channels

#define adcChannels  2
#define adcSelectChannel(channel) \
  (ADMUX = (ADMUX & (~((1<<MUX3) | (1<<MUX2) | (1<<MUX1) | (1<<MUX0)))) | (channel))

uint8_t adcChannel = 0;
volatile uint32_t adcOversampledValue[adcChannels];

ISR(ADC_vect) 
{
  // adcOversampledValue[adcChannel] = ADC; // <- klappt, 0-1023
  
  static uint32_t adcOversampleSum[adcChannels];
  static uint8_t  adcOversampleCounter[adcChannels];

  adcOversampleSum[adcChannel] += ADC;

  if (++adcOversampleCounter[adcChannel] >= 128) {
    adcOversampledValue[adcChannel] = adcOversampleSum[adcChannel] >> 5;
    adcOversampleSum[adcChannel] = 0;
  }

  if (++adcChannel > (adcChannels - 1)) {
    adcChannel = 0;
  }

  adcSelectChannel(adcChannel);
}

[●● pit ●. (Gast)]

OK, hab's gefunden.

Man muss jetzt

adcOversampleCounter

auf 0 setzen, vorher hat der überlauf des uint8_t von 127->0 das wohl 
erledigt.

[Klaus W. (mfgkw)]

BTW: Wenn adcChannel außerhalb der ISR angefasst wird, sollte sie 
volatile sein.
Oder wenn nicht: dann würde ich sie static in die ISR nehmen, damit es 
klar ist.

[MWS (Gast)]

pit p. schrieb:
> Man muss jetzt
> adcOversampleCounterauf 0 setzen, vorher hat der überlauf des uint8_t von
> 127->0 das wohl  erledigt.

Seit wann läuft denn ein uint8 nach 127 über ?

[●● pit ●. (Gast)]

ja das ist mir dann im bett auch eingefallen :)
man sollte um 2 uhr nicht mehr alten code debuggen...

zwischenzeitlich im git nachgeschaut: war immer ein uint8_t und hatte 
ihn nicht zurück gesetzt in dem if()....

habe also noch immer keinen schimmer warum das jemals geklappt hat...


Klaus: adcChannel wird nur innerhalb angefasst, außer wenn cli()