Sound-Probleme mit dem Demoprogramm in C + kurze Fragen

[Manni2k]

Hallo,

1)

das RN-Control Demoprogramm in C sorgt bei mir dafür, dass sich der Sound schrecklich anhört, in folgendem Sinne:
Er ist sehr lang und grell und unschön - sicher nicht gewollt - ich konnte ihn ja mit der Startmelodie von dem Bascom-Demoprogramm vergleichen, was anfangs dort drauf war.
Ich habe mir mal die sound()-Funktion kurz angeschaut und festgestellt, dass der Aufruf "sound(6, 270);" (ein Teil der Melodie) insgesamt etwa 4 Sekunden wartet durch die beiden Aufrufe von "_delay_ms(hoehe);" - das heißt die Startmelodie dauert durch 7x Aufrufe von sound() etwa 28 Sekunden ?! Ist das gewollt ?! Irgendwas muss da doch faul sein? Ich kann den Sound gerne mal aufnehmen und rein stellen

2)

Das RN-Control Demoprogramm in C reagiert nicht auf den Reset-Taster - muss man den selbst im Code ansprechen und darauf reagieren oder ist der direkt am Mikrocontroller angeschlossen, sodass dieser sich unabhängig vom Programm-Code IMMER resettet? Letzteres kann eigentlich nicht sein, denn das LED-Lauflicht wird durch das Drücken des Reset-tasters nicht unterbrochen...

3)

Noch eine kurze Frage zum RN-Control:
Der An/Aus-Knopf fehlt komplett, oder? Hat das einen speziellen Grund? Ich meine, ist ja schon blöd, wenn Batterien dran hängen.

Vielen Dank & Viele Grüße,
Sören

[Manni2k]

Zu 1):
Ich habe mal "_delay_us" statt "_delay_ms" genommen - und schon kommen da jedenfalls brauchbare, nicht mehr klirrende, wackelige Töne raus, sondern hohe Piepser - nur ist die Startmelodie immernoch unharmonisch und nicht "originale" - und die Töne werden, je nach Parameter "hoehe" unterschiedlich lange abgespielt.

[Max Web]

Hallo,

Zu 1): Das würde mich auch sehr interessieren, ich selbst habe es noch nicht hinbekommen. Auch mit der im Internet kursierenden delay_x nicht umgehbar.

Zu 2): Der Reset-Taster zieht den Reset-Pin am Mega32 auf Masse, dadurch startet der Mega32 neu.

Zu 3): Einen solchen Knopf hat das Board nicht. Man kann aber auch einfach einen Schalter anschließen.

[Manni2k]

Danke erstmal für deinen Beitrag.
Zu 2): Das dachte ich mir auch, hat auch wunderbar funktioniert bei dem BASCOM-Demoprogramm was anfangs drauf war, allerdings reagiert das C-Demoprogramm gar nicht mehr auf den Reset-Knopf - wie schaut es bei dir/euch aus?

[Hubert.G]

Das mit dem Demo wurde hier schon einige mal behandelt.
Es hängt damit zusammen das dem _delay_ms keine Variable übergeben werden darf, nur Konstante sind zugelassen, siehe avr-libc-user-manual.
Umgehungsmöglichkeit mit while.

[Manni2k]

Das mit dem Demo wurde hier schon einige mal behandelt.

Das ist schön. Aber warum integriert man die Diskussion nicht dahin, wo sie hingehört: Und zwar in die Diskussions-Seite über das C-Demoprogramms? Oder warum wird das Demoprogramm nicht mal angepasst, wenn Fehler drin sind? Ich würde es ja selber auch machen, wenn ich denn eine Lösung parat hätte bzw. das Problem verstehen würde, wo wir gleich beim nächsten Punkt sind:

Es hängt damit zusammen das dem _delay_ms keine Variable übergeben werden darf, nur Konstante sind zugelassen, siehe avr-libc-user-manual.

Ich habe mal in der Doku nachgeschaut, dort steht zu "_delay_ms" folgendes:

Perform a delay of __ms milliseconds, using _delay_loop_2().

The macro F_CPU is supposed to be defined to a constant defining the CPU clock frequency (in Hertz).

The maximal possible delay is 262.14 ms / F_CPU in MHz.

When the user request delay which exceed the maximum possible one, _delay_ms() provides a decreased resolution functionality. In this mode _delay_ms() will work with a resolution of 1/10 ms, providing delays up to 6.5535 seconds (independent from CPU frequency). The user will not be informed about decreased resolution.

Wo steht, dass nur Konstanten erlaubt sind?
Leider scheint es auch nicht daran zu liegen: Ich habe mal eine Konstante eingesetzt, und das Problem besteht immer noch. Außerdem nochmal: Wenn delay_ms wirklich stimmen sollte und das Grundgerüst auch, so würde die Startmelodie über 20 Sekunden dauern (s.o.) - das wird so nicht richtig sein! Und nochmal: Nach einem Wechsel auf "delay_us" (Also WENIGER warten -> Mikro- statt Millisekunde) kommen bei mir jedenfalls brauchbarere Töne raus.

Kann wer, der diesen Piepser brauchbar in C ansprechen kann, nicht mal seinen Code veröffentlichen?

Vielen Dank nochmal für jegliche Mühe!

Viele Grüße,
Sören

[Hubert.G]

Das steht in der GCC Referenz Lib

In order for these functions to work as intended, compiler optimizations must be enabled, and the delay time must be an expression that is a known constant at compile-time. If these requirements are not met, the resulting delay will be much longer (and basically unpredictable), and applications that otherwise do not use floating-point calculations will experience severe code bloat by the floating-point library routines linked into the application.

Wie hast du denn die Fuses deines Mega32 eingestellt?

[Manni2k]

Ah okay, danke für das Zitat.

Die Fuses- und Lockbits habe ich unverändert gelassen, die waren bei Auslieferung ja schon richtig konfiguriert, nicht?
Die Frequenz passt auch, das Lauflicht läuft genauso, wie es soll.

[Manni2k]

Ich denke, dass ich es jetzt hinbekommen habe - leider fehlt mir die Original-Bascom-Startmelodie, aber diese hier klingt jetzt jedenfalls auch schön.

Verwendet habe ich die Basis-Delay-Funktion _delay_loop_2(i), welche bei 16MHz und i=1 etwa (+/- Setup-Delay) 0,25 µs wartet.

Ich ersetze in der sound()-Funktion lediglich jeweils _delay_ms(hoehe); durch _delay_loop_2(150*hoehe); - was dann in seiner Gesamtheit folgendermaßen aussieht:

void sound(uint8_t hoehe, uint16_t laenge)
{
	for(uint16_t i=0; i<laenge*15; i=i+(2*hoehe))
	{
		setportdon(7);
		_delay_loop_2(150*hoehe);
		setportdoff(7);
		_delay_loop_2(150*hoehe);
	}
}

Man kann sich mit Hilfe von void _delay_loop_2 (uint16_t __count) auch eine zweite waitms-Funktion bauen, die folgendermaßen aussieht (erfolgreich getestet bei z.B. 3s-Wartezeit):

void waitms2(uint16_t ms)
{
	for(; ms>16; ms-=16)
	{
		_delay_loop_2(64000); // 64000 = 16 * 4000
	}

	_delay_loop_2(4000*ms);
}

Kann mir jemand bitte bestätigen, dass es wie hier beschrieben dann funktioniert?

Den Sound kann man mit der Konstante 150 noch ein wenig variieren.

Ich würde dann auch gerne das C-Demoprogramm modifizieren wollen, wohl wissend, dass es insgesamt vlt. sinnvoller wäre, Timer zu verwenden - aber das würde für das Demoprogramm wohl zu weit gehen. Ich denke, dass diese kleine Änderung (im Prinzip nur die beiden Zeilen) okay sind - oder?

Viele Grüße,
Sören

[Manni2k]

2) auch gelöst: Wenn der ISP-Programmer angesteckt ist, funktioniert der Reset-Button nicht - nimmt man ihn raus, reagiert das Programm wieder auf den Reset. Mir auf die Schnelle schleierhaft, aber egal