AD - Wandlung

[Hüsameddin N. (hsameddin_n)]

Hallo Leute,
ich habe ein Programm geschrieben mitdem man an einem Eingang AD - 
Wandeln konnte. Jetzt habe ich das Programm umgeschrieben sodass es nun 
für 2 Eingänge möglich sein sollte. Jedoch funktioniert die methode 
nicht mitder ich das gemacht habe.... was müsste ich wie ändern damit es 
geht. Es sollen immer 2 Messignale vom ADC0 und 2 Messignale vom ADC1 an 
die serielle Schnittstelle gesendet werden.

Danke LG Thomas

#include <avr/io.h>
#include <inttypes.h>
#define FOSC 12000000UL  //12MHz Taktfrequenz
#define BAUD 9600UL    //9600 Baud
#define MYUBRR (FOSC/(16*BAUD))-1

/* ADC initialisieren */
void ADC_Init(void) {
uint16_t result;
ADMUX = (1<<REFS1) | (1<<REFS0); // interne Referenzspannung nutzen
ADCSRA = (1<<ADPS1) | (1<<ADPS0); // Frequenzvorteiler
ADCSRA |= (1<<ADEN); // ADC aktivieren
/* nach Aktivieren des ADC wird ein "Dummy-Readout" empfohlen, man liest
also einen Wert und verwirft diesen, um den ADC "warmlaufen zu lassen" */
ADCSRA |= (1<<ADSC); // eine ADC-Wandlung
while (ADCSRA & (1<<ADSC) ); // auf Abschluss der Konvertierung warten
/* ADCW muss einmal gelesen werden, sonst wird Ergebnis der nächsten
Wandlung nicht übernommen. */
result = ADCW;
}


while (1)
{
/***************************************************************************************************************/

for(i=0;i<=2;i++)
{
/* ADC Einzelmessung */

uint16_t ADC_Read( uint8_t channel )
{
// Kanal waehlen, ohne andere Bits zu beeinflußen

ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);  //Channel ADC0


ADCSRA |= (1<<ADSC); // eine Wandlung "single conversion"
while (ADCSRA & (1<<ADSC) ) // auf Abschluss der Konvertierung warten
;
return ADCW; // ADC auslesen und zurückgeben
}

void USART_Transmit( unsigned char data )
{
  while (!(UCSRA &(1<<UDRE)))
    ;

  UDR = data;
}

void USART_Transmit_String( char* s )
{
  while( *s != '\0' ) 
  {
    USART_Transmit( *s );
    ++s;
  }
}

int main (void)
{

 UBRRH=(unsigned char)(MYUBRR>>8); //4 höheren Bits des UBRR
 UBRRL=(unsigned char) MYUBRR; //4 niedere Bits des UBRR

 UCSRB=(1<<RXEN)|(1<<TXEN); //Empfangen und Senden ermöglichen
 UCSRC|=(1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //UCSRC Register wählen und via Z1 und Z0 8bit als Ergebnisslänge zum Einlesen festlegen

ADC_Init();


char Werte[10];

while(1)
{

int i = ADC_Read(0);

    itoa( i, Werte, 10 );
    USART_Transmit_String( Werte );
    USART_Transmit( ' ' );

}
}
}

/******************************************************************************************************************/

for(b=0;i<=2;i++)
{
/* ADC Einzelmessung */

uint16_t ADC_Read( uint8_t channel )
{
// Kanal waehlen, ohne andere Bits zu beeinflußen

ADMUX = 1;        //Channel ADC1


ADCSRA |= (1<<ADSC); // eine Wandlung "single conversion"
while (ADCSRA & (1<<ADSC) ) // auf Abschluss der Konvertierung warten
;
return ADCW; // ADC auslesen und zurückgeben
}

void USART_Transmit( unsigned char data )
{
  while (!(UCSRA &(1<<UDRE)))
    ;

  UDR = data;
}

void USART_Transmit_String( char* s )
{
  while( *s != '\0' ) 
  {
    USART_Transmit( *s );
    ++s;
  }
}

int main (void)
{

 UBRRH=(unsigned char)(MYUBRR>>8); //4 höheren Bits des UBRR
 UBRRL=(unsigned char) MYUBRR; //4 niedere Bits des UBRR

 UCSRB=(1<<RXEN)|(1<<TXEN); //Empfangen und Senden ermöglichen
 UCSRC|=(1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //UCSRC Register wählen und via Z1 und Z0 8bit als Ergebnisslänge zum Einlesen festlegen

ADC_Init();


char Werte[10];

while(1)
{

int i = ADC_Read(0);

    itoa( i, Werte, 10 );
    USART_Transmit_String( Werte );
    USART_Transmit( ' ' );

}
}
}


}

[spess53 (Gast)]

Hi

>// Kanal waehlen, ohne andere Bits zu beeinflußen

>ADMUX = 1;        //Channel ADC1

Wirklich?

MfG Spess

[Helfer (Gast)]

Oje, das ist ja ein wilder Mix in deinem Sourcecode. 
Funktionsdefinitionen doppelt und mitten in Anweisungsblöcken. Da ist 
ein C-Kurs in die Hose gegangen :(

[Helfer (Gast)]

Anm.:

>  UCSRC|=(1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //UCSRC Register wählen ...

Das sollte man nicht mit |= machen, sondern mit der direkten Zuweisung 
=, wenn UCSRC und UBRR am gleichen Speicherplatz liegen und URSEL 
entscheidet, auf welches Spezialregister zugegriffen wird.
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Der_UART#UART_initialisieren

>       itoa( adc_wert, Werte, 10 );

Hier ist utoa sinnvoller.