ADC am ATTiny24A hängt Controller auf

[Christian B. (luckyfu)]

Ich habe offensichtlich solch ein Brett vor dem Kopf, daß ich mal 
Unterstützung von außen benötige.
Ich will an einem ATTiny24A am Port PA0 eine analoge Spannung messen. 
Daß das nicht gerade der glücklichste Port dafür ist, das habe ich auch 
schon gemerkt, aber generell funktionieren sollte es ja schon damit, 
oder nicht?

anbei der aktuelle code, Das Problem ist: Im Simulator läuft der 1a 
durch, nur im Controller tut sich nichts.
Er Springt einmal in die Unterfunktion, bleibt dann aber hängen (nachdem 
er sie das 2. mal angesprungen hat), da offensichtlich keine Wandlung 
beendet wird. ich bin nun schon unzählige male durch die Initialisierung 
gegangen und habe das ein paar mal umgeschrieben, auch ein Programm, 
welches auf einem anderen Controller läuft geht hier nicht. Ich bin 
langsam echt am verzweifeln.

hier ist der code der .c Datei

#include <avr/io.h>
#include <avr/interrupt.h>

#include "mydefs.h"
#include "Lipo_Saver.h"
#include <util/delay.h>

#define Durchlaeufe_ADC 5

 
void init(void)
{
     /*  Schaltausgang 1: PB0
       Schaltausgang 2: PB1
       Fahrsignal out/In; PB2 
       Schaltausgang 3: PA7
       Schaltausgang 4: PA6
       Schaltausgang 5: PA5
       Schaltausgang 6: PA4
       Schaltausgang 7: PA3
       Schaltausgang 8: PA2
       Schalteingang: PA1
       AkkuSpannung analog in: PA0
     */
   DDRA = 0xFC;    //Datenrichtung der Pins festlegen
   DDRB = 0x03;
   PORTA = 0x01;    // Pull Ups einschalten
  
   ADMUX = (1 << REFS1);
   ADCSRA = (1 << ADPS0) | (1 << ADPS1);
   DIDR0 = (1 << ADC0D);      
}
 
int ADC_Messung(uint8_t Kanal)
{
  uint8_t i;
  uint16_t Messwert;
  uint32_t Mittelwert = 0;
  
  ADMUX &= ~((1 << MUX0) | (1 << MUX1) | (1 << MUX2) | (1 << MUX3));
  ADMUX |= Kanal;

  //for(i = 0; i < Durchlaeufe_ADC; i++)
  {
    Out4 = ON;
    
    while(ADCSRA & (1<<ADSC))
    {
      asm volatile("NOP");
    }

//Diese Stelle erreicht er beim 2. Aufruf der Funktion nicht mehr

  Out4 = OFF;  
  Mittelwert += ADC;
  }
  
  //ADCSRA &= ~(1 << ADSC);
  
  Messwert = (uint16_t)Mittelwert / Durchlaeufe_ADC;

  return Messwert;
}

int main(void)
{
  uint16_t Akkustatus;
  ADCSRA |= (1 << ADSC);  //Start der ersten AD Wandlung

   TCNT0 = 0;
   Drive_Out = OFF;
  
   Out4 = ON;    //nur für debugzwecke
   _delay_ms(500); //
   Out4 = OFF;    //
  _delay_ms(500);  //
 
//   ADCSRA |= (1 << ADIE);  
   ADCSRA |= (1 << ADSC);
   while(1)
   {      
    Akkustatus = ADC_Messung(0);
         
     if (Akkustatus <= Abschalten)
     {
       Drive_Out = OFF;  //Ausgang auf Low legen
       Drive_DDR = ON;    //Eingang auf Ausgang umschalten -> Puls wird unterdrückt -> Failsave springt an        
     } 
     else if(Akkustatus <= Warnlevel)
     {
//       Out4 = ON;      //Vorerst nur debug, Später Blinkmodus einschalten
     }
     else
     {
       Drive_DDR = OFF;  //Falls Eingang als Ausgang -> zurück auf Eingang stellen
//      Out4 = OFF;
     }    
    }
}

[g457 (Gast)]

> ich bin nun schon unzählige male durch die Initialisierung gegangen

Wozu? Die wird doch garnicht ausgeführt ∗hüstel∗

[Christian B. (luckyfu)]

oh weia ist das peinlich!
das scheint irgendwo zwischendrin rausgeflogen zu sein.

ok, mit ausführen der init scheint er jetzt AD Wandlungen zu machen, 
leider bleibt es beim Anschein, denn: der AD Wert ist immer 0, den ich 
zurücklese. Ich hab jetzt das Ganze soweit umgebaut, daß er nur noch bei 
einem AD Wert von 0 den Ausgang setzt, sonst nicht.

Leider ist der Ausgang immer gesetzt.

desweiteren hab ich diese Zeile

 ADMUX = (1 << REFS1);

ebenfalls gelöscht, somit sollte er ja automatisch VCC als Referenz 
nutzen.

ich hänge nochmal den ganzen Code an:

#include <avr/io.h>
#include <avr/interrupt.h>

#include "mydefs.h"
#include "Lipo_Saver.h"
#include <util/delay.h>
 
void init(void)
{
     /*  Schaltausgang 1: PB0
       Schaltausgang 2: PB1
       Fahrsignal out/In; PB2 
       Schaltausgang 3: PA7
       Schaltausgang 4: PA6
       Schaltausgang 5: PA5
       Schaltausgang 6: PA4
       Schaltausgang 7: PA3
       Schaltausgang 8: PA2
       Schalteingang: PA1
       AkkuSpannung analog in: PA0
     */
   DDRA = 0xFC;    //Datenrichtung der Pins festlegen
   DDRB = 0x03;
   PORTA = 0x01;    // Pull Ups einschalten
 
   ADCSRA = (1 << ADPS0) | (1 << ADPS2);
   DIDR0 = (1 << ADC0D);      
}
 
int ADC_Messung(uint8_t Kanal)
{
  uint8_t i;
  uint16_t Messwert;
  uint32_t Mittelwert = 0;
  
  ADMUX &= ~((1 << MUX0) | (1 << MUX1) | (1 << MUX2) | (1 << MUX3));
  ADMUX |= Kanal;
  Mittelwert = 0;
  
  for(i = 0; i < ADC_Messungen; i++)
  {
    ADCSRA |= (1 << ADSC);
    while(ADCSRA & (1<<ADSC))
    {
      asm volatile("NOP");
    }
    Mittelwert += ADC;
  }
  
  ADCSRA &= ~(1 << ADSC);
  
  Messwert = (uint16_t)(Mittelwert / ADC_Messungen);

  return Messwert;
}

int main(void)
{
  uint16_t Akkustatus;
  
  init();
  
  ADCSRA |= (1 << ADSC);  //Start der ersten AD Wandlung

   TCNT0 = 0;
   Drive_Out = OFF;
  
   Out4 = ON;    //nur für debugzwecke
   _delay_ms(500); //
   Out4 = OFF;    //
  _delay_ms(500);  //
 
   ADCSRA |= (1 << ADSC);
   while(1)
   {      
    Akkustatus = ADC_Messung(0);
         
     if (Akkustatus <= Abschalten)
     {
       Drive_Out = OFF;  //Ausgang auf Low legen
       Drive_DDR = ON;    //Eingang auf Ausgang umschalten -> Puls wird unterdrückt -> Failsave springt an        
     } 
     /*else*/ if(Akkustatus <= Warnlevel)
     {
       Out4 = ON;      //Vorerst nur debug, Später Blinkmodus einschalten
     }
     else
     {
       Drive_DDR = OFF;  //Falls Eingang als Ausgang -> zurück auf Eingang stellen
      Out4 = OFF;
     }    
    }
}

die Lip_Saver.h hat folenden Aufbau

#ifndef _Lipo_Saver_h_
#define _Lipo_Saver_h_

#define  Out1    SBIT( PORTB, 0 )
#define  Out2    SBIT( PORTB, 1 )
#define  Out3    SBIT( PORTA, 7 )
#define  Out4    SBIT( PORTA, 6 )
#define  Out5    SBIT( PORTA, 5 )
#define  Out6    SBIT( PORTA, 4 )
#define  Out7    SBIT( PORTA, 3 )
#define  Out8    SBIT( PORTA, 2 )

#define  Drive_Out  SBIT( PORTB, 2 )
#define  Drive_In  SBIT( PINB, 2 )
#define  Drive_DDR  SBIT( DDRB, 2 )

#define  In1      SBIT( PINA, 1 )

/*
Schaltausgang 1: PB0
Schaltausgang 2: PB1
Fahrsignal out/In; PB2
Schaltausgang 3: PA7
Schaltausgang 4: PA6
Schaltausgang 5: PA5
Schaltausgang 6: PA4
Schaltausgang 7: PA3
Schaltausgang 8: PA2
Schalteingang: PA1
AkkuSpannung analog in: PA0 */

#define Warnlevel 0    //Warnwert, max 1024
#define Abschalten 200    //Abschaltwert, max 1024
#define ADC_Messungen 1  //Anzahl der Messungen aus welchen der Mittelwert gebildet wird. Maximal möglich: 4194304

#define ON 1
#define OFF 0

#endif

ich wird morgen mal nen anderen Controller ausprobieren, vielleicht hat 
der durch meine Experimente auch nen Treffer abbekommen.

[Stefan E. (sternst)]

Christian B. schrieb:
> scheint er jetzt AD Wandlungen zu machen

Sehr erstaunlich, wo doch der ADC niemals enabled wird.

[Peter D. (peda)]

Christian B. schrieb:
> Ich will an einem ATTiny24A am Port PA0 eine analoge Spannung messen.
> Daß das nicht gerade der glücklichste Port dafür ist, das habe ich auch
> schon gemerkt,

Woran hast Du das gemerkt?


Christian B. schrieb:
> ich hänge nochmal den ganzen Code an:

Dann tue das auch!

Antwort schreiben
Wichtige Regeln - erst lesen, dann posten!

    Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

[Christian B. (luckyfu)]

so, nun bitte ich nochmals um Entschuldigung. ich habe so viel hin und 
her probiert, daß offensichtlich auch das ADEN setzen zwischendrin 
entfallen war. ABER: auch das hat keine Besserung gebracht.

Heute habe ich den Controller gewechselt und siehe da: es funktioniert 
auf Anhieb (nachdem ich das ADEN vorher beim alten Controller gesetzt 
habe)

es lag also am Hardwaredefekt. Beim Herumprobieren hab ich dann 
offensichtlich noch mehr "Schaden" angerichtet, der sich aber wieder 
richten lies :)

Vielen Dank an alle die geholfen haben, wenngleich es alles nichts 
nützte