ATmega328P Pin0 funktioniert nicht wie gewollt

[Sascha W. (botimbo)]

Hallo liebe Gemeinde,

ich habe vor kurzem wieder angefangen, mich mit dem Thema 
Mikrocontroller
zu befassen und komme auch schon zu einem kleinen Problem.
Ich habe zusammen mit einem Arbeitskollegen eine Platine entwickelt.
Diese soll über die Analogen Eingänge ADC0 - ADC5 mithilfe eines 
Spannungs.
teilers den angeschlossenen Widerstand messen (Ist ein Meldelinientester 
für
Alarmanlagen) und mit zwei LED's den Zustand der jeweiligen Linie 
Ausgeben.
Zum Beispiel:
-bei einem Kurzschluss soll die Rote LED blinken
-bei einer offenen Linie (unendlicher Widerstand) soll die Rote LED 
konstant leuchten, bis diese über einen Reset-Taster wieder zurück 
gesetzt wird
-ist der Widerstand messbar (+/- 5%), leuchtet die Grüne LED

Soweit funktioniert das ganze auch wie gewollt. Ich habe nur Probleme 
mit der Auswertung von Linie 1 und 6.
Wenn der ATmega gestartet ist, blinkt die LED (grün) an Pin0 
(PCINT17/TXD).
Wenn ich den Aufgesteckten Widerstand (an Linie 6) entferne und somit 
eine offene Linie Simuliere, sollte LED rot von Linie 6 (PCINT16/RXD) 
aufleuchten.
Diese reagiert aber nicht (auch mit, über die Programmierung, 
abgeschaltetem Seriellen Port).
Wenn ich an Linie 1 den Widerstand abziehe, soll dementsprechend die 
LED's für Linie 1 reagieren (Pin10->PCINT2/SS und Pin11->PCINT3/MOSI) 
überhaupt nicht, stattdessen leuchtet die LED (grün von Linie 6) 
konstant.
Ich habe mittlerweile herausgefunden, dass es mit der Programmierung 
nichts zu tun hat (ich habe eine neue Programmierung erstellt, in der 
nur die LED's nach und nach aufleuchten und sich wieder abschalten).
Habt ihr eine Idee was ich übersehen haben könnte oder habt ihr einen 
passenden
Tipp parat?

Ich danke schon mal im vorraus für die Antworten.

[Dieter F. (Gast)]

Sascha W. schrieb:
> Ich habe mittlerweile herausgefunden, dass es mit der Programmierung
> nichts zu tun hat

Na, dann wird es wohl die Schaltung sein :-)

Da Du uns beides vorenthältst kann Dir nicht geholfen werden.

[Sascha W. (botimbo)]

> Da Du uns beides vorenthältst kann Dir nicht geholfen werden.

Keine Angst Schaltung und Programmierung kommen jetzt ;-)

[Sascha W. (botimbo)]

// Taster definieren
int Reset = 13;
int Modi = 12;
int RstStat = 0;
int ModStat = 0;
int Zugang_Modi=3000;

// LED definieren
int LEDrot[5];
int LEDgruen[5];
int LEDpause = 50;
// ------------------

// Meldegruppen dimensionieren
int MG[5];
int R1 = 10000.0; // Konstanter Widerstand
int R2 = 10000.0; // Zu messender Widerstand
long Messwert;
float SpannungR2; // Spannung über dem zu messenden Widerstand
float Widerstand;
int ML_Start=1;
int ML_Ende=5;
// ---------------------------

// Alarmmeldung oder Auslösung definieren
int Alarm[6];

void setup()
{
  //LED den ausgängen zuordnen
  LEDrot[0] = 10;
  LEDrot[1] = 8;
  LEDrot[2] = 6;
  LEDrot[3] = 4;
  LEDrot[4] = 2;
  LEDrot[5] = 0;
  LEDgruen[0] =11;
  LEDgruen[1] = 9;
  LEDgruen[2] = 7;
  LEDgruen[3] = 5;
  LEDgruen[4] = 3;
  LEDgruen[5] = 1;
  //---------------------------
  
  Serial.end();
  
  // Taster als Input setzen
  pinMode(Reset, INPUT);
  pinMode(Modi, INPUT);
  digitalWrite(Reset, HIGH);
  digitalWrite(Modi, HIGH);

  // LEDs den Pins zuordnen und einschalten
  for (int x = ML_Start; x < ML_Ende; x++)
  {
    pinMode(LEDrot[x], OUTPUT);
    pinMode(LEDgruen[x], OUTPUT);
  }

  // Analoge Pins für Meldegruppe zuordnen
  MG[0] = A0;
  MG[1] = A1;
  MG[2] = A2;
  MG[3] = A3;
  MG[4] = A4;
  MG[5] = A5;

  LED_Test();
}

void LED_Test()
{
    for (int x = ML_Start; x < ML_Ende; x++)
  {
    digitalWrite(LEDrot[x], HIGH);
    digitalWrite(LEDgruen[x], HIGH);
    delay(LEDpause);
  }
    for (int x = ML_Start; x < ML_Ende; x++)
  {
    digitalWrite(LEDrot[x], LOW);
    digitalWrite(LEDgruen[x], LOW);
    delay(LEDpause);
  }
}
void Widerstandmessen(int MG_pin)
{
  // 5 Messungen machen und den Mittelwert errechnen
  Messwert = 0;
  SpannungR2 = 0;
  Widerstand = 0;
  for (int i = 0; i < 5; i++)
  {
    Messwert += analogRead(MG[MG_pin]);
  }
  Messwert = trunc(Messwert / 5);

  // Spannung berechnen
  SpannungR2 = (5.0 / 1023.0) * Messwert;

  // Widerstand berechnen
  Widerstand = R1 * (SpannungR2 / (5.0 - SpannungR2));
  //delay(1000);
}

void StatusMeldegruppe(int MG_SM)
{
  if (Widerstand >= (R2 / 1.52) && Widerstand <= (R2 * 1.52))
  {
    if (Alarm[MG_SM] == 0) {
      digitalWrite(LEDrot[MG_SM], LOW); // Wenn kein Alarm ansteht, wird die rote Meldegruppen LED ausgeschaltet
    }
    digitalWrite(LEDgruen[MG_SM], HIGH);
  }
  if (Widerstand <= 50)
  {
    digitalWrite(LEDgruen[MG_SM], LOW);
    for (int i = 0; i < 3; i++)
    {
      digitalWrite(LEDrot[MG_SM], HIGH);
      delay(100);
      digitalWrite(LEDrot[MG_SM], LOW);
      delay(100);
    }
    if (Alarm[MG_SM] >= 0) 
    {
      digitalWrite(LEDrot[MG_SM], HIGH); // Wenn ein Alarm ansteht, wird die rote Meldegruppen LED wieder konstant eingeschaltet
    }
  }
  if (Widerstand >= 45000)
  {
    Alarm[MG_SM] += Alarm[MG_SM]++;
    digitalWrite(LEDgruen[MG_SM], LOW);
    digitalWrite(LEDrot[MG_SM], HIGH);
  }
}

void loop()
{
  for (int i = ML_Start; i < ML_Ende; i++)
  {
    //LEDgn = 2 + i;
    //LEDrt = 3 + i;
    Widerstandmessen(i);
    StatusMeldegruppe(i);
    delay(25);
  }
  RstStat = digitalRead(Reset);
  if (RstStat == LOW)
  {
    for (int i = ML_Start; i < ML_Ende; i++)
    {
      Alarm[i] = 0;
    }
  }
  ModStat=digitalRead(Modi);
  if(ModStat==LOW&&RstStat==LOW)
  {
    LED_Test();
  }
}

Ignoriert bitte den Piepser im Schaltplan (ist noch nicht angeschlossen)

[Karl M. (Gast)]

Welchen Sinn hat den das, der Datentyp int ?

int LEDrot[5];
int LEDgruen[5];

[Karl M. (Gast)]

Wo entprellst Du denn eine Eingänge ?

[Sascha W. (botimbo)]

Karl M. schrieb:
> Welchen Sinn hat den das, der Datentyp int ?
> int LEDrot[5];
> int LEDgruen[5];

Eine Variable für die for-Schleife, z.B. wird jeweils bei LEDrot und 
LEDgruen
mithilfe der Schleife, der jeweilige Ausgang genutzt

Bin auch gerne für Verbesserungsvorschläge offen, stehe noch recht weit 
am
Anfang der Materie. ;-)

[Karl M. (Gast)]

Sascha,

ich bin der Meinung man sollte alle Datentypen kennen und damit auch den 
Zahlenbereich.

Wie viele Bit hat dein AVR µC ? Und was gilt für integer?

[Sascha W. (botimbo)]

Karl M. schrieb:
> Sascha,
>
> ich bin der Meinung man sollte alle Datentypen kennen und damit auch den
> Zahlenbereich.
>
> Wie viele Bit hat dein AVR µC ? Und was gilt für integer?

Ist mir schon klar, das man alle Datentypen kennen sollte.
Frage: Was hat der Elefant mit einem Stück Kuchen zu tun?

[Dieter F. (Gast)]

Sascha W. schrieb:
> // LEDs den Pins zuordnen und einschalten
>   for (int x = ML_Start; x < ML_Ende; x++)

Also von 1 bis 4.

0 und 5 schauen dumm aus der Wäsche :-)


Sascha W. schrieb:
> void LED_Test()
> {
>     for (int x = ML_Start; x < ML_Ende; x++)

dto.

Sascha W. schrieb:
> void loop()
> {
>   for (int i = ML_Start; i < ML_Ende; i++)
>   {

Konsequent :-)

[Sascha W. (botimbo)]

Dieter F. schrieb:
> 0 und 5 schauen dumm aus der Wäsche :-)

Ich habe bewusst 0 und 5 ausser acht gelassen, um nur quasi Eingang 2 - 
5 erstmal zu nutzen, bis ich (auch mithilfe externer) das Problem in den 
Grid bekommen habe.

Ursprünglich ist es folgende Definition:

int ML_Start=0;
int ML_Ende=6;

[Karl M. (Gast)]

Hallo Sacha,

ich hatte es - das Problem - etwas suspekter hingeschrieben.

Und Dieter F. stößt dich direkt drauf.

Ich bin gespannt, ob Du das Problem finden kannst.
Es mit mit [] und deren Bedeutung zu tun.

[(prx) A. K. (prx)]

LEDs funktionieren in Durchlassrichtung besser.

[Karl M. (Gast)]

Was ist den dein "Grid" ?
Kenne ich nicht.

[Sascha W. (botimbo)]

Karl M. schrieb:
> Was ist den dein "Grid" ?
> Kenne ich nicht.

Sorry Karl, ich meinte griff nicht Grid

[Sascha W. (botimbo)]

Ist schon ein wenig spät für mich (versuche trotzdem geistig auf der 
höhe zu bleiben)

Das was ich bisher an meiner dargestellten Programmierung sehe ist, das 
durch die Definition von ML_Start=1 und ML_Ende=5, Eingang 1 und 6 gar 
nicht erst dargestellt werden.
Ist auch vollkommen richtig, da ich bewusst Meldelinie 2 bis 5 auswerten 
wollte, bis ich Programmiertechnisch das Problem in den Griff bekommen 
habe.

Wie weiter oben bereits beschrieben, heißt es eigentlich:

ML_Start=0;
ML_Ende=6;

Oder habe ich da einen gedanklichen Fehler?

[Karl M. (Gast)]

Das 3te mal:

Es hat mit [] und deren Bedeutung zu tun.

[Sascha W. (botimbo)]

Brillinat....ich blindfisch XD (Danke Dieter und Karl)
Korrektur:

// LED definieren
int LEDrot[6];
int LEDgruen[6];
int LEDpause = 50;
// ------------------

// Meldegruppen dimensionieren
int MG[6];

Meldelinie 1 und 6 funktionieren, wie sie sollen.