Problem mit ADC und ATTINY13

[Roland (Gast)]

Hey Allerseits,
hat eventuell einer von euch eine Idee, warum ich immer 0 vom ADC 
zurückbekomme? (OSZI und Multimeter zeigen 0.08V direkt am pin PB4 => 
ADC-Wert sollte also 80 sein...)

/*
 * InPlugV1_ATTINY13.c
 *
 * Created: 21.10.2012 13:08:11
 *  Author: Roland Blank
 */ 



// Definitionen
#define F_CPU 9600000                    // Frequenze des Controllers


#define LWEISON PORTB |= (1<<PINB1)
#define LWEISOFF PORTB &= ~(1<<PINB1)
#define LROTON PORTB |= (1<<PINB2)
#define LROTOFF PORTB &= ~(1<<PINB2)
#define TransON PORTB |= (1<<PINB0)
#define TransOFF PORTB &= ~(1<<PINB0)

// Includes
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <stdlib.h>
#include <stdint.h>
#include <avr/eeprom.h>

uint16_t readADC(uint8_t channel);

int main(void)
{
  //Init
  DDRB |= (1<<PINB1) | (1<<PINB2) | (1<<PINB0);      // PB0 = TransOUT, PB1 = LED-Weis, PB2 = LED-Rot
  DDRB &= ~(1<<PINB4);                  // PB4 als Eingang
  PORTB |= (1<<PINB4);
  //Var
  uint16_t t = 0;
  
  //Endlosschleife
  while(1)
    {
    LWEISON;
    TransON;
    _delay_ms(3);
    t = readADC(2);
    _delay_ms(1);
    if ((t>0))
    {
      LROTON;
    }else{
      LROTOFF;
    }
    LWEISOFF;
    TransOFF;
    _delay_ms(5);
    }
}

uint16_t readADC(uint8_t channel) {
  uint8_t i;
  uint16_t result = 0;
  
  // Den ADC aktivieren und Teilungsfaktor auf 64 stellen
  ADCSRA |= (1<<ADEN) | (1<<ADPS2) | (1<<ADPS1);

  // Kanal des Multiplexers waehlen
  // Interne Referenzspannung verwenden (also 2,56 V)
  ADMUX |= channel | (1<<REFS0);
  
  // Den ADC initialisieren und einen sog. Dummyreadout machen
  ADCSRA |= (1<<ADSC);
  while(ADCSRA & (1<<ADSC));
  
  // Jetzt 3x die analoge Spannung and Kanal channel auslesen
  // und dann Durchschnittswert ausrechnen.
  for(i=0; i<3; i++) {
    // Eine Wandlung
    ADCSRA |= (1<<ADSC);
    // Auf Ergebnis warten...
    while(ADCSRA & (1<<ADSC));
    
    result += ADCW;
  }
  
  // ADC wieder deaktivieren
  ADCSRA &= ~(1<<ADEN);
  
  result /= 3;
  
  return result;
}

Könnte es ein Problem sein, dass ich Debugwire aktive habe und den 
Controller mittels ICE3 + SPI/Debugwire programmiere/debugge?

Vielen Dank für eueren reichhaltigen Erfahrungsschatz :D

Gruß
Roland

[Spess53 (Gast)]

Hi

Hat es einen bestimmten Grund, das du bei jedem readADC den ADC neu 
initialisierst?

MfG Spess

[Roland (Gast)]

Nein eigentlich nicht... Es sollte aber auch kein Problem darstellen???

[Eumel (Gast)]

Hmmm, im Datenblatt steht was von: "Das erste Resultat nach der 
Initialisierung ist Müll"
Lass es einfach, es ist unnötig ;)

[katastrophenheinz (Gast)]

hi,

die interne Referenz ist 1.1V

[katastrophenheinz (Gast)]

Hi,

mit

PORTB |= (1<<PINB4)

aktivierst du den internen PullUp für den Eingang.
Ich glaube

PORTB &= ~(1<<PINB4)

wäre besser

[Roland (Gast)]

@Eumel: Die erste Convertierung wird jeweils nicht ausgewertet, dass 
Problem kann also nicht hier liegen...

@katastrophenheinz: Ja, die interne Referenz soll 1,1V sein...

[Roland (Gast)]

Leider löst

PORTB &= ~(1<<PINB4);

das Problem auch nicht :(

[katastrophenheinz (Gast)]

Hi,

Das hilft die jetzt auch nur bedingt weiter:
Bei mir läuft dein Code. Hab ich zwar nicht auf nem ATtiny ausprobiert,
sondern mit nem ATmega. Kommt genau das raus, was rauskommen soll. Der 
erste Wert weicht gering von den beiden folgenden ab, das liegt m.E. 
daran, dass die Wartezeit nach ADEN=1 zu kurz ist.

Zurück zu deinem Problem: Gibts beim ATtiny vllt. noch irgendein 
Register, mit dem man den ADC global anfeuern muss ?

[katastrophenheinz (Gast)]

Hi,


ich würde nochmal folgendes ausprobieren:

1) in readADC zuerst ADMUX schreiben, dann ADCSRA ( so stehts im manual 
)

2) während der Initialisierung den digitalen Modus von PB4 komplett zu 
deaktivieren: DIDR0 = ( 1 << ADC2D )

3) Prescaler höher setzen, z.b auf Maximum.

4) nach dem Setzen von ADEN noch mal eine ms warten.

[katastrophenheinz (Gast)]

Hi,

vergiss Punkt 1), Manual sagt nur, daß ADMUX nach Beginn der Wandlung 
nicht mehr geändert werden kann. Das machst du ja nicht.

[Roland (Gast)]

Die Benutzung von Debugwire kann kein Problem sein?

[Katastrophenheinz (Gast)]

Hi, debugwire: ich denke nicht, da nur die pins miso Mosi sck und Reset 
über den 6-poligen Stecker verbunden werden. Pb4 vom attiny13 ist keiner 
davon. Nutzt du debugwire, um durch den Code zu Steppen? Ich habe keine 
ahnung, wie sich der Adc verhält, wenn man da mit debugwire durchsteppt.
In welcher Umgebung rennt dein attiny? Irgendein protoboard? Pb4 evtl 
davon belegt?
hast du die möglichkeit, mit nem anderen atmel Controller, der einen 
Uart hat, zu testen?
Hat den Vorteil, dass man nicht nur auf blinkende LEDs starren, sondern 
auch mal Text/Werte ausgeben und interaktiv testen kann. Wenn das dann 
funzt, dann auf den attiny bringen. Das adc Zeug kann man fast ohne 
Anpassung von Typ zu Typ übertragen.

[Stone (Gast)]

Einen richtigen Fehler hab ich auf die schnelle nicht gefunden, aber 
gefährliche Sachen die sich irgendwann rächen werden.

  // Den ADC aktivieren und Teilungsfaktor auf 64 stellen
  ADCSRA |= (1<<ADEN) | (1<<ADPS2) | (1<<ADPS1);

  // Kanal des Multiplexers waehlen
  // Interne Referenzspannung verwenden (also 2,56 V)
  ADMUX |= channel | (1<<REFS0);

Besser

  // Den ADC aktivieren und Teilungsfaktor auf 64 stellen
  ADCSRA = (1<<ADEN) | (1<<ADPS2) | (1<<ADPS1);

  // Kanal des Multiplexers waehlen
  // Interne Referenzspannung verwenden (also 2,56 V)
  ADMUX = channel | (1<<REFS0);

Warum?
Wenn dein Programm z.B. Erst Channel 1 und dann Channel 2 auslesen soll 
ließt zu bei der zweiten Wandlung Channel 3 statt 2 aus. Du setz das 
ADMUX Register nie zurück, durch "=" statt "|=" wird das Register genau 
auf den Wert gesetzt und nicht mit einem möglicherweise falschen Inhalt 
verodert.
Positiver Effekt, durch das weggelassene verodern schneller+weniger 
Programmspeicher

  // Jetzt 3x die analoge Spannung and Kanal channel auslesen
  // und dann Durchschnittswert ausrechnen.
  for(i=0; i<3; i++) {
    // Eine Wandlung
    ADCSRA |= (1<<ADSC);
    // Auf Ergebnis warten...
    while(ADCSRA & (1<<ADSC));
    
    result += ADCW;
  }
  
  
  result /= 3;

Such dir lieber ne 2^n zahl aus 2,4,8,16,... Teilen im µC ist immer 
langwierig und Programmspeicher intensiv bei einer 2^n Zahl ist es eine 
einfache Shift-Operation, die geht fix und kostet kaum Speicher.


Wie sich Debugwire auf den adc auswirkt kann ich dir nicht sagen, aber 
setzt doch einfach nen Breakpoint nach der ADC-Funktion und schau dir 
das Ergebnis an dann sollte es keine Probleme geben.

Gruß Matthias

[Roland (Gast)]

@Katastrophenheinz: Es ist ein selbsterstelltes Board, welches sehr 
klein ist. Mehr als ein OSZI und zwei LEDS steht mir leider nicht zur 
Verfügung.

@Matthias: Danke für die Tips, ich habe sie gleich in meinen Code 
einfließen lassen. Leider lösen sie auch nicht das Problem.

Ich werde heute mal einen anderen Attiny13A verlöten, vielleicht hat nur 
er defekt.

Kann es Schaltungstechnik noch einen Fehler geben, selbst wenn ich 
direkt an PB4 mein Signal per OSZI messen kann?

[Roland (Gast)]

OK, hab das Problem gefunden. Der Controller scheint defekt zu sein. 
Nachdem ich einen Neuen eingebaut habe, hat es sofort funktioniert :D

Vielen Dank noch einmal für eure Hilfe!!!