ADCW - Compiler meldet Fehler

[hal (Gast)]

Hallo,
in meinen ATmega128 Projekt habe ich folgende Routine:

uint16_t read_adc(uint8_t adc_input)
{
  ADMUX = (ADMUX & 0b11110000)|(adc_input & 0b00001111);
  
  ADCSRA |= (1<<ADSC);
  while(ADCSRA & (1<<ADSC));
  
  return ADCW;
  
}

Das Ding hat immer funktioniert. Jetzt meldet der Compiler allerdings 
folgenden Fehler:

Error  1  attempt to use poisoned "ADCW"  Y:\...\XXX.c  401  9  XXX
Error  2  'ADCW' undeclared (first use in this function)  Y:\...\XXX.c  401  9  XXX

Ich verstehe leider nur Bahnhof. Weiß jemand Rat?

Gruß
hal

[Magnus M. (magnetus)]

Der ATmeg128 hat kein Register mit dem Namen "ADCW".

[Karl H. (kbuchegg)]

hal schrieb:

> Ich verstehe leider nur Bahnhof. Weiß jemand Rat?

Autsch. Wasn da passiert?
Sieht so aus, als ob der Compiler da irgendwas verwechselt.


Kontrollier mal die Projekteinstellungen. Ist da sicher ein Mega128 
eingestellt? Hast du sicher avr/io.h inkludiert (und keines der 
speziellen io-Prozessor Files))

Welche includes hast du sonst noch?

[hal (Gast)]

Ja, stimmt! Hat aber bisher immer funktioniert? Das Register heißt ADC.

In der iom128.h hab ich folgendes gefunden.

/* ADC Data Register */
#define ADCW      _SFR_IO16(0x04) /* for backwards compatibility */
#ifndef __ASSEMBLER__
#define ADC       _SFR_IO16(0x04)
#endif
#define ADCL      _SFR_IO8(0x04)
#define ADCH      _SFR_IO8(0x05)

danke.

gruß
hal

[Karl H. (kbuchegg)]

Magnus M. schrieb:
> Der ATmeg128 hat kein Register mit dem Namen "ADCW".

Aber der Compiler weiß, was er damit machen muss.
Zumindest die vorhergehenden Versionen wussten, dass mit ADCW das 
Registerpärchen ADCH und ADCL gemeint ist.


Das das Teil ADCW heißt, hat historische Gründe, damit das ganze nicht 
mit der µC-Assembler Instruktion ADC (Add with Carry) verwechselt wird.
(weil die Include Files auch beim Assembler benutzt wurden)

[Jörg W. (dl8dtl) (Moderator)]

Lösch die Zeile

#pragma GCC poison ADCW

aus dem Header.  Da war Eric etwas vorschnell.

[Michael B. (planlessmichi)]

Ich bin mit dem neuen AVR-Studio jetzt eben über die gleichen Fehler 
gestoßen und die Google-Suche schickte mich dann auf den Thread hier.
Danke für den Tipp mit dem Löschen in der iom128.h

Jörg Wunsch schrieb:
> Lösch die Zeile
> #pragma GCC poison ADCW


Wenn ich das include-File aber richtig verstehe, wurde das absichtlich 
in einen Preprozessor-Block gebunden, der mit 
"__AVR_LIBC_DEPRECATED_ENABLE__" ein- und ausgeschaltet werden kann...
Ich habe bei mir jetzt in den Projekteinstellungen dieses define gesetzt 
und denke, dass das der sauberer Weg sein sollte.
Bei einem Update der AVR-Sourcen würde man ja sonst die gelöschten 
Zeilen wieder mit "aktivieren" und das Problem ist wieder da.

[Jörg W. (dl8dtl) (Moderator)]

Michael B. schrieb:
> Ich habe bei mir jetzt in den Projekteinstellungen dieses define gesetzt
> und denke, dass das der sauberer Weg sein sollte.

Nein, eigentlich ist das Löschen dieser Zeile schon der saubere
Weg.  Die hat nur aus Versehen das #pragma poison bekommen, weil
da oben bei der Definition stand "for backwards compatibility".
Das war aber ein uralter Kommentar, und wir (die avr-libc-Leute)
sind uns einig, dass ADCW durchweg unterstützt werden soll.

Im SVN sind beides (der alte Kommentar und das #pragma) bereits
entfernt, in der nächsten avr-libc-Version wird das also korrigiert
sein.

[Michael B. (planlessmichi)]

Ah, o.k.; hoffentlich bekommen die ATMEL-Leute das dann auch bald mit 
:-)
Ich habe nämlich gerade eben die derzeit aktuellste Studio6-Version 
(6.0.1996 Service Pack 2) heruntergeladen und auf meinem frisch 
aufgesetztem Rechner installiert. Und plötzlich konnte ich das Projekt, 
das vor zwei Wochen auf dem alten Rechner mit WinAVR noch einwandfrei 
compilierbar war, nicht mehr übersetzen.
Ich hatte allerdings diese poison-Meldung bei einer SIG_OUTPUT_COMPARE1A 
ISR...

Also verstehe ich das richtig, dass man vorsichtshalber alle pragma 
poison Zeilen löschen sollte? Falls ja: Wo ist der Unterschied zu dem 
angesprochenem define?

[Marius W. (mw1987)]

Michael B. schrieb:
> Ich hatte allerdings diese poison-Meldung bei einer SIG_OUTPUT_COMPARE1A
> ISR...

Die SIG_*-Bezeichnungen sind deprecated. Vielleicht ein Wink mit dem 
Zaunpfahl die mal umzubenennen...

Gruß
Marius

[Jörg W. (dl8dtl) (Moderator)]

Marius Wensing schrieb:
> Die SIG_*-Bezeichnungen sind deprecated.

Und dies bereits seit vielen Jahren.

Michael B. schrieb:
> Also verstehe ich das richtig, dass man vorsichtshalber alle pragma
> poison Zeilen löschen sollte?

Keineswegs.  Ausschließlich diese eine in iom128.h.