Multiplexer bei dem Analog Comparator

[Starter (Gast)]

Hallo,
Funktionsweise soll so sein das er 2 unterschiedliche werte mit einem 
Sollwert vergleichen soll mit dem internen Comparator.

Mein Problem ist es das der Multiplexer einfach nicht das macht was er 
soll.
Er soll die eingelesen Werte vom PORTbereich vom AD Wandler (PINC0, und 
PINC1) vergleichen mit dem vorgegebenen Sollwert der auf PIND6 steckt.
Mein jetziger Quelltext sieht so aus.


http://pastebin.com/GWEnNE0z

[Stefan B. (stefan)]

Das sieht für mich fischig aus:

In zwei unmittelbar aufeinander folgenden C-Anweiungen den ADC-Eingang 
wechseln und darauf hoffen, dass die ISR in den paar Takten dazwischen 
den korrekten Vergleich macht?

Das sei() würde ich vor die while-Schleife ziehen.

Die Hexwerte bei der ADC Initialisierung habe ich nicht mit dem 
Datenblatt aufgeschlüsselt.

Quellcode auf pastebin posten, damit man schlecht Copy&Pasten kann, 
finde ich doof, wo es doch hier den schönen Anhang und Codeformatierung 
gibt.

[Starter (Gast)]

Ja erstmal danke für die Antwort,
Aber siehe mein Name...
ich bin blutiger Anfänger, und wurden in der Schule ziemllich ins Kalte 
Wasser geworfen.
Unser Lehrer hat nicht mal mit der C Programmierung angefangen
und wir sollen nun ein Projekt auf die Beine stellen, nach 2 Monaten 
einarbeiten... =(
Ich werde mal die paar änderungen einbauen, ich hätte ja gerne noch NOP; 
Befehle eingebaut aber der Befehl funktioniert nicht

[Stefan B. (stefan)]

Als Starter bist du ganz gut dabei, da kommen andere mit deutlich 
weniger an! Viel Erfolg!

Wenn du am ändern bist, überleg dir den Programmablauf.

Wenn du die Tastenabfrage wie jetzt in den Analog Compare Interrupt 
packst führt das zu Einschränkungen: Die Tasten reagieren dann nur, wenn 
der Analog Compare Interrupt auch aufgerufen wird! So etwas wäre fast 
nur brauchbar, wenn es z.B. darum geht einen Alarm über den Analog 
Compare Interrupt zu entdecken und dann per Taste auszuschalten.

Wenn es eine unabhängige Tastenbedienung werden soll (z.B. Batterietest 
o.ä.), nimmst du besser einen Timer Interrupt für die Tasten oder 
benutzt das Warteschleifenverfahren innerhalb while. Beides ist in 
Entprellung beschrieben.

[Starter (Gast)]

Ja also das Projekt soll im weitesten Sinne ein "Rauchmelder" darstellen
Der Interrupt soll eben der Alarm sein den man Quittieren kann
inzwischen hab ich die nop befehle eingebaut und es funktioniert auch
was mich nur sehr verwundert, ist das der Interrupt erst auslöst nach ca 
2 sec nach überschreiten des Schwellwerts

[Stefan B. (stefan)]

Bevor man den falschen Quellcode examiniert: Ist der Quellcode oben noch 
aktuell oder bereits wie angekündigt verändert?

[Starter (Gast)]

ich poste mal den neuen :)
einen moment

[Starter (Gast)]

/* ------------------------------------------------------------
  Titel : Projekt Rauchmelder
 -------------------------------------------------------------- 
 Funktion :  Pt100 und Photodiode überwachen
 --------------------------------------------------------------
 Prozessor : ATmega8
 Taktfrequenz externer Quarz:    3,686411 MHz 
 Sprache : C 
 Datum : ©2010 
 Version : 1.1 
 ------------------------------------------------------------ */

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
 
 ISR (ANA_COMP_vect)                                 // Interrupt  Analog Comparator
  {
  PORTB  |= 0x04;
  while(bit_is_set (PINB,2))
   {
    for(int x=0;x<=500;x++)
     {
      PORTB |= 0x02;
      _delay_ms(2.5);
      PORTB &=~ 0x02;
      _delay_ms(2.5);
     }
   _delay_ms (200);
   for(int y=0;y<=500;y++)
    {
     PORTB |= 0x02;
     _delay_ms(3);
     PORTB &=~ 0x02;
     _delay_ms(3);
    }
   }
  }

void main (void)
{
  DDRB  = 0x03;
  PORTB = 0x04;
  PORTD = 0x00;
  DDRD  = 0x00;
  ACSR  = 0x0b; // Analog Comparator Status Register
  SFIOR = 0x08; // Multiplexer ACME Bit 1
  ADCSRA&=~(1<<ADEN);//make sure ADC is OFF 
  
    while (1)
    {
      PORTB = 0x01;
  
    _delay_ms(2500);
    PORTB = 0x00;
    _delay_ms(6000);
  
    
    ADMUX = 0; // Wert PINC0 einlesen
      asm ("nop"::);
      asm ("nop"::);
        asm ("nop"::);
        asm ("nop"::);
         
    ADMUX = 1; // Wert PINC1 einlesen
         sei ();
   
    }
}