Überprüfung meines Teilcodes

[Robert M. (andro86)]

Hallo,

ich versuche gerade eine PWM für einen DC-DC Wandler zu erzeugen. Dazu 
erfasse ich die benötigten Größen über einen ADC, berechne die benötigte 
PWM und gib diese raus.
Den Teilcode mit der ADC Erfassung und der PWM Einstellung habe ich euch 
mal hier rein kopiert. Nun da ich noch ein ziemlicher Anfänger auf 
diesem Gebiet bin, bitte ich euch mal über meinen Teilcode rüber zu 
sehen und mir eure Meinung mit zu teilen. Kann das so, wie ich es 
geschrieben habe auch funktionieren?

/***********************************************************************
Used µC       : ATmega32U4
Compiler      : AVR Studio 5.1
Configuration : External oscillator with 16 MHz. ************************************************************************

/***********************************************************************************************Adjusted Fuse-Bits************************************
**************************************************************************/  
 
/*
Notice: [x] means programmed (enabled or 0), while [] mean unprogrammed (diasabled or 1)!

BODLEVEL           [4V3]
HWBE    [x]
OCDEN    []
JTAGEN    []...need to be disabled because of the ADC pins
SPIEN    []
WDTON    []
EESAVE    []
BOOTSZ    [2048W_3800]
BOOTRST    []
CKDIV8    []
CKOUT    []
SUT_CKSEL  [EXTXOSC_8MHz_XX_258CK_65MS]
*/

/**************************************************************************
**************************** Header-files *********************************
**************************************************************************/

// <avr/io.h> integrates all the necessary PORT and register names.
#include <avr/io.h>

//<stdint.h> Integer types with predefined width such as uint8_t or int16_t.
#include <stdint.h>
  
//<stdbool.h> the use of Boolean. The type is bool.
#include <stdbool.h>

/***********************************************************************************************Variable Definition Global *********************      
**************************************************************************/

#define F_CPU 16000000UL     /* External quartz with 16 MHz */

volatile uint32_t duty_cycle = 127;

/**************************************************************************
************************ Declaration of functions *************************
**************************************************************************/

void ADC_Initialization(void);
  
uint16_t ADC_Read_Value(uint8_t pin_channel);

/**************************************************************************
******************************* Main program ******************************
**************************************************************************/
{
  bool sampling_end = false;  
  
  uint8_t pin_channel = 0x04;
  
    // set Pin F4-7 as Input (ADC Input Channels).
  DDRF &= ~((1<<DDF4)|(1<<DDF5)|(1<<DDF6)|(1<<DDF7));
    // deactivate Pull-Up Resistor on Pin F4-7.
  PORTF &= ~((1<<PF4)|(1<<PF5)|(1<<PF6)|(1<<PF7));

/**************************************************************************
****************************** Timer/Counter ******************************
****************************** PWM-generator ******************************
**************************************************************************/
TCCR0A|=(1<<WGM01)|(1<<WGM00); // Fast PWM-Mode                        
TCCR0A|=(1<<COM0A1); // OC0A become low at a match and high at top.
                    
                                                                        
// PWM-frequency = timer clock/(Prescaler * 256)= 16 MHz/(1*256) = 62.5 kHz 
TCCR0B|=(1<<CS00) // Prescaler = 1.                                       

TCCR0B&=~(1<<WGM02); // TCNT0 run continuous after match until TOP reaches.

OCR0A =  duty_cycle; // Clock ratio/ trigger the PWM.                                                      

DDRB|=(1<<PB7); // Set Pin P7 for OC0A as output. 

/**************************************************************************
******************************* Main Routine ******************************
**************************************************************************/
 while(1)
{
 // Sampling the current and voltage values.
while(!sampling_end) // while sampling_end is not satisfied.
 {  
   switch(pin_channel)
   {  
      // First measurement of the input voltage at pin PF4.
    case 0x04:  ADC_Initialization();
    u_in = ADC_Read_Value(pin_channel); 
    pin_channel = 0x05;
    break;
     // Second measuring input current at pin PF5.
    case 0x05:  ADC_Initialization();
    i_in = ADC_Read_Value(pin_channel);
           pin_channel = 0x06;
           break;
     // Third measurement output current at pin PF6.
    case 0x06:    ADC_Initialization();
    i_out = ADC_Read_Value(pin_channel);
    pin_channel = 0x07;
    break;
     //Fourth measurement output voltage on pin PF7.
    case 0x07:  ADC_Initialization();
    u_out = ADC_Read_Value(pin_channel);
    pin_channel = 0x04                   sampling_end = true;
    break;    
    default:
    break;
   }
 }
    
sampling_end = false; // For the next measuring cycle.  

/**************************************************************************
********************************* Algorithm *******************************
**************************************************************************/
( Hier wir das duty_cycle ermittelt

} // end main routine

)

/**************************************************************************
************************** Definition of the functions ********************
**************************************************************************/

void ADC_Initialization(void)
{
  ADMUX|=(1<<REFS0); // Set reference voltage to AVcc.

  ADCSRA|=(1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0);  // Set prescaler to 128.

  ADCSRA|=(1<<ADEN); // Activate converter.

  ADCSRA|=(1<<ADSC); //AD conversion is started
    
  while (ADCSRA & (1<<ADSC) )// Wait for completion of the conversion.
  {  
   } /* ADCW must be read once, otherwise the result of the next conversion is not taken. */

(void) ADCW;    
}    
  
uint16_t ADC_Read_Value(uint8_t pin_channel)
{
 /* ADMUX &= ~(0x1F) means: delete only the first 5 bit position and then put them accordingly to pin_channel.*/

 ADMUX = (ADMUX & ~(0x1F)) | (pin_channel & 0x1F);

 ADCSRA |= (1<<ADSC); // one Conversion "single conversion"

 while (ADCSRA & (1<<ADSC) ) 
 {                 // Wait for completion of the conversion
 }
 return ADCW;
}

[Karl H. (kbuchegg)]

    case 0x04:  ADC_Initialization();
    u_in = ADC_Read_Value(pin_channel);

du brauchst doch den ADC nicht dauernd immer wieder neu zu 
initialisieren.
Initialisierungen macht man einmal am Programmanfang und gut ists.


Warum so kompliziert

 // Sampling the current and voltage values.
while(!sampling_end) // while sampling_end is not satisfied.
 {  
   switch(pin_channel)
   {  
      // First measurement of the input voltage at pin PF4.
    case 0x04:  ADC_Initialization();
    u_in = ADC_Read_Value(pin_channel); 
    pin_channel = 0x05;
    break;
     // Second measuring input current at pin PF5.
    case 0x05:  ADC_Initialization();
    i_in = ADC_Read_Value(pin_channel);
           pin_channel = 0x06;
           break;
     // Third measurement output current at pin PF6.
    case 0x06:    ADC_Initialization();
    i_out = ADC_Read_Value(pin_channel);
    pin_channel = 0x07;
    break;
     //Fourth measurement output on pin PF7.
    case 0x07:  ADC_Initialization();
    u_out = ADC_Read_Value(pin_channel);
    pin_channel = 0x04                   sampling_end = true;
    break;    
    default:
    break;
   }
 }

wenn ich mir deine Variablen und deren Veränderung ansehe (und mir das 
Initialize vorgezogen denke), dann steht da nichts anderes als

    u_in = ADC_Read_Value( 4 );
    i_in = ADC_Read_Value( 5 );
    i_out = ADC_Read_Value( 6 ); 
    u_out = ADC_Read_Value( 7 );

und das ist dann doch ein ganzes Stück einfacher als deine komplizierte 
Version mit switch/case und Schleife rundherum.

[Steffen (Gast)]

Robert Malle schrieb:
> TCCR0B|=(1<<CS00) // Prescaler = 1.

hier fehlt das ;

[Karl H. (kbuchegg)]

Abgesehen davon sind da noch ein paar andere Syntax Fehler. Aber die 
sagt dir dann schon der Compiler.


Generell: schreib nicht haufenweise Code in einem Zug!
ALles was du damit tust ist: du stehst dann mit einem Haufen Code da, 
der voller Fehler ist und du hast keine Ahnung wo du mit fixen anfangen 
sollst.

[Steffen (Gast)]

Robert Malle schrieb:
>     pin_channel = 0x04                   sampling_end = true;

da ist noch was faul ;-)

[Robert M. (andro86)]

Syntaxfehler hab ich in meinem Code nicht, zumindest konnte ich es 
compilieren ohne das der Compiler gemeckert hat. Die fehlenden 
Simikolons hier verschwanden sicherlich, als ich meinen Code hier hin 
kopiert und "zurecht gerückt" hab.

[Karl H. (kbuchegg)]

Robert Malle schrieb:
> Syntaxfehler hab ich in meinem Code nicht, zumindest konnte ich es
> compilieren ohne das der Compiler gemeckert hat.

Das glaub ich nicht.
Wo zb ist denn die main() Funktion? Also der Funktionskopf der main() 
Funktione?

 Die fehlenden
> Simikolons hier verschwanden sicherlich, als ich meinen Code hier hin
> kopiert und "zurecht gerückt" hab.

Mag sein.
Was sagt uns das?
Poste Code als Anhang und tippe ihn nicht ab.