Ringpuffer mit ADC Werten füllen und versetzt ausgeben

Vermutlich scheitert das schon daran, dass der Compiler deinen Quellcode 
nicht schluckt, oder?

Tip: "sizeof" ist eine Funktion

Auch wenn ich das "sizeof ringbuffer" einfach durch 256 ersetze komme 
ich nicht ans ziel.

Die Ausgabe von "c" mittels sprintf ergibt immer 0.

Gehe zurück auf "Los". Ziehe keine 4000 Euro ein.

Dir fehlen massiv C-Grundlagen.

Ausserdem nimmst Du die Situation nicht ernst genug. Der vorliegende 
Code lässt sich nicht kompilieren. Statt uns aber entweder den 
kompletten Code zu zeigen oder die Fehlermeldungen zu nennen, schreibst 
Du nur "Geht nicht".
So können wir Dir nicht helfen.

Entschuldigt bitte, wenn ich euch zu wenig Infos gegeben habe. Hier ist 
nun das komplette Projekt zum downloaden:

http://dl.dropbox.com/u/6671425/Projekt.zip

>Entschuldigt bitte, wenn ich euch zu wenig Infos gegeben habe.
Die Entschuldigung kannst Du Dir in die Haare schmieren. ;-)

Was fehlt sind Infos, Fehlermeldungen und Problembeschreibungen.

Und so ein Archiv zu entpacken ist erstmal unnötig aufwendig für uns 
(wenn es auch einige gibt die das machen), wenn man den kompletten 
Quellcode direkt lesen kann.

Es geht ja nicht primär darum, das wir zuwenig Informationen haben, 
sondern darum, das Du lernen musst ein Problem zu analysieren. Dazu 
gehört ersteinmal es zu beschreiben : Das Du es dann oft trotzdem noch 
nicht allein lösen kannst liegt an einem fehlenden weiteren Lernschritt. 
Aber die Beschreibung einer Situation ist der ersten Schritt.

"Nomen est Omen", "Am Anfang war das Wort und das Wort war Geist" etcpp.

uin=uin ???
Hä ??
Weißt du eigentlich was ein Ringpuffer ist ??
Man benutzt ihn so : Wenn er am Ende angekommen ist dann soll er zum 
Anfang gehen ! (Das machst du zwar aber danach machst du den Buffer 
Größer, also jedesmal wenn du durchgelaufen bist solange bis kein 
Speicher mehr da ist bzw. er den Stack oder deine Variablen 
überschreibt)
und wo zum Teufel sind eigentlich die Aufrufe für getring und putring.
Also wenn du die nicht aufrufst werden die auch nichts machen außerdem 
dürfen die nicht ind der main funktion stehen (Funktionen die Funktionen 
enthalten ?!?!!!!)
also
1.verstehen wie nen Ringbuffer funktioniert
2.verstehen was Pointer,Adressen,FIFOs ,stack,usw sind
3.Sich ne Einfürung in C durchlesen (für so ein programm solte ne 
einführung schon reichen)
4. loslegen
is noch kein Meister vom Himmel gefallen

Hier das komplette main.c

1

#include <avr/io.h>

2

#include <avr/interrupt.h>

3

#include <util/delay.h>

4

#include <stdlib.h>

5

#include "usbdrv/usbdrv.h"

6

#include "printf.h"

7

#include "usbsend.h"

8

9

10

volatile unsigned int count;

11

12

13

unsigned char b[] = {0x00, 0x01, 0x03, 0x07,

14

                     0x0F, 0x1F, 0x3F, 0x7F};

15

int uin;

16

int c;

17

18

19

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

20

void init (void)

21

{

22

  DDRA  = 0x00;   // Eingang

23

    DDRC  = 0xFF;   // Port C.0-7 = Ausgang

24

   PORTC = 0x00;   // LEDs loeschen

25

}

26

27

unsigned int read_ADC(unsigned char chanel)

28

{

29

   unsigned int ADC_value=0;

30

31

        ADCSRA = (1<<ADEN) | (1<<ADPS1) | (1<<ADPS0);    // Frequenzvorteiler / 8

32

  ADMUX = chanel & 0b11000111;

33

34

35

  ADCSRA |= (1<<ADSC);              // ADC aktivieren

36

     while(!(ADCSRA & (1<<ADIF)));     // auf Abschluss der Konvertierung warten (ADIF-bit)

37

38

     ADCSRA &= ~(1<<ADEN);             // ADC deaktivieren

39

     ADC_value = ADC;

40

41

     return ADC_value;

42

  }

43

44

int main(void)

45

{

46

  char line[101];

47

     init ();  

48

  DDRC=0xFF;

49

  usbInit();                  // init USB stack

50

  sei();                    // set enable interrupt on

51

  for(;;)                    // the main event loop

52

  {

53

          //uin = read_ADC(5); // Port A.5

54

        //uin=uin;

55

56

      // Kontroll LED

57

      //if(uin > 500)

58

      //  PORTC=0xFF;

59

      //else

60

      //   PORTC=0x00;

61

62

      static unsigned char ringbuffer[ 40 ];

63

      static unsigned short getindex = 0;

64

      static unsigned short putindex = 0;

65

      static unsigned short buffersize = 0;

66

67

      int putring( int uin )

68

      {

69

        //if ( buffersize >= sizeof ringbuffer )

70

        //  return -1;

71

72

        uin = read_ADC(5);

73

           uin=uin;

74

75

        ringbuffer[ putindex ] = uin;

76

        putindex++;

77

        if ( putindex >= 40 )

78

          putindex = 0;

79

        buffersize++;

80

        return 0;

81

      }

82

83

      int getring( void )

84

      {

85

        //if ( !buffersize )

86

        //  return -1;

87

        {

88

        buffersize--;

89

        c = ringbuffer[ getindex ];

90

        getindex++;

91

        if ( getindex >= 40 )

92

        getindex = 0;

93

        return c;

94

        }

95

      }

96

97

          PORTC = uin;     

98

      sprintf(line,"Wert = %4d \n", c);

99

      outprint(line);

100

  }

101

  return 0;

102

}

Beim Build kommen keine Fehlermeldungen. Mein Problem ist es, dass ich 
die Werte, die ich mittels read_ADC() auf uin gebe nicht in den 
Ringpuffer gespeichert werden. Nach dem auslesen des Ringpuffers bekomme 
ich nur 0 zu sehen. Also wird dieser wohl leer sein.
Soll heißen, dass ich keine Werte dort hinein bekomme. Ich glaube, dass 
die der Fehler in diesen Zeilen steckt:

1

uin = read_ADC(5);

2

uin=uin;

3

ringbuffer[ putindex ] = uin;

>Mein Problem ist es, dass ich
>die Werte, die ich mittels read_ADC() auf uin gebe nicht in den
>Ringpuffer gespeichert werden.

Woran merkst Du das?

>Nach dem auslesen des Ringpuffers bekomme ich nur 0 zu sehen.
An welcher Stelle im Ablau bekommst Du nur 0 zu sehen? Durch welches 
Mittel?
Simulator? Glaskugel?

> Also wird dieser wohl leer sein.
Aha. Wieder Glaskugel?

>Ich glaube, dass
die der Fehler in diesen Zeilen steckt:
Und nun kommt noch die Religion in's Spiel.

Nun ich will DIch nicht weiter quälen.

Sie Dir mal die Ausgabe der Variable von c am Ende von main an.
Beantworte mir folgende Frage? Welchen Wert soll c haben und durch 
welche Zuweisung? Qual Dich nicht. Ich weiss.
Aber frage Dich folgendes "Wird durch die Definition einer Funktion auch 
gleichzeitig bewirkt, das die Funktion an der Stelle aufgerufen wird"?

Irgendwas solltest Du mal selbst beantworten.

Wenn du eine Compileroptimierung eingeschaltet hast, gibt es deine 
Funktionen putring() und getring() wahrscheinlich gar nicht in 
übersetzten Code.

Um dich mal auf den richtigen Weg zu bringen ...
Weisst du, worin sich eine Funktionsdefinition von einem Funktionsaufruf 
unterscheiden ?

Eine Funktionsdefinition sieht folgendermaßen aus:

1

uint8_t funktion(uint8_t variable) {

2

  return variable + 1;

3

}

Ein Aufruf dieser Funktion eher so:

1

funktion(2);

Dein Problem liegt darin, dass du innerhalb der main-Funktion deine 
Funktionen getring und setring definierst, aber nicht aufrufst.
Eine Funktionsdefinition innerhalb einer Funktion sollte soviel ich weiß 
direkt zu einem Compilerfehler führen ( darum auch die vielen Nachfragen 
deswegen hier ).

getring und setring werden in der jetzigen Version gar nicht aufgerufen, 
daher wird auch nichts am Puffer geändert und auch gar nichts mit dem 
ADC gemessen.