Atmega16 ADC. Wo ist der Fehler?

[Robert P. (robert-p)]

Hallo,

ich habe eine Schaltung mit einem Mega16 und KTY81-110 aufgebaut. 
Schaltplan im Anhang.

Derzeit gebe ich die ADC-Werte und die berechnete Temperatur auf einem 
Display aus und die Werte stimmen nicht mit den erwarteten überein.

Bei ca 22°C Raumtemperatur sollte der ADC einen Wert von 276 liefern, 
jetzt komme ich aber nur auf 9°C und einen Wert von 255. Die 255 macht 
mich schon stutzig, wenn ich aber den Sensor abziehe, steigt der 
ADC-Wert wie erwartet auf 1023 an. Stecke ich einen Jumper statt des KTY 
auf, kommt wie erwartet eine 0.

Ein paar relevante Routinen:

/***********************************************************************************************/
int16_t calculate_temperature(uint16_t adc){
  //t=0,64593625*adc-155,4
  //#warning "TODO: update equation" done ;)
  int32_t adc_temp = 13229 * (uint32_t) adc;
  adc_temp = adc_temp / 2048;
  adc_temp = adc_temp - 1554;
  adc_temp = adc_temp / 10;
  return (int16_t) adc_temp;
}
/***********************************************************************************************/
void sprint_temperature(char* str,int16_t temperature){
  uint8_t one = 0, ten = 0, houndred = 0;
  bool _signed = false;
  if(temperature < 0){
    temperature *= -1;
    str[0] = '-';
    _signed = true;
    if(temperature > 99) temperature = 99;
  }
  one = temperature % 10;
  ten = (temperature % 100) / 10;
  houndred = (temperature % 1000) / 100;
  if(houndred == 0){
    if(ten == 0){
      if(_signed){
        str[1] = str[0];
      }else{
        str[1] = ' ';
      }
      str[0] = ' ';
    }else{
      if(_signed){
        str[0] = str[0];
      }else{
        str[0] = ' ';
      }
      str[1] = ten + '0';
    }
  }else{
    str[0] = houndred + '0';
    str[1] = ten + '0';
  }    
  str[2] = one + '0';
}
/***********************************************************************************************/
void adc_init(void){
  uint16_t result;
  ADMUX |= (1<<REFS0);                    // VCC with external capacitor at AREF
  ADCSRA |= (1<<ADEN) |/* (1<<ADPS2) | (1<<ADPS1) | */(1<<ADPS0); // ADC enabled, prescaler: 1
  ADCSRA |= (1<<ADSC);
  while(ADCSRA & (1<<ADSC)){;}
  result = ADCW; //dummy readout
  //ADCSRB = 0x00;
}
/***********************************************************************************************/
uint16_t read_adc(uint8_t portbit){
  /*uint16_t result;*/
  //ADMUX |= portbit;
  //uint16_to_string(str5,ADMUX);
  ADCSRA |= (1<<ADSC);
  while(ADCSRA & (1<<ADSC)){;}
  /*result = ADCW; //dummy readout
  result = 0;
  for(i=0;i<4;i++){
    while(ADCSRA & (1<<ADSC)){;}
    result += ADCW;
  }
  //ADMUX &= ~portbit;*/
  return ADCW;
}
static void avr_init(void)
{
  
  //Enable SPI
  spi_init();
  adc_init();
  
  //init communication
  //*
  USART_Init(0);
  
  
  RELAY_DDR |= (1<<RELAY_DISPLAY) | (1<<RELAY_CAM);

  DDRC |= (1<<PORTC5);

  CTRL_DDR |= (1<<REQUEST) | (1<<DISPLAY_READY);
  CTRL_PORT |= (1<<REQUEST) | (1<<DISPLAY_READY);
  //*/
  //Set power mode
  set_sleep_mode(SLEEP_MODE_IDLE);   
  
  //init PAL SYNC port
  set_bit(DDRC,7);
  //B.4 is sync:1000 ohm + diode to 75 ohm resistor
  //MOSI is video:330 ohm + diode to 75 ohm resistor
    
  // Initialize Sync for PAL
  OCR1A = 1024;        //One PAL line 64us
  TCCR1B = (1<<WGM12)|(1<<CS10);//full speed; clear-on-match
  TCCR1A = 0x00;      //turn off pwm and oc lines
  TIMSK = 1<<OCIE1A;    //enable interrupt from Timer1 CompareA
  return;
}

Im Hauptprogramm werden nur diese Routinen aufgerufen, der Sensor hängt 
am PA0.

Ich habe auch einen zweiten Mega16 probiert, da ist das gleiche Bild. 
Hängt man ein Poti statt des Sensors dran, springt der ADC-Wert von 316 
auf 255 und dann auf 0.

Der Widerstand des 2k7 Pullup stimmt nach Multimetermessung auch und 
kalte Lötstellen konnte ich nicht entdecken.

Ich danke euch für eure Hilfe.

[Sven (Gast)]

was ist "houndred" ;-)

ohne groß in dein Programm geschaut zu haben:
versuch Mal den Orginal-ADC Wert auszugeben

beim Debuggen immer alles in Einzelschritten testen

[holger (Gast)]

>  ADCSRA |= (1<<ADEN) |/* (1<<ADPS2) | (1<<ADPS1) | */(1<<ADPS0); // ADC 
>enabled, prescaler: 1

Mal den Dreck wegräumen

  ADCSRA |= (1<<ADEN) | (1<<ADPS0); // ADC enabled, prescaler: 1

Läuft deine CPU mit 200kHz?
Ich würde den Prescaler mal höher machen.

[Robert P. (robert-p)]

> Läuft deine CPU mit 200kHz?
> Ich würde den Prescaler mal höher machen.
Danke! Das hat geholfen. Ich habe die ADC-Routinen zum Teil aus anderen 
Programmen kopiert. deshalb habe ich ja gefragt.

Danke, danke, danke!

[Bronco (Gast)]

Der ADC braucht eine gewisse Zeit für Sample und Hold.
Ich weiß es beim AVR nicht auswendig, aber bei anderen µCs kann man 
diese Zeiten einstellen, und zwar in ADC-Clock-Takten.
Wenn die Zeiten zu Kurz sind (weil Dein ADC-Takt zu hoch), kann der ADC 
nicht genau arbeiten.