Laufwerkmusik geht doch nicht

[J. T. (chaoskind)]

MoinMoin

ich mal wieder. Nachdem es bis gestern nun lief, und ich 8 
unterschiedliche Töne gleichzeitig ausgeben konnte, geht es nun wieder 
nicht mehr.

Nun lädt er mir meine neuen Werte nicht mehr ins Tonregister ?!?!?!

int main(void)
{
    uint8_t i = 0;
    uint8_t n = 0;
    
    for (i = 0; i < 8; i++)
    {
        Kanal.Tonhoehe[i] = Melodie1[0].Tonhoehen[i];
    }
    
    init();    
    TEMPO = (45000/BPM);
    
    
    while(1)
    {
        //while (n < 20)                          //Prüfen ob Anzahl gespielte Töne kleiner als Liste
       // {
             if (SysTick_64tel == Melodie1[n].Wechsel_bei64tel)         //"Warten" bis es Zeit für den nächsten Tonwechsel ist
            {
                n++;                                                    //"Pseudozeiger" auf die nächste Toneinstellung setzen
            
                for (i = 0; i < 8; i++)
                {
                    Kanal.Tonhoehe[i] = Melodie1[n].Tonhoehen[i];       //Die neuen Tonhöhen übergeben
                }
            }
       // }
        
    if (n >= 20)                          //Falls alle Töne gespielt, wieder auf Anfang setzen
    {    
        n = 0;
      SysTick_64tel = 0;
    }
    }
}

obwohl die Werte für Systick_64tel steigen, und auch im 
Melodie1[n].Wechsel_bei64tel die richtigen Werte stehen, wird der 
Vergleich nie ausgeführt, bzw der Vergleich wird nie als gleich erkannt, 
und somit auch keine neuen Werte in Kanal.Tonhoehe geladen....

Im Anhang nochmal der gesamte Quelltext. Wie gesagt, bis gestern ging 
es, und eigentlich hab ich nichts geändert..... ?!?!?!?!

MfG Chaos

[dummy (Gast)]

>Wie gesagt, bis gestern ging
>es, und eigentlich hab ich nichts geändert..

Dann nimm halt den Code von gestern.

[J. T. (chaoskind)]

Wär der nicht durch den von heute überspeichert worden, hätte ich das 
sicher schon gemacht....

[lex (Gast)]

Sei froh dass dir das passiert ist.
Du hast einen Fehler gemacht, der jeden Anfänger mal passiert.

Jetzt ist der Zeitpunkt, sich über eine Versionsverwaltung Gedanken zu 
machen (z.B. git). Danach willst du nie wieder ohne entwickeln.

[J. T. (chaoskind)]

so so git.. noch nie von gehört, gleich mal Tante Google fragen.

und um es nochmal zu verdeutlichen

Systick_64tel und Melodie1[n].Wechsel_bei64tel stehen beide auf 32, aber 
dennoch wird der Teil in den geschweiften Klammern nicht ausgeführt.

if (Systick_64tel == Melodie[n].Wechsel_bei64tel)
{
   neue Töne laden
}

aber die neuen Töne werden nicht geladen.


Woran kann das liegen?

[dummy (Gast)]

volatile uint16_t SysTick_64tel = 0;

[J. T. (chaoskind)]

erstmal Feierabend, das mit Volatile werd ich direkt zuhause versuchen.

Ich war eigentlich auch der Meinung, in der ISR muss es volatile sein, 
aber gestern gings auch ohne..  Meld mich nachher nochmal.

[J. T. (chaoskind)]

Jau danke dir, mit volatile gehts wieder...


Nun wäre natürlich noch intressant zu wissen, wieso es gestern ohne 
ging...
Aber das wird ohne den Quelltext vermutlich nicht mehr zu rekonstruieren 
sein

MfG Chaos

[Falk B. (falk)]

@ j. t. (chaoskind)

>Jau danke dir, mit volatile gehts wieder...

>Nun wäre natürlich noch intressant zu wissen, wieso es gestern ohne
>ging...

Wahrscheinlich weil die Optimierung im COmpiler ausgeschaltet war.

[J. T. (chaoskind)]

AAAHHHHHHH!!!! klar!

Ich hab zuhaus die Optimierungen aus, und am andren Rechner hab ich nur 
das c-file geladen (per mail geschickt, die USB-Sticks gehen entweder 
viel zu leicht verloren, oder sind viieeel zu schwer :D) und ein neues 
Projekt erstellt.

Dann ließ sich ja doch alles rekonstruieren!! =)

vielen Dank für deine Aufklärung, du hast meinen Glauben an die Logik 
und all den Krams wiederhergestellt.

MfG Chaos

[Bronco (Gast)]

Kleiner Tip:
Wenn Du noch nicht mit einer Versionsverwaltung arbeitest, dann packe 
Dir jeden funktionsfähigen Stand (das ganze Projektverzeichnis) einfach 
komplett in ein ZIP und gib diesem einen eindeutigen Namen.
Ist zwar nicht mit einer Versionierung zu vergleichen, aber wenigstens 
hast Du dann Dein altes Zeug noch...

[J. T. (chaoskind)]

Auf die Idee bin ich auch gekommen, und mach das nun vorerst so. Bis ich 
mal die Zeit finde, mich in diese Versionsverwaltungsgeschichte 
reinzufuchsen.

Vor Änderung einer laufenden Version wird die letzte lauffähige 
gesichert.

[Takao K. (takao_k)]

Wieso musik mit diskettenlaufwerk?

Die Laustaerke ist doch wohl nur gering und Bass geht nicht besonders 
gut.

Ausserdem braucht es viel Platz.