m328 adc-Verständnisproblem

[Karl K. (leluno)]

hallo,

ich verstehe es mal wieder nicht.

an dieser Stelle -
lcd_int4(adc_read()); // <== hier mehrere ADC-Auslesungen angezeigt
- werden mehrere Messungen angezeigt und überschrieben. D.h. die 
Funktion 'lcd_int4' wird nur einmal ausgeführt da sonst hintereinander 
geschrieben würde - aber der Wert wird mehrfach angezeigt.




Wie kann das sein? Was mache ich falsch?

void main(void)
//
{

clock_prescale_set(clock_div_2);//8mhz

init_timer_smh();
led1_init;

//usart_init;
//USART0_init(9600);
//relais_init;
i2c_init();
lcd_init();
adc_init();

lcd_clear(RED);
lcd_int4(adc_read());  // <== hier eine ADC-Auslesung

//while(1);




timer_smh=0;

while(1)
{

//------------------- smh timer ------------------
  if (timer_smh>99)
  {
    sec=sec++;
    timer_smh=0;
      if(sec>59){
        sec=sec-60;  min=min+1;
        lcd_goto(1,4);//              
        lcd_int2(min);//
        lcd_write(":");
      if(min==60){
        min=0; hou++;
        if(hou==24)
        {
        hou=0;
        sub_day_start();
        
        }
        lcd_goto(1,1);//              
        lcd_int2(hou);//
        lcd_write(":");              
        }
      }
    lcd_goto(1,7);//              
    lcd_int2(sec);//
//    led1_tog;              
  }
//------------------------------------------------

if(sec==3 && min%6==0/**/  )//alle sechs Minuten auslesen
{

  if(min==0/**/)//jede Stunde -------
  {
  }


//wr_read_reg();
//sub_regelung();
}


if(sec%5==1)
{
  u8 i=0;
  lcd_goto(10+i++,1);
lcd_int4(i);// <== hier 1mal angezeigt
  
lcd_int4(adc_read()); // <== hier mehrere ADC-Auslesungen angezeigt

}




 }//while
}//main

[Nemopuk (nemopuk)]

Deine Anfrage ist unklar formuliert. Gebe mal Beispiele der fehlerhaften 
Ausgabe und was du srattdessen erreichen willst.

Korrektes Einrücken hilft ungemein, den Quelltext zu verstehen.

Wo sind die Quelltexte von lcd_int4() und adc_read()? Ich will sie 
sehen, nicht nur anhand des Namens erraten, was sie tun sollten.

[Karl K. (leluno)]

Nemopuk schrieb:
> Wo sind die Quelltexte von lcd_int4() und adc_read()? Ich will sie
> sehen, nicht nur anhand des Namens erraten, was sie tun sollten.

pardon gehören natürlich dazu:

void adc_init()
{
   // Set the ADC reference to AVcc
    ADMUX = (1 << REFS0);
    
    // Enable ADC and set prescaler to 128 (16 MHz / 128 = 125 kHz)
    ADCSRA = (1 << ADEN) | (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0);
}


#define ADCchannel 2

int16_t adc_read()
{
    // Select ADC channel (0-7) by setting the lower bits of ADMUX
    ADMUX = (ADMUX & 0xF8) | (ADCchannel & 0x07);
    
    // Start the conversion
    ADCSRA |= (1 << ADSC);
    
    // Wait for the conversion to complete
    while (ADCSRA & (1 << ADSC));
    
    // Return the ADC result
    return ADC;
}

 lcd_int4(i16 x){
  uint8_t dum[5]={' ',' ',' ','0','\0'};
  u8 neg=0;
if (x!=0)  
{
  
if (x<0){x=x*-1;neg=1;}

    if (x>999){
        dum[0]=(x/1000+48);
        x=x%1000;
    dum[1]='0';
    dum[2]='0';
    
        }
  
    if (x>99){
        dum[1]=(x/100+48);
        x=x%100;
    if(neg){neg=0;dum[0]='-';}
     dum[2]='0';
       }
    if (x>9){
        dum[2]=(x/10+48);
        x=x%10;
    if(neg){neg=0;dum[1]='-';}
        }
         dum[3]=(x+48);
     if(neg){dum[2]='-';}


}
    lcd_write(dum);
}

Die ADC-Wandlung wird auf dem Bild unten mehrfach ausgeführt und 
angezeigt - sollte aber nur ein mal ausgeführt und angezeigt werden.

[Wastl (hartundweichware)]

Karl K. schrieb:
> pardon gehören natürlich dazu:

.... und vielleicht mal an die Hinweise zum Posten von
Quellcode denken.

[Nemopuk (nemopuk)]

Es ist zwar nicht dein Hauptproblem, ich wollte es aber trotzdem zeigen:

Deine Funktion lcd_int4() gibt positive Zahlen mit bis zu 4 Stellen aus. 
Negative Zahlen mit mehr als 3 Stellen führen zu einer fehlerhaften 
Ausgabe:

Der Test

int main(void) {
    lcd_int4(5);
    lcd_int4(50);
    lcd_int4(500);
    lcd_int4(5000);
    lcd_int4(-5);
    lcd_int4(-50);
    lcd_int4(-500);
    lcd_int4(-5000);
    return 0;
}

erzeugt folgende Ausgabe:

   5
  50
 500
5000
  -5
 -50
-500
50-0

[Nemopuk (nemopuk)]

So sieht main() einigermaßen ordentlich formatiert aus:

void main(void) {
    
    clock_prescale_set(clock_div_2);  //8mhz  
    init_timer_smh();
    led1_init;
    //usart_init;
    //USART0_init(9600);
    //relais_init;
    i2c_init();
    lcd_init();
    adc_init();
    lcd_clear(RED);
    lcd_int4(adc_read());  // <== hier eine ADC-Auslesung
       
    timer_smh = 0;
    while (1) {
        
        //------------------- smh timer ------------------        
        if (timer_smh > 99) {
            
            sec = sec++;
            timer_smh = 0;
            
            if (sec > 59) {
                sec = sec - 60;
                min = min + 1;
                
                lcd_goto(1, 4); 
                lcd_int2(min);  
                lcd_write(":");
                
                if (min == 60) {
                    
                    min = 0;
                    hou++;
                    
                    if (hou == 24) {                        
                        hou = 0;                        
                        sub_day_start();
                    }
                    
                    lcd_goto(1, 1);                     
                    lcd_int2(hou);                     
                    lcd_write(":");
                }
            }
            
            lcd_goto(1, 7);             
            lcd_int2(sec);             
        }
        
        //------------------------------------------------       
        if (sec == 3 && min % 6 == 0) { //alle sechs Minuten auslesen
            
            if (min == 0)  //jede Stunde -------
            {
                // nichts?
            }
            
            //wr_read_reg();            
            //sub_regelung();
        }
        
        if (sec % 5 == 1)   {
            
            u8 i = 0;            
            lcd_goto(10 + i++, 1);
            
            lcd_int4(i);  // <== hier 1mal angezeigt
            lcd_int4(adc_read());  // <== hier mehrere ADC-Auslesungen angezeigt
        }
        
    }  //while
    
}  //main

Und auf den problematischen Teil reduziert:

void main(void) {
    
    while (1) {
        
        if (timer_smh > 99) {            
            sec = sec++;
            timer_smh = 0;
        }
                   
        if (sec % 5 == 1)   {
            
            u8 i = 0;            
            lcd_goto(10 + i++, 1);
            
            lcd_int4(i);  // <== hier 1mal angezeigt
            lcd_int4(adc_read());  // <== hier mehrere ADC-Auslesungen angezeigt
        }
        
    }  //while
    
}  //main

Hier fallen mir folgende Dinge auf:

Ist timer_smh als "volatile" und 8 Bit Integer deklariert? Wenn nicht, 
können Lesezugriffe im Hauptprogramm durch Schreibzugriffe in der 
Timer-ISR gestört werden und zu unerwartetem Verhalten führen. 
Allerdings nicht zu dem konkreten Verhalten, dass wir hier sehen. Dafür 
muss es noch einen anderen Grund geben.

> lcd_goto(10 + i++
ergibt immer 10, weil i immer 0 ist.

> lcd_int4(i);
gibt immer 1 aus, weil i immer 1 ist.

> lcd_int4(adc_read());
Gibt mit Sicherheit nur eine ADC Messung aus. Denn etwas anderes kann 
diese Funtkion nicht tun.

Offenbar kommt die Ausgabe "   1 813" von diesen beiden Aufrufen.

Die unerwarte Ausgabe in der dritten Display-Zeile stimmt nicht mit dem 
Quellcode überein. Denn er enthält kein "lcd_goto(3,1)". Sie kann auch 
nicht von "lcd_int4()" kommen, weil diese Funktion immer 4 Ziffern 
rehtsbündig ausgibt.

Vermutlich ist der Fehler ganz woanders, in einem Stück Code, den du 
nicht gezeigt hast. Ich würde mal alle andere Display-Ausgaben 
auskommentieren um auszuschließen, dass sie stören.

[Karl K. (leluno)]

Nemopuk schrieb:
> Die unerwarte Ausgabe in der dritten Display-Zeile stimmt nicht mit dem
> Quellcode überein.

stimmt - "lcd_goto(3,1)" hab ich nachträglich eingefügt weil es in der 
ersten Zeile sonst blöd aussieht.


lcd_goto(10 + i++
ergibt immer 10, weil i immer 0 ist.

stimmt auch, habe die Deklaration nach oben gezogen. Ergebnis bleibt. 
Stelle wird nur einmal aufgerufen.

Nemopuk schrieb:
> Negative Zahlen mit mehr als 3 Stellen führen zu einer fehlerhaften
> Ausgabe:

Das ist bewusst in kauf genommen. Die Funktion dient zur Positionierung 
auf dem Display. Wenn größere negative Werte zu erwarten sind muss man 
das anders machen.


Nemopuk schrieb:
> Allerdings nicht zu dem konkreten Verhalten, dass wir hier sehen. Dafür
> muss es noch einen anderen Grund geben.

Jedenfalls Dank für dein Bemühen. Es hilft schon weiter zu wissen, dass 
mein Unverständnis nicht ganz unberechtigt ist.

[Nemopuk (nemopuk)]

Karl K. schrieb:
> stimmt - "lcd_goto(3,1)" hab ich nachträglich eingefügt
> stimmt auch, habe die Deklaration nach oben gezogen

Wenn du hilfreiche Hilfe bekommen willst, solltest du deine Probleme 
ordentlich beschreiben. Wenn man sich Mühe gibt, erkennt man die Ursache 
beim verfassen der Problembeschreibung sogar oft selbst.

> Ergebnis bleibt. Stelle wird nur einmal aufgerufen.

Damit kann ich nichts anfangen. Ist das jetzt eine gute oder eine 
schlechte Nachricht?

Nochmal von vorne: Zeige den kompletten Quelltext (ordentlich 
formatiert!), die dazugehörige Ausgabe, und was du stattdessen 
erwartest.

[Wastl (hartundweichware)]

Nemopuk schrieb:
> So sieht main() einigermaßen ordentlich formatiert aus:

Stefan Frings darf sich auch an die Regeln zum Posten von
Sourcecode halten.

[Karl K. (leluno)]

Nemopuk schrieb:
> Nochmal von vorne: Zeige den kompletten Quelltext (ordentlich
> formatiert!), die dazugehörige Ausgabe, und was du stattdessen
> erwartest.

ich werde den code noch mal komplett neu aufsetzen und dann hier 
einstellen.

Das Merkwürdige ist, dass die Messwerte überschrieben werden obwohl die 
Cursorposition bei lcd_write eigentlich mit jedem Wert fortgeschrieben 
wird:

void lcd_write_(uint8_t* str)
{
while(*str>31)
    {
        lcd_char_(*str++);
        curposx =curposx+ f_width;
        if(frame)curposx++;
    }
}

[Harald K. (kirnbichler)]

Karl K. schrieb:
> ich werde den code noch mal komplett neu aufsetzen und dann hier
> einstellen.

Aber dann bitte als Dateianhang, nicht im Posting selbst.

[Joachim B. (jar)]

Wastl schrieb:
> Stefan Frings darf sich auch an die Regeln zum Posten von
> Sourcecode halten.

ach Regeln?
er editiert ja nicht mal Beiträge innerhalb von 2 Minuten nur um seinen 
Beitragszähler hochzuschrauben.

[Harald K. (kirnbichler)]

Joachim B. schrieb:
> er editiert ja nicht mal Beiträge innerhalb von 2 Minuten nur um seinen
> Beitragszähler hochzuschrauben.

a) Geht das?

b) Ist der Beitragszähler irgendwie wichtig?

[Joachim B. (jar)]

Harald K. schrieb:
> a) Geht das?

solange keiner geantwortet hat kann jeder editieren, es müssen keine 2 
Beiträge innerhalb 2 Minuten im Thread sein.

Harald K. schrieb:
> b) Ist der Beitragszähler irgendwie wichtig?

frage Stefan, niemand nutzt so viele Nickname wie er und immer mit 
derselben langsam unglaubwürdigen Begründung und oft nur Gesabbel am 
Thema vorbei und ohne Sachkenntnis.

[Arduino F. (Firma: Gast) (arduinof)]

Joachim B. schrieb:
> und immer mit
> derselben langsam unglaubwürdigen Begründung und oft nur Gesabbel am
> Thema vorbei und ohne Sachkenntnis.

Projektion!

[Joachim B. (jar)]

Arduino F. schrieb:
> Projektion!

keine Ahnung was du meinst, habe seit ich hier angemeldet bin nur ein 
Name und in gleicher Zeit nur 1/6 Beiträge und verkneife mir im 
Gegensatz zu Stefan auch mal Kommentare.

[Manfred P. (pruckelfred)]

Joachim B. schrieb:
> und verkneife mir im
> Gegensatz zu Stefan auch mal Kommentare.

Lügner!

In diesem Thread hast Du bislang drei Beiträge, die nichts zur Sache 
beitragen. Du plapperst den Schulmeister nach und führst einen 
persönlichen Krieg, was der Sache einfach nur schadet.

Stefan hat hier versucht, etwas zur Sache beizutragen - das storpedierst 
Du mit vermeidbarer Leermeisterei.