Melodiengong

[Florian S. (didi34)]

Ich habe folgendes Programm programmiert, welches sechs Töne 
nacheinander in aufsteigender Tonleiter auf einem Lautsprecher ausgeben 
soll, jedoch kommen nur Tiefe und Hohe Töne durcheinander aber nicht 
nach der reihe. Was mache ich falsch?

/*
 * melodiengong.c
 *
 * Created: 01.10.2012 13:56:27
 *  Author: Florian
 */ 

#define F_CPU 8000000
#include <avr/io.h>
#include <util/delay.h>
#include <math.h>
#include <avr/interrupt.h>

#define int overflow=0,i=1;


void sleep ( uint8_t ms )
{
  for(; ms > 0; ms--) _delay_us(1);
}

void melodie_spielen(float periode)
{
    DDRB = 0xff;
    PORTB |= (1<<PB2);// high halbe Periodendauer
    sleep(periode/2);  
    PORTB &= ~(1<<PB2);// low halbe Periodendauer
    sleep(periode/2);
      
    
}
int main(void)
{
  sei();
  TIMSK = (1<<TOIE0);
  TCCR0 = (1<<CS02);
  TCNT0 = 0x00;
  while(1)
  {
    while(i==1)
    melodie_spielen(3370);
    while(i==2)
    melodie_spielen(3370);
    while(i==3)
    melodie_spielen(3030);
    while(i==4)
    melodie_spielen(2840);
    while(i==5)
    melodie_spielen(2525);
    while(i==6)
    melodie_spielen(2273);
    }  
    return 0;
  }

ISR(TIMER0_OVF_vect) // Timer interrupt
{
  if(overflow <=61)
  {
    overflow++;
  }
  else
  {
    if(i==6)
    {
      i=1;
    }

    else
    {
      i=i+1;
    }    
      overflow = 0;
  }
}

[avr (Gast)]

Florian Schuller schrieb:
> void sleep ( uint8_t ms )
> {
>   for(; ms > 0; ms--) _delay_us(1);
> }

Wann wird ms wohl kleiner 0 sein...

[avr (Gast)]

Florian Schuller schrieb:
> #define int overflow=0,i=1;

Das #define ist auch sinnlos...

Außerdem fehlt hier ein volatile.

[Joachim D. (Firma: JDCC) (scheppertreiber)]

avr schrieb:
> Wann wird ms wohl kleiner 0 sein...

Muß es nicht, gleich null reicht ja ...

[loeti (Gast)]

Sollte da nicht _delay_ms() aufgerufen werden?

[Florian S. (didi34)]

>Sollte da nicht _delay_ms() aufgerufen werden?

Nein die Periodendauer wird in microsekunden angegeben

[avr (Gast)]

Joachim Drechsel schrieb:
> avr schrieb:
>> Wann wird ms wohl kleiner 0 sein...
>
> Muß es nicht, gleich null reicht ja ...

stimmt da war ich wohl selbst blind.

[Walter (Gast)]

lässt sich das überhaupt compilieren mit dem komischen define?
sollte da nicht i undefiniert sein.

float braucht man hier auch nicht!

[Bernhard S. (b_spitzer)]

Das kann so nix werden. Erstmal was über Datentypen in C nachlesen!!

void sleep ( uint8_t ms )

Erwartet 8-Bit Werte (1 Byte, 0-255)

void melodie_spielen(float periode)

Hätte eigentlich gerne Fleißkommazahlen (4Byte...)

sleep(periode/2);

Ruft dann obige Funktion auf, die nur bis 255 funktioniert,

melodie_spielen(3370);

Ruft die Funktion mit einem 16-Bit (2 Byte) Ganzzahl-Wert auf, der 
halbiert aber immer noch viel zu groß für sleep() wäre....

Dabei macht der Compiler munteres Byte-Raten und packt Dir die unteren 8 
Bit des Wertes in den Übergabewert von sleep().

[Achim M. (minifloat)]

Florian Schuller schrieb:
> float periode
Florian Schuller schrieb:
>     sleep(periode/2);

Und schon wird ein Typ implizit gewandelt. uint16_t hätte es auch getan. 
Das muss auch nicht erst kompliziert emuliert werden, bläst den Code 
nicht auf, verbraucht weniger Rechenzeit und Flashspeicher.

Ganz gewiefte Jungs und Mädels machen die Tonerzeugung über zwei Timer. 
Einer macht die Töne, einer macht die Dauer.

[avr (Gast)]

Joachim минифлоть schrieb:
> Ganz gewiefte Jungs und Mädels machen die Tonerzeugung über zwei Timer.
> Einer macht die Töne, einer macht die Dauer.

Wobei der Aufwand dann schon DDS übersteigt. Mit DDS ist es dann auch 
spielend leicht mehrere Töne gleichzeitig abzuspielen.