8051 Ride7 Delay-Funktion

[Nil (nilsnilss)]

Ich brauch für ein simples Programm eine ganz einfache Delay-Routine, 
die nicht besonders genau sein soll, sondern einfach ca. X ms 
Verzögerung im Programm hervorrufen soll.
Unten ein kurzes Test-Programm welches einfach im Sekundentakt die LEDs 
an P2 blinken lassen soll.
Leider passiert gar nichts.
Werden die beiden Schleifen einfach wegoptimiert und wenn ja, was kann 
ich dagegen tun?

Ah und der Controller läuft übrigens mit 1Mhz (12Mhz Quarz, Takt wird 
intern aber noch durch 12 geteilt).

#include <at89c5131.h>
#include <intrins.h>

void delay_ms(int k)
{
    int i=1000;  

    while(--k)
    {
        while(--i)
        {
            _nop_();
        }
        
    }
}

int main(void)
{ 
    while(1)
    {
        delay_ms(1000);
    
        P2 = 0xFF;
    
        delay_ms(1000);
    
        P2 = 0;
    }

    return 0;
}

[Bernd N (Gast)]

volatile int i=1000;

[Nil (nilsnilss)]

Daran hab ich auch schon gedacht, bringt leider keinerlei Änderung.

[Georg G. (df2au)]

Und denk dran, dass dein i nur beim ersten Durchlauf der Schleife auf 
1000 gesetzt wird. Danach ist es 0 und die innere Schleife dauert 
entsprechend länger.

[Georg G. (df2au)]

Nils Friess schrieb:
> Werden die beiden Schleifen einfach wegoptimiert

Das sollte ein schneller Blick auf den Object Code zeigen.

[jjflash (Gast)]

1.) volatile ist zwingend notwendig, da ansonsten der Compiler das 
tatsächlich "wegoptimiert" ... (war mal ein Fehler bei meinen 
Auszubildenden)

2.) würdest du mit einem Oszilloskop an den Port 2 gehen würdest du wohl 
höchstwahrscheinlich was sehen. Welche Frequenz hast du dort ?

[jjflash (Gast)]

Georg G. schrieb:
> Und denk dran, dass dein i nur beim ersten Durchlauf der Schleife auf
> 1000 gesetzt wird. Danach ist es 0 und die innere Schleife dauert
> entsprechend länger.

volatile int i;

i = 1000;

So ist die beim Eintritt immer 1000 (nehm ich zumindest mal an ... )

[Nil (nilsnilss)]

Oszilloskop hab ich keins zur Verfügung.
Hier mal das Listing. Also für mich sieht das nicht wirklich nach großer 
Optimierung aus. Die Funktion wird quasi eins zu eins übersetzt.

            ; FUNCTION _delay_ms (BEGIN)
              ; Register R6R7 is assigned to parameter k
              ; R2R3 is assigned to i
                                           ; SOURCE LINE # 6 
0000 7A03           MOV    R2,#003H
0002 7BE8           MOV    R3,#0E8H
0004         ?WHILE2:
                                           ; SOURCE LINE # 8 
0004 EF             MOV    A,R7
0005 1F             DEC    R7
0006 7001           JNZ    ?LAB3
0008 1E             DEC    R6
0009         ?LAB3:
0009 EE             MOV    A,R6
000A 4F             ORL    A,R7
000B 600C           JZ     ?NXT2
000D         ?WHILE3:
                                           ; SOURCE LINE # 10 
000D EB             MOV    A,R3
000E 1B             DEC    R3
000F 7001           JNZ    ?LAB4
0011 1A             DEC    R2
0012         ?LAB4:
0012 EA             MOV    A,R2
0013 4B             ORL    A,R3
0014 60EE           JZ     ?WHILE2
                                           ; SOURCE LINE # 12 
0016 00             NOP    
0017 80F4           SJMP   ?WHILE3
0019         ?NXT2:
                                           ; SOURCE LINE # 16 
0019 22             RET    

            ; FUNCTION _delay_ms (END)

[Nil (nilsnilss)]

Habs jetzt mal mit For-Schleifen probiert, den Wert von i ein wenig 
variiert und siehe da: es klappt (einigermaßen genau)

#include <at89c5131.h>
#include <intrins.h>

sfr at P2 A;

volatile int i;

void delay_ms(int k)
{
    for(k; k>0; k--)
    {
        for (i = 50; i>0; i--)
        {
            _nop_();
        }
        
    }
}

int main(void)
{ 
    A = 0x12;

    while(1)
    {
        delay_ms(1000);
    
        A = 0xFF;
    
        delay_ms(1000);
    
        A = 0;
    }

    return 0;
}

[Georg G. (df2au)]

jjflash schrieb:
> So ist die beim Eintritt immer 1000 (nehm ich zumindest mal an ... )

Soweit richtig. Aber nach dem ersten Durchlauf durch die innere Schleife 
wird sie nicht wieder mit 1000 initialisiert sondern mit 0, entsprechend 
65000. Deine Sekunde Verzögerung wird also in erster Näherung eine 
Minute sein. Probiere es noch einmal und hab einfach viiiiiieeeeeeel 
Geduld.

[nils (Gast)]

Aaah jetzt hat's Klick gemacht, werd ich mal ausprobieren sobald ich 
wieder zuhause bin.

[jjflash (Gast)]

Georg G. schrieb:
> Aber nach dem ersten Durchlauf durch die innere Schleife
> wird sie nicht wieder mit 1000 initialisiert sondern mit 0,

stimmt