Init ADC beim ATT85

[Pierre G. (bqube)]

Guten Abend,

ich schreibe momentan für ein anderes Projekt ein Program, wo ich die 
Spannung über den ADC (PB4) Eingang am ATT85 einlese, verwerte und 
ausgebe.

Ich hatte schon mal erfolgreich mit einem Atmega8 eine ADC Wandlung plus 
Auswertung zum laufen gebracht und dachte mir ich benutzte die gleiche 
Init. die im GCC Tut angegeben ist für den ATT85.

void ADC_Init(void)
{
 
  ADMUX = (1<<REFS2);    
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);
  ADCSRA |= (1<<ADEN);
  ADCSRA |= (1<<ADSC);
  while (ADCSRA & (1<<ADSC) ) 
  {         
  }
  (void) ADCW;
}

Nach der Init. soll eine LED ein paar mal Blinken aber soweit kommt es 
garnicht, klammer ich die Init. aus (rufe sie vom Program her nicht auf)
dann fängt die LED an zu Blinken.

Stimmt was mit meiner Initialisierung nicht ?

Mfg Bqube

[Thomas E. (thomase)]

Pierre Gnauck schrieb:
> Stimmt was mit meiner Initialisierung nicht ?

Eigentlich nicht. Zumindest sollte der Controller nach der Init mit 
seinem Normalprogramm weiterlaufen.

Das ist allerdings Quatsch:
> ADMUX = (1<<REFS2);

Wenn du Vcc als Referenz nutzen willst, musst du an der Stelle gar 
nichts einstellen. Der Wert von REFS2 ist hier zwar egal, aber dann 
lässt man ihn auf 0. Alles andere verwirrt nur.

Was allerdings nicht eingetellt ist, ist der Eingang. PB4 ist ADC2. Im 
Moment misst du an ADC0, also an PB5(Reset). Das schadet aber auch 
nicht.

Zeig mal das ganze Programm.

mfg.

[Pierre G. (bqube)]

Das hier wäre das gesamte Programm:

#include <avr/io.h>
#ifndef F_CPU
#define F_CPU 1000000
#endif
#include <util/delay.h>

void ADC_Init(void)
{
 
  ADMUX = (1<<REFS2);    
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);
  ADCSRA |= (1<<ADEN);
  ADCSRA |= (1<<ADSC);
  while (ADCSRA & (1<<ADSC) ) 
  {         
  }
  (void) ADCW;
}



uint16_t ADC_Read( uint8_t channel )
{
  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);
  ADCSRA |= (1<<ADSC);
  while (ADCSRA & (1<<ADSC) ) {
  }
  return ADCW;
}

int main( void )
{
  DDRB = 0x03;
  uint16_t adcval;
  ADC_Init();
  
  PORTB |= (1<<0);
  _delay_ms(500);
  PORTB &=~ (1<<0);
  _delay_ms(500);
  PORTB |= (1<<0);
  _delay_ms(500);
  PORTB &=~ (1<<0);
  
  while( 1 ) {
    adcval = ADC_Read(2);
    ADC_Read();
    
    if(adcval >= 860)
    {
      PORTB |= (1<<0);
    }
    
    if(adcval <=840)
    {
      PORTB &=~ (1<<0);
    }
  }
}

Mfg Bqube

[Thomas E. (thomase)]

Das Programm läuft fast.

Das geht natürlich nicht:

Pierre Gnauck schrieb:
> ADC_Read();

Das kompiliert gar nicht. Aber sonst tut es eigentlich, was es soll.

#ifndef F_CPU
   #define F_CPU 1000000      //Unschoen
//   #error F_CPU not defined.  //Besser. Hat man sich aucht anders verdient.
#endif

#include <avr/io.h>
#include <util/delay.h>

void ADC_Init(void)
{ 
  ADCSRA = (1<<ADPS1) | (1<<ADPS0) | (1<<ADEN) | (1<<ADSC);
  while (ADCSRA & (1<<ADSC));
  (void)ADC;
}

uint16_t ADC_Read(uint8_t channel)
{
  ADMUX &= ~(0x0F);
  ADMUX |= (channel & 0x0F);
  ADCSRA |= (1<<ADSC);
  while (ADCSRA & (1<<ADSC));
  return ADC;
}

int main( void )
{
  DDRB |= (1 << 1) | (1 << 0);
  uint16_t adcval;
  ADC_Init();
  
  PORTB |= (1<<0);
  _delay_ms(500);
  PORTB &= ~(1<<0);
  _delay_ms(500);
  PORTB |= (1<<0);
  _delay_ms(500);
  PORTB &= ~(1<<0);
  
  while(1) 
  {
    adcval = ADC_Read(2);
    
    if(adcval >= 860)
    {
      PORTB |= (1<<0);
    }
    
    if(adcval <=840)
    {
      PORTB &= ~(1<<0);
    }
  }
}

Ich habe ein paar Kleinigkeiten verändert. Sonst schiesst du dir ins 
Knie, wenn du das mal anders verwendest(ADMUX/Ref). Es lief vorher aber 
auch. Aber jetzt ist alles richtig. Richtiger sozusagen.

mfg.

[Pierre G. (bqube)]

Vielen Dank für deine Hilfe, ich hab das ganze Compilert und direkt mal 
ausprobiert :-).

Jetzt schau ich mir noch deine Veränderung an und warum mein 
Uhrsprungscode nicht funktioniert hat.

Falls es jemanden Interessiert hier ist das Projekt dazu.

Beitrag "Spg_Auswertung_KFZ"

Mfg Bqube

[Pierre G. (bqube)]

Moin moin,
Mir ist bis auf den Funktions Aufruf ADC_READ(); warum auch immer ich 
den drin hatte, nur aufgefallen das es nicht wie bei mir (void)ADCW; 
sonder nur (void)ADC; heißt ? War das etwar der Fehler.

MFG Bqube

[Thomas E. (thomase)]

Läuft es denn jetzt?

Pierre Gnauck schrieb:
> ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);

Da steckt der Fehler drin. Das hat in diesem Fall aber zufällig keine 
Auswirkungen. Das Programm läuft ja auch in deiner Version. Bei auf 
diesen Lapsus hier:
> ADC_Read();
Aber damit kompiliert es auch nicht.

Warum es bei dir nicht lief oder immer noch nicht läuft, kann ich dir 
nicht sagen.

Aber das ADMUX wird hier platt gemacht, obwohl dort die Konfiguration 
der Referenzspannung drinsteckt. Und dann steht sie eben nicht mehr 
drin. Das hat hier keine Auswirkung, da die Referenzbits bei Vcc ohnehin 
alle 0 sind. Stellst du allerdings eine andere Referenz ein, schiesst du 
dir damit ins Knie und suchst dich tot. Weil sonst funktionierte es ja 
immer. Die zweite Sache dabei ist, dass auch bei Veroderung 5 Bits 
gelöscht werden, der Muxer aber nur mit 4 Bits eingestellt wird.

Ob ADC oder ADCW gelesen wird ist egal, da beides definiert ist:

#ifndef __ASSEMBLER__
#define ADC     _SFR_IO16(0x04)
#endif
#define ADCW    _SFR_IO16(0x04)

mfg.