Piep mit PWM - "Blop"

Gast #3049187
Lesenswert?

Hallo

Ich habe mit einem AVR (Mega8) und Beispielen von Aquaticus und Scott 
Harden einen einfachen Piepgenerator für Morsecode programmiert unter 
Verwendung des PWM mit einer Frequenz von etwa 1 kHz.

Das Problem ist, dass alle Punkte und Striche von einem "Blob" gefolgt 
sind, ihr könnt es im beiliegenden Beispiel hören und im Bild auch 
sehen.

Dieses Audiofile ist zwar vom Empfänger aufgenommen, aber es tönt auch 
so, wenn ich einen Lautsprecher lokal anschliesse. Hier mein Code:
1
void beep() {
2
  OCR1B = DEFAULT_VOLUME;
3
  ICR1 = 500;
4
  _delay_ms(50);
5
  ICR1 = 0;
6
}
7

8
void rest() {
9
  _delay_ms(100);
10
}
11

12
void dot() {
13
  beep();
14
  rest();
15
}
16

17
void dash() {
18
  beep();
19
  beep();
20
  beep();
21
  rest();
22
}
23

24
void space() {
25
  rest();
26
  rest();
27
}
28

29
void space2() {
30
  space();
31
  space();
32
}

und
1
dot(); dot(); dot(); dot; space();    // H
2
dot(); space();                       // E
3
dot(); dash(); dot(); dot(); space(); // L
4
dot(); dash(); dot(); dot(); space(); // L
5
dash(); dash(); dash(); space();      // O
6
space2();
7
dot(); dash(); dash(); space();       // W
8
dash(); dash(); dash(); space();      // O
9
dot(); dash(); dot(); space();        // R
10
dot(); dash(); dot(); dot(); space(); // L
11
dash(); dot(); dot(); space();        // D

Ich bin mir nicht sicher, wo sich das Problem befindet, in der Software 
oder in der Hardware. Kann mir jemand sagen, ob dieses Phänomen in so 
einer Anwendung bekannt ist?

Grüsse
Martin
Angehängte Dateien:
Gast #3049197
Lesenswert?

Grausliges Programm.

ICR wird der Top-Wert des Zählers zur Frequenzerzeugung sein, wird die 
auf 0 gesetzt, ist die PWM nicht aus, sondern läuft auf Max-Speed, zwar 
im unhörbaren Bereich, aber eben mit Nebenwirkungen.

Zum Abchalten der PWM muss das OCR kontrolliert werden.
Gast #3049249
Lesenswert?

Erstmal vielen Dank für die superschnellen Antworten. Das hatte ich 
glatt übersehen und war natürlich genau die Lösung:
1
void beep() {
2
  OCR1B = DEFAULT_VOLUME;
3
  _delay_ms(50);
4
  OCR1B = 0;
5
}

ICR1 wird jetzt beim Setup des Timers gesetzt. Das Resultat hört sich 
jetzt so an wie ich es haben wollte.

Die Variante von Matthias habe ich eben erst gesehen und noch nicht 
ausprobiert.

Gruss
Martin
Angehängte Dateien:

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren