AtMega8 ADC: Funktioniert dieser Code?

[Roland N. (eroli)]

Hallo zusammen,

ich versuche gerade den ADC am Controller zu testen. Dafür hab ich einen 
Helligkeitssensor wie im Tutorial beschrieben an PC0 angeschlossen. Mein 
Code zum testen sieht nun so aus:

.include "m8def.inc"  ;Die Definitionsdatei des ATmega8 einfügen.

.def temp     = R16  ;Arbeitsregister

rjmp RESET ; Reset Handler
reti  ; IRQ0 Handler
reti  ; IRQ1 Handler
reti  ; Timer2 Compare Handler
reti  ; Timer2 Overflow Handler  
reti  ; Timer1 Capture Handler
reti  ; Timer1 CompareA Handler
reti  ; Timer1 CompareB Handler
reti  ; Timer1 Overflow Handler
reti  ; Timer0 Overflow Handler
reti  ; SPI Transfer Complete Handler
reti  ; USART RX Complete Handler
reti  ; UDR Empty Handler
reti  ; USART TX Complete Handler
reti  ; ADC Conversion Complete Handler
reti  ; EEPROM Ready Handler
reti  ; Analog Comparator Handler
reti  ; Two-wire Serial Interface Handler
reti  ; Store Program Memory Ready Handler

reset:
ldi temp, LOW(RAMEND)  ;Initialisieren des StackPointers:
out SPL, temp
ldi temp, HIGH(RAMEND)
out SPH, temp


ldi temp, 0xFF      ;Arbeitsregister "temp" mit Einsern füllen.
out DDRB, temp      ;Datenrichtungsregister des Ports B mit Einsern aus "temp" füllen (= alle Pins des Ports als AUSGANG)
out PORTB, temp      ;Alle Pins des Ports auf's High-Level bringen

out DDRD, temp      ; Alle Pins von PortD als Ausgang festlegen
out PORTD, temp      ; Alle Pins auf HIGH setzen

ldi temp, 0
out DDRC, temp  ; Port C als Eingang

; ADC einstellen
ldi     temp, (1<<REFS0)                   ; Kanal 0, interne Referenzspannung 5V
out     ADMUX, temp
ldi     temp, (1<<ADEN) | (1<<ADPS2) | (0<<ADPS1) | (1<<ADPS0) | (1<<ADLAR)
out     ADCSRA, temp


sei    ;Interrupts freigeben
;------------------------------

; ============================ Hauptprogramm ============================
loop:  
sample_adc:
    sbi     ADCSRA, ADSC        ; den ADC starten
wait_adc:
    sbic    ADCSRA, ADSC        ; wenn der ADC fertig ist, wird dieses Bit gelöscht
    rjmp    wait_adc

    in      temp, ADCL         ; immer zuerst LOW Byte lesen
    in      temp, ADCH        ; danach das mittlerweile gesperrte High Byte

  cpi temp, 0x99        ; temp >= 0x99 --> higher else lower
  brge higher
  brlo lower
higher:
  ldi temp, 0xFF
  out PORTB, temp        ; LEDs an
  out PORTD, temp
  rjmp loop          ; Von vorne beginnen
lower:
  ldi temp, 0
  out PORTB, temp        ; LEDs aus
  out PORTD, temp
  rjmp loop          ; Von vorne beginnen

; ============================ Hauptprogramm Ende ============================

Und es sollte sich so verhalten: temp >= 0x99 --> higher else lower.
Aber irgendwie geht er immer in Higher?
Sieht jemand meinen Denkfehler?

Dankeschön :-)

[spess53 (Gast)]

Hi

>    in      temp, ADCL         ; immer zuerst LOW Byte lesen
>    in      temp, ADCH        ; danach das mittlerweile gesperrte High Byte

Es reicht, ADCH einzulesen.

>  cpi temp, 0x99        ; temp >= 0x99 --> higher else lower
>  brge higher
>  brlo lower
>higher:

Das 'brge higher' kannst du weglassen. Wenn nicht nach 'lower' 
gesprungen, wird 'higher' eh ausgeführt. Außerdem ist 'brge' für 
vorzeichenbehaftete Werte.

MfG Spess

[Roland N. (eroli)]

Hi,

hatte deine Antwort zu spät gelesen, nun ist mein Progrämmchen so:

loop:  
sample_adc:
    sbi     ADCSRA, ADSC        ; den ADC starten
wait_adc:
    sbic    ADCSRA, ADSC        ; wenn der ADC fertig ist, wird dieses Bit gelöscht
    rjmp    wait_adc

    in      temp, ADCH

  cpi temp, 0xFF        ; temp >= 0xFF --> higher else lower
  brsh higher
  ldi temp, 0          ; Lower:
  out PORTB, temp        ; LEDs aus
  out PORTD, temp
  rjmp loop          ; Von vorne beginnen
higher:
  ldi temp, 0xFF
  out PORTB, temp        ; LEDs an
  out PORTD, temp
  rjmp loop          ; Von vorne beginnen

Im Debugger funktioniert alles wunderbar, allerdings geht er in der 
Realität IMMER in den HIGHER-Zweig (egal ob ich den Schwellwert auf 0xFF 
(wie oben) oder auf 0x00) habe).
Das heißt doch, dass der ADC  immer 0xFF in mein temp schreibt, richtig?

[spess53 (Gast)]

Hi

>Das heißt doch, dass der ADC  immer 0xFF in mein temp schreibt, richtig?

Sieht so aus. Hast du alle Versorgungsspannungen angeschlossen?

MfG Spess

P.S. Wozu ist 'sei    ;Interrupts freigeben' ?

[Roland N. (eroli)]

Das "sei" ist nur ein Überbleibsel, sollte wohl aber nicht stören, oder?

Habe gerade nochmal nachgemessen, die Spannung an PC0 ändert sich jenach 
dem von 0.4V bis 3.4V. Das sollte doch eine ausreichend große Differenz 
sein, um sie irgendwie zu detektieren, oder?

Habe nun auch nochmal die Versorgungsspannungen nachgemessen.

AREF (21) und AVCC (20) liegen beide bei 5V. GND (22) ist auch vorhanden 
(VCC (7) und GND (8) sind natürlich auch da). Ist doch richtig, oder?

[spess53 (Gast)]

Hi

>Das "sei" ist nur ein Überbleibsel, sollte wohl aber nicht stören, oder?

Nein.

>Ist doch richtig, oder?

Ja.

Hast du an den Ports B und D an allen Pins Leds? Wenn ja, dann gib doch 
mal den ADC-Wert dort aus.

MfG Spess

[Roland N. (eroli)]

Ja, sind überall LEDs dran. Habe ich auch schon gemacht. Ergebnis: Alle 
LEDs leuchten... ???