ADC-Interrupt und UART keine Übertragung

[Marko (Gast)]

Guten leute,

Ich habe ein Fehler im Code komme aber nicht drauf habe schon gesucht 
aber beim vergleichen ist mir kein Fehler aufgefallen, denke es liegt am 
Interrupt vom ADC.
Kurz zum Code & Hardware, ATmega8L mit Ext. 4MHz Quarz und gebrannt wird 
er mit stk500 auf einem Steckbrett.
Das Programm soll später in 2 Versionen laufen
1)Einmal den ADC wert lesen und am PC schicken.
z.B:
Spannung = 1.1V
2)Wenn PB0 auf masse ist, dauer haft messen und an PC schicken.
z.B:
Spannung = 1.1V
Spannung = 1.9V
Spannung = 2.1V usw.

Hoffe ist verständlich genug :)

Ob die Übertragung richtig ist, weis ich nicht da am terminal nichts an 
kommt denke es liegt am ADC-Interrupt.

#ifndef F_CPU
#warning "F_CPU war noch nicht definiert, wird nun nachgeholt mit 4000000"
#define F_CPU 4000000UL  
#endif
#define BAUD 9600UL      // Baudrate
 
// Berechnungen
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)   // clever runden
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))     // Reale Baudrate
#define BAUD_ERROR ((BAUD_REAL*1000)/BAUD) // Fehler in Promille, 1000 = kein Fehler.
 
#if ((BAUD_ERROR<990) || (BAUD_ERROR>1010))
  #error Systematischer Fehler der Baudrate grösser 1% und damit zu hoch! 
#endif 

#include <avr/io.h> 
#include <util/setbaud.h>
#include <avr/interrupt.h>
#include <stdlib.h>

volatile uint8_t Faktor = 27;
volatile uint16_t Teiler = 10240;

float Volt_val = 0;
 

void uart_init(void)
{
  UBRRH = UBRR_VAL >> 8;
  UBRRL = UBRR_VAL & 0xFF;
 
  UCSRB |= (1<<TXEN);  // UART TX einschalten
  UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);  // Asynchron 8N1 
}
int uart_putc(unsigned char c)
{
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
    while (*s)
    {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" */
        uart_putc(*s);
        s++;
    }
}
int main()
{
/*Analog-Digital-Wandler-Init*/
ADMUX  = (1<<REFS0);//ReferenzSpannung AVcc | Kanal 0 ADC0(PC0)
ADCSRA = (1<<ADEN)|(1<<ADSC)|(1<<ADIE)|(1<<ADPS2)|(1<<ADPS1);//ADC enable| Singel Mode| Adc Interrupt | Frequenzvorteiler F_CPU/64

cli();
ADCSRA |= (1<<ADSC);//Dummy-wert ohne Interrupt auslösen 
while (ADCSRA & (1<<ADSC) ) {}        

sei(); //Interrupt Erlauben

  if(!(PINB & (1<<PINB0)))//wenn auf masse dann ADC-dauerhaft einsetzten und jedes mal Interrupt auslösen
  {
    while(1)
    {
      ADCSRA |= (1<<ADSC);
      while (ADCSRA & (1<<ADSC) )
      {}
  ADCSRA =(1<<ADIF);//Interrupt auslösen
    }//Ende while
  }//Ende if

  else //Einmal Messung durch führen und Interrupt auslösen
  {
    
    ADCSRA |= (1<<ADSC);
    while (ADCSRA & (1<<ADSC) ) //warte bis messung fertig ist
    {}
    ADCSRA =(1<<ADIF);//Interrupt auslösen 
  }//Ende else

return 0;

}//Ende main

ISR(ADC_vect)
{
uart_init();
char Wert [] = "Spannung = ";
Volt_val = ((ADCW * Faktor)/ Teiler);

dtostrf(Volt_val,1,2,Wert);

itoa(ADCW,Wert,2);
uart_puts(Wert);

}

Danke für die Hilfe und Tipps.

MfG Marko

[Karl H. (kbuchegg)]

Ich sehe in deinem Code keine Stelle an der die UART jemals 
initialisiert werden würde.

[Karl H. (kbuchegg)]

> Danke für die Hilfe und Tipps.

* In einer ISR werden keine UART Ausgaben gemacht.
  Die dauern viel zu lange

* Für den ADC brauchst du eigentlich überhaupt keinen Interrupt
  Der macht das ganze System nur unnötig kompliziert.

* Ähm. Deine 'Interrupt-Steuerung', wozu soll die überhaupt gut sein?

* Für die ADC Funktionalität wären ein paar Funktionen angebracht,
  damit du die Hauptschleife möglichst sauber und klein bekommst.
  Das würde deiner Programmübersicht gut tun.

* Codeformatierung!
  Nicht irgendwann, sondern jetzt!

* Anforderungen überprüfen!
> 1)Einmal den ADC wert lesen und am PC schicken.
  Das ist keine sehr sinnvolle Anforderung. Was soll 'einmal' messen
  bedeuten? Wie wird diese Einmalmessung angestossen.
  Nach dem Anlegen der Versorgungsspannung soll der ADC einmal messen
  ist normalerweise keine sinnvolle Anforderung. Das solltest du nochmal
  überprüfen, ob das wirklich das ist, was du willst.

* Du willst dein Programm auf keinen Fall aus main() rauslassen. Selbst
  wenn die Aufgabe darin besteht eine Einmalmessung zu machen, musst du
  das Programm innerhalb mein() halten.

* Schmeiss dein Programm weg und schreib es neu.
  Geht schneller, als da jetzt alles zurechtzurücken.
  Aber schreib es diesmal vernünftig! Deine main() könnte so aussehen.
  Das ist leicht zu überblicken und Fehler sind leichter zu sehen, als
  wie wenn du da einen haufen Code unstrukturiert aufstapelst. Was
  rauskommt, wenn du die Dinge unübersichtlich schreibst hast du ja
  jetzt gesehen. Also mach es nicht.

...

void PerformMeasurement()
{
  uint16_t adc_Value;
  float    Volt;
  char     Buffer[20];

  adc_Value = adc_read();
  Volt = ((adc_Value * Faktor)/ Teiler);

  sprintf( Buffer, "%d %.2f\n", adc_Value, Volt );
  uart_puts( Buffer );
}

int main()
{
  uart_init();
  adc_init();

  DDRB &= ( 1 << PB0 );   // PB0 auf Eingang
  PORTB |= ( 1 << PB0 );  // Pullup an PB0 einschalten

  if( PINB & ( 1 << PB0 ) ) {
    PerformMeasurement();
    while( 1 )
      ;
  }

  else {
    while( 1 )
      PerformMeasurement();
  }
}

[Marko (Gast)]

Danke für die Tipps habe es geändert/neu geschrieben.
sieht auch übersichtlicher aus, habe aber probleme mit der sprintf().
in deinem Beispiel wird
sprintf( Buffer, "%d %.2f\n\r", adc_Value, Volt );
bekomme ich den richtigen decimal wert aber der float zeigt nur die 
Vorkommastelle an und Nachkommastelle bleiben 00
999 2.00
999 2.00
999 2.00
sollte aber 2.63 sein.
Habe häufig an sprintf rum experimentiert kam aber oft nur misst bei 
rum,
ich kam zu dem Ergebnis

sprintf( Buffer, "\n\rSpannung= %.2f V",Volt);

Spannung=2î
Spannung=2î
Spannung=2î
Spannung=1î
Spannung=1î
Spannung=1î
Spannung=0®
Spannung=0®
Spannung=0®
Spannung=0®
Den Buffer habe ich groß genug gewählt wunder mich warum er bei dem 
String so ein murks darstellt(Ich weis der Programmierer macht Fehler, 
nicht das Programm).

den bin ich auch im avr simulator durch gegangen und da läuft er. Es 
werden zwar auch die Nachkommastellen als nullen dargestellt, aber läuft 
nur einmal durch und stoppt. Die Realität sieht immer anders aus.



Habe die Baud Berechnung raus gelassen, ist noch die Selbe wie oben.

#include <avr/io.h> 
#include <util/setbaud.h>
#include <avr/interrupt.h>
#include <stdlib.h>
#include <stdio.h>

volatile uint8_t Faktor = 27;
volatile uint16_t Teiler = 10240;


void adc_init(void)
{
uint16_t result;
/*Analog-Digital-Wandler-Init*/
ADMUX  = (1<<REFS0);//ReferenzSpannung AVcc | Kanal 0 ADC0(PC0)
ADCSRA = (1<<ADEN)|(1<<ADSC)|(1<<ADPS2)|(1<<ADPS1);
//ADC enable| Singel Mode|  Frequenzvorteiler F_CPU/64 

ADCSRA |= (1<<ADSC);                  // eine Dummy-Wandlung 
while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten

 result = ADCW;
}

void uart_init(void)
{
  UBRRH = UBRR_VAL >> 8;
  UBRRL = UBRR_VAL & 0xFF;
 
  UCSRB |= (1<<TXEN);  // UART TX einschalten
  UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);  // Asynchron 8N1 
}

int uart_putc(unsigned char c)
{
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
    while (*s)
    {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" */
        uart_putc(*s);
        s++;
    }
}

uint16_t ADC_Read( uint8_t channel )
{
  // Kanal waehlen, ohne andere Bits zu beeinflußen
  ADMUX = (ADMUX & ~(0b00001111)) | (channel & 0b00001111);
  ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
  return ADCW;                    // ADC auslesen und zurückgeben
}

void PerformMeasurement()
{
  uint16_t adc_Value;
  double    Volt;
  char     Buffer[25];

  

  adc_Value = ADC_Read(0);
  Volt = ((adc_Value * Faktor)/ Teiler);
  
  sprintf( Buffer, "\n\rSpannung=%.2f V",Volt );
  uart_puts( Buffer );
}


int main()
{

adc_init();
uart_init();

DDRB &= ( 1 << PB0 );   // PB0 auf Eingang
PORTB |= ( 1 << PB0 );  // Pullup an PB0 einschalten



 if( PINB & ( 1 << PB0 ) ) {
    while( 1 )
      PerformMeasurement();
  }

  else {
    PerformMeasurement();
  while( 1 )
      ;
  }


return 0;

}//Ende main

@ Karl Heinz
danke für die Tipps und für die Funktion PerformMeasurement()
hat mir sehr geholfen.

[Karl H. (kbuchegg)]

Marko schrieb:

> sieht auch übersichtlicher aus, habe aber probleme mit der sprintf().
> in deinem Beispiel wird
> sprintf( Buffer, "%d %.2f\n\r", adc_Value, Volt );
> bekomme ich den richtigen decimal wert aber der float zeigt nur die
> Vorkommastelle an und Nachkommastelle bleiben 00

Siehe
http://www.mikrocontroller.net/articles/FAQ#Datentypen_in_Operationen

und wenn du gleich dabei bist
http://www.mikrocontroller.net/articles/FAQ#Aktivieren_der_Floating_Point_Version_von_sprintf_beim_WinAVR_mit_AVR-Studio

[Marko (Gast)]

Danke für die hinweiße den zweiten link hatte ich schon gefunden danach 
lief erst float mit Nachkommastellen, dern erstenlink wollte ich gerade 
testen aber ich kann kein Projekt mehr builden, kommt die error

rm -rf main.o  adc-uart.elf dep/* adc-uart.hex adc-uart.eep adc-uart.lss adc-uart.map
Build succeeded with 0 Warnings...
avr-gcc  -mmcu=atmega8 -Wall -gdwarf-2 -std=gnu99 -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT main.o -MF dep/main.o.d  -c  ../main.c
../main.c:2:2: warning: #warning "F_CPU war noch nicht definiert, wird nun nachgeholt mit 4000000"
avr-gcc -mmcu=atmega8 -Wl,-Map=adc-uart.map main.o     -o adc-uart.elf
avr-gcc: CreateProcess: No such file or directory
make: *** [adc-uart.elf] Error 1
Build failed with 1 errors and 1 warnings...

es kommt immer der selbe Fehler ich habe alte Projekte versucht, das 
gleiche problem

[Nn N. (jaytharevo)]

Hast du irgendwas beim Projekt verändert?

Der Error "avr-gcc: CreateProcess: No such file or directory" deutet 
darauf hin das eine Datei fehlt.

Sonst einfach mal Fehlermeldungen Googeln, dass hilft meistens ;)!

www.google.at/search?q=avr-gcc%3A+CreateProcess%3A+No+such+file+or+direc 
tory

[Marko (Gast)]

viel gesucht nichts gefunden zu mindestens nicht brauchbares.
hat jemand ein Rat für mich?

[Marko (Gast)]

Nah dem ich avr Studio neu installiert hatte lief es habe mir bei links 
durch gelesen. das habe dann eingesetzt und eine neue Variable erstellt 
des typs double mit 0.00263671 (Faktor/Teiler). Bekomme aber den 
Übertragungsfehler wie oben? dann wird auch
sprintf( Buffer, "%d %.4f\n\r", adc_Value, Volt );
nicht richtig Übertragen es kommt dann,
3ð32 0.0844
3õ37 0.0976
3ù40 0.1055
4ò42 0.1107
4ó44 0.1160
4ô45 0.1187

Habe da noch eine Verständnis frage zu Strings
wenn ich "Hallo Welt" habe wird das doch so definiert
char Buffer[10] = "Hallo\n\r";
                   123456789+1 für \0

oder liege ich da falsch?

[Karl H. (kbuchegg)]

\n ist 1 Zeichen. Genauso wie \r

[Marko (Gast)]

ok also char buffer [8] würde es dann heißen, denn ich habe mal mit dem 
Buffer rum gespielt und gelesen (finde die Seite aber nicht) das wenn 
ich
char buffer[2] = "hallo" würde dann Ha ausgeben.
Habe dann mal in den code Buffer [1]
sprintf( Buffer, "Spannung=%.2f V",Volt );
Eingeben was dann eigentlich zu einem S führt?
bekam aber trotzdem Spannung=2î
Spannung=1î
Spannung=1î
Spannung=1î

[Karl H. (kbuchegg)]

Marko schrieb:
> ok also char buffer [8] würde es dann heißen,


lass doch den Compiler die Buchsataben zählen.

  char Buffer[] = "Hallo\n\r";

> denn ich habe mal mit dem
> Buffer rum gespielt und gelesen (finde die Seite aber nicht) das wenn
> ich
> char buffer[2] = "hallo" würde dann Ha ausgeben.

Nö. Das ist undefiniert. Alles mögliche kann passieren.

> Habe dann mal in den code Buffer [1]
> sprintf( Buffer, "Spannung=%.2f V",Volt );
> Eingeben was dann eigentlich zu einem S führt?

Zähl die Buchstaben. Wieviele brauchst du?

Hinweis: Es lohnt momentan noch nicht, da zu sparen!
Mach den Buffer 20 oder 30 Zeichen groß, so dass der Text auf jeden Fall 
reinpasst. Zu groß ist (bis auf den Speicherverbrauch) kein Problem, zu 
klein aber schon.

[Marko (Gast)]

klar habe die buchstaben gezählt und komme auf 18+1 habe den buffer dann 
auf 20 gestellt, aber egal was ich eingebe es kommt immer das selbe raus 
ob 30, 25 oder 100.

[Marko (Gast)]

Weiss langsam nicht mehr weiter, bin nochmal alles durch gegangen.

Die Ausgabe am terminal Passt nicht wird nicht richtig dargestellt.
Die Übertragung läuft auch dauerhaft, was eigentlich nicht seit sollte.

[Karl H. (kbuchegg)]

Programm?

Irgendwo müssen die Zeichen ja herkommen. Sieht so aus, als ob du 
irgendwo einen Buffer-Overflow oder einen fehlenden \0 hast.

[Marko (Gast)]

#include <avr/io.h> 
#include <util/setbaud.h>
#include <avr/interrupt.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <util/delay.h>



void adc_init(void)
{
uint16_t result;
/*Analog-Digital-Wandler-Init*/
ADMUX  = (1<<REFS0);//ReferenzSpannung AVcc | Kanal 0 ADC0(PC0)
ADCSRA = (1<<ADEN)|(1<<ADSC)|(1<<ADIE)|(1<<ADPS2)|(1<<ADPS1);//ADC enable| Singel Mode| Adc Interrupt | Frequenzvorteiler F_CPU/64 // (1<<ADIF)| gelöscht

ADCSRA |= (1<<ADSC);                  // eine ADC-Wandlung 
while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten

 result = ADCW;
}

void uart_init(void)
{
  UBRRH = UBRR_VAL >> 8;
  UBRRL = UBRR_VAL & 0xFF;
 
  UCSRB |= (1<<TXEN);  // UART TX einschalten
  UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);  // Asynchron 8N1 
}

int uart_putc(unsigned char c)
{
  UCSRA |= (1<<UDRE);
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
  while(*s)
    {   // so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" 
        uart_putc(*s);
        s++;
    }
}

uint16_t ADC_Read( uint8_t channel )
{
  
  
  // Kanal waehlen, ohne andere Bits zu beeinflußen
  ADMUX = (ADMUX & ~(0b00001111)) | (channel & 0b00001111);
  ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
  
  return ADCW;                    // ADC auslesen und zurückgeben
}

void PerformMeasurement()
{
uint8_t Faktor = 27;
double Teiler = 10240.00;
 
  uint16_t adc_Value;
  double    Volt;
  char     Buffer[20];
  
  
  adc_Value = ADC_Read(0);
  Volt = ((adc_Value  * Faktor) / Teiler);
  

  sprintf( Buffer, "\n\rSpannung= %.2f"Volt );
 
  
  uart_puts( Buffer );

}


int main()
{

adc_init();
uart_init();

DDRB &= ( 1 << PB0 );   // PB0 auf Eingang
PORTB |= ( 1 << PB0 );  // Pullup an PB0 einschalten




 if( PINB & ( 1 << PB0 ) ) {
    while( 1 )
      PerformMeasurement();
  }

  else {
    
  PerformMeasurement();
  while( 1 )
     ;
  }



return 0;

}//Ende main

Hatte den code nicht eingefügt weil er sich nicht groß ändert.
hatte mit unterschiedlichen string funktionen experementiert (dtostrf 
strcat usw.) kam aber immer zum selben prolem

[Karl H. (kbuchegg)]

Wie hast du das

  sprintf( Buffer, "\n\rSpannung= %.2f"Volt );

durch den Compiler gekriegt? Da fehlt ein ,

[Marko (Gast)]

das war mein fehler beim kopieren und habe beim Korrigieren übersehen.

[Karl H. (kbuchegg)]

Marko schrieb:
> das war mein fehler beim kopieren und habe beim Korrigieren übersehen.

Welches Korrigieren?

Häng doch bitte dein File so an, wie du es compilierst!
Das ist wichtig. Denn mit diesem Fehler ist es unmöglich, dass du ein 
HEX-File bekommst, welches du in den µC brennen kannst. D.h. du kannst 
schon ein HEX-File brennen, nur entspricht das eben nicht dem, was du 
jetzt programmiert hast.

Auf dieser Seite des Monitors kann ich nicht entscheiden, was da jetzt 
Sache ist. Im Zweifelsfall gehe ich davon aus, dass der gezeigte Code 
dann eben nicht der Code ist, der tatsächlich im µC läuft, womit deine 
ganze Fehlerbeschreibung hinfällig ist.

[Marko (Gast)]

Habe das ganze Projekt habe ich angehängt.

Und mit korrigieren meine ich, ich lese mir den post durch(LRS).

Ich kann den µC brennen, gehe den Code im AVR Simulator und da läuft die 
UART-Übertragen nur einmal durch. Nur auf dem µC läuft er in einer dauer 
schleife.

[Karl H. (kbuchegg)]

Da wird dann eben der Schalter nicht so schalten, wie du dir das 
vorstellst.

Ist das ein Schalter oder ein Taster?
Wie ist er angeschlossen?
Annahme: Taster, und der schaltet sie wie es üblich ist, nach Masse 
durch.
-> Dir ist klar, dass du noch vor dem Programmstart den Taster drücken 
und gedrückt halten musst, bis die erste Messung da ist?

[Karl H. (kbuchegg)]

int uart_putc(unsigned char c)
{
  UCSRA |= (1<<UDRE);
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }

was macht die Oder-Zuweisung an UCSRA da an dieser Stelle?
Damit hebelst du schon mal den Schutz gegen 'Überfahren' des UART aus, 
was dein Problem erklären würde.

[Karl H. (kbuchegg)]

"\n\rSpannung= %.2f\0"


Du brauchst den String nicht mit \0 abschliessen. Das macht schon der 
Compiler für dich. Jetzt hat dein String 2 Stück \0 Bytes am Ende :-)

[Karl H. (kbuchegg)]

>  DDRB &= ( 1 << PB0 );   // PB0 auf Eingang

Da fehlt ein ~

(Wirkt sich bei dir jetzt nicht grossartig aus. Ist aber trotzdem ein 
Fehler)

[Karl H. (kbuchegg)]

Ich denke mal, die UCSRA Sache könnte es gewesen sein. Schmeiss das raus 
und teste nochmal.

[Stefan E. (sternst)]

Karl Heinz Buchegger schrieb:
> was macht die Oder-Zuweisung an UCSRA da an dieser Stelle?
> Damit hebelst du schon mal den Schutz gegen 'Überfahren' des UART aus,
> was dein Problem erklären würde.

Wie das? Man kann UDRE nicht manuell setzen.

[Marko (Gast)]

int uart_putc(unsigned char c)
{

    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}

war eine kleine Hilfe für den Avr Simulator damit ich in I/O View nicht 
immer
das bit setzten muss.

Ich gehe davon das die Taster auf masse sind (STK500), der µC ist auf 
dem steckbrett aufgebaut.

[spess53 (Gast)]

Hi

>war eine kleine Hilfe für den Avr Simulator damit ich in I/O View nicht
>immer das bit setzten muss.

Im wirlichen Leben wird UDRE so gelöscht .

MfG Spess

[Stefan E. (sternst)]

spess53 schrieb:
> Im wirlichen Leben wird UDRE so gelöscht .

Nein, auch nicht.
Das ist ein reines Read-Only-Bit.

[Karl H. (kbuchegg)]

spess53 schrieb:
> Hi
>
>>war eine kleine Hilfe für den Avr Simulator damit ich in I/O View nicht
>>immer das bit setzten muss.
>
> Im wirlichen Leben wird UDRE so gelöscht .

Wobei man der Fairness halber sagen muss, dass das so von Atmel nicht 
dokumentiert ist, da hat Stefan Ernst schon recht. (Und ich habs auch 
erst jetzt nachgeschlagen).

D.h. keine Ahnung ob und wenn ja, was da wirklich passiert. Auf jeden 
Fall aber gehört das da nicht hin.

[spess53 (Gast)]

Hi

Entschuldigung. Verwechslung mit TXC.

MfG Spess

[Marko (Gast)]

Ich hatte mal alles aus der main gelöscht damit die
PerformMeasurement();
auch einaml durchlaufen wird aber, im terminal wird es immer als 
schleife ausgegeben

int main()
{

adc_init();
uart_init();

DDRB &= ~( 1 << PB0 );   // PB0 auf Eingang
PORTB |= ( 1 << PB0 );  // Pullup an PB0 einschalten
    
  PerformMeasurement();

return 0;

}//Ende main

[Stefan E. (sternst)]

ADCSRA = (1<<ADEN)|(1<<ADSC)|(1<<ADIE)|(1<<ADPS2)|(1<<ADPS1);
                             ^^^^^^^^^

[Karl H. (kbuchegg)]

Oh Shit.

Und ich hab mir die ADC Sachen nicht mehr angesehen.

[Marko (Gast)]

Die ADC interrupt erlauben hat doch nichts mit dem UART zu tuhen oder 
sehe ich das falsch?

[Karl H. (kbuchegg)]

Marko schrieb:
> Die ADC interrupt erlauben hat doch nichts mit dem UART zu tuhen oder
> sehe ich das falsch?

Ein erlaubter INterrupt, zusammen mit einem sei()

UND

einer nicht implementierten ISR

führt zu einem reset des µC.

(Da kommen dann auch deine scheinbaren Dauermessungen her)

[Marko (Gast)]

Das wusste ich nicht, hatte es auch raus genommen weil ich es ja nicht 
brauche.
Es Wird aber immer noch eine dauer übertragung durchgeführt.
kann es sein das es ander uart_puts() Liegen? weil es ja eine schleife 
ist

int uart_putc(unsigned char c)
{

    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
  while(*s)
    {   // so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" 
        uart_putc(*s);
        s++;
    }
}

[Karl H. (kbuchegg)]

Marko schrieb:
> Es Wird aber immer noch eine dauer übertragung durchgeführt.
> kann es sein das es ander uart_puts() Liegen? weil es ja eine schleife
> ist


Nein
Ist der Text jetzt richtig?

[Marko (Gast)]

Nein immer noch der selbe Fehler.
habe auch schon den Pfad überprüft ob der überhaupt der Richtige ist.
Ja ist der Richtige

[Karl H. (kbuchegg)]

OK.
Nur um sicher zu gehen:

Kannst du mal wo eine LED anschliessen?

Und dann dein Programm so umändern (ich hab jetzt mal eine LED an B-3 
angenommen:

int main()
{
  DDRB |= ( 1 << PB3 );     // LED

  PORTB |= ( 1 << PB3 );
  _delay_ms( 500 );
  PORTB &= ~( 1 << PB3 );
  _delay_ms( 500 );

  adc_init();
  uart_init();

  ....

Die LED darf/muss einmal beim Programmstart kurz aufleuchten. Und danach 
nie wieder. Wenn du sie danach wieder aufleuchten siehst, dann hast du 
noch andere Dinge, die zum Reset führen.

[Marko (Gast)]

Ja das macht sie, Sie Flackert.
Das heist Irgend wo wird das Programm neu gestart?

[Karl H. (kbuchegg)]

Marko schrieb:
> Ja das macht sie, Sie Flackert.
> Das heist Irgend wo wird das Programm neu gestart?

genau.
Also alles noch mal durchsuchen, wo du einen Interrupt freigegeben hast, 
für den du keine ISR hast.

(Oder du nimmst einfach mal alle sei() raus. Für einen kurzen Test ist 
das ok. Ansonsten finde ich persönlich das unbefriedigend, wenn man 
einen Fehler im Programm hat, den man im Grunde 'nur abschaltet' ohne zu 
wissen, wo der Fehler liegt.)

[Karl H. (kbuchegg)]

Du hast aber nicht zufällig den Watchdog per Fuse eingeschaltet?

[Marko (Gast)]

Nein Das einzige was drin gewessen ist war avr/interrupt.h die habe ich 
raus genommen ändert sich aber nichts.(um sicher zu sein habe ich den 
avr studio nach folgenden sachen suchen lassen sei, cli, isr und sind 
nicht im code vorhanden)
Mir ist aber noch was aufgefallen wenn ich die Delay änder die Frequenz 
beleibt die selbe, was eigendlich nicht sein sollte.

[Marko (Gast)]

Der WDTON war gesetzt, danke noch mal für die Hilfe.

[Mathias O. (m-obi)]

Könntest du bitte mal das aktuelle Studio-Projekt (*.aps) posten?

[Marko (Gast)]

Ja klar, Bitte.

[m-obi (Gast)]

Deine main sieht ja immernoch so seltsam. Und bei dem While-Schleifen 
fehlen die geschweiften Klammern. Läuft es denn bei dir so wie es soll?