Fehler suche

[amal]

#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdint.h>
#include <util/delay.h>
#include <stdio.h>
#include <stdlib.h>





// PD0 = Taster für Richtung für alle Motoren
// PD1 = Taster für Start
// PD2 = Taster für Interrupt0 
// PD3 = Taster für Interrupt1 
// PD4 = Taster für Stop
// PD5 = Taster für Reset

// PB4 = Richtung für alle Motoren
//******************************
// MOTOR 1 Richtung X
//******************************
// PB0 = Enable M1
// PB1 = Clock M1 

//******************************
// MOTOR 2 Richtung Y
//******************************
// PB2 = Enable M2
// PB3 = Clock M2


// PC0 = Trigger
// PD2 = Interrupt0 
// PD3 = Interrupt1 

volatile uint16_t zx,zy,D;


void pause(D)
{
    for(uint16_t  i=0;i<D;i++)
     asm volatile ("nop");
}
void STOP()
{
 
    
    PORTB&=~(1<<PB0);//Mx En löschen
    PORTB&=~(1<<PB2);//My En löschen
    
}

ISR (INT0_vect)    
{                

        PORTB|=(1<<PB0);//Mx En
        PORTB|=(1<<PB2);//My En
        PORTB|=(1<<PB4);// Richtung 

        for(unsigned int i=0;i<10;i++)
        {zx=0;
         zy=0;


            for(unsigned int j=0;j<10;j++)
            {
                for(unsigned int k=0;k<10;k++)
                     {
                        
                        PORTB|=(1<<PB1);  //
                        pause(1000);   //  Clk Mx
                        PORTB&=~(1<<PB1); //
                        zx++;
                    }

                PORTC|=(1<<PC0);        //
                pause(2000);            // Triggern 5hz
                PORTC&=~(1<<PC0);        //

            }
            for(unsigned int l=0;l<10;l++)
            {
                for(unsigned int m=0;m<10;m++)
                    {
                    PORTB|=(1<<PB3);    //
                    pause(1000);        // Clk My
                    PORTB&=~(1<<PB3);    //
                    zy++;
                    }
                PORTC|=(1<<PC0);    //
                pause(2000);        // Triggern 5hz
                PORTC&=~(1<<PC0);    //
            }
            PORTB&=~(1<<PB4);// Richtung ändern
            for(unsigned int n=0;n<10;n++)
            {
                for(unsigned int p=0;p<10;p++)
                    {
                    PORTB|=(1<<PB2);  //
                    pause(1000);   // Clk Mx
                    PORTB&=~(1<<PB2); //
                    zx--;
                    }
                
                PORTB|=(1<<PC0);
                pause(2000); //Triggern 5hz
                PORTB&=~(1<<PC0);
              }
                
            PORTB|=(1<<PB4);// Richtung 

            for(unsigned int r=0;r<10;r++)
            {
                for(unsigned int s=0;s<10;s++)
                    {
                    PORTB|=(1<<PB3);    //
                    pause(1000);        // Clk My
                    PORTB&=~(1<<PB3);    //
                    zy++;
                    }
                PORTC|=(1<<PC0);    //
                pause(200);        // Triggern 5hz
                PORTC&=~(1<<PC0);    //
            }

    
    
        
}


ISR (INT1_vect)                    
    
{        
        void STOP();
        pause(1000);
        PORTB|=(1<<PB0);//Mx En
        PORTB&=~(1<<PB4);// Richtung 
        if(zy>0)
        {
        for(unsigned int q=0;q<zy;q++)
            {
            PORTB|=(1<<PB3);    //
            pause(1000);        // Clk My
            PORTB&=~(1<<PB3);    //    
            }
        }
        if(zx>0)
        {
        for(unsigned int q=0;q<zx;q++)
            {
            PORTB|=(1<<PB1);    //
            pause(1000);        // Clk Mx
            PORTB&=~(1<<PB1);    //    
            }
        }    
    


}






    
    





//Hauptprogramm START----------

int main(void)

{



    
    DDRD=0b11100000;                // PIN 0-4 als Eingang definieren
    DDRB=0xb11111111;                // PortB als Ausgang definieren
    DDRC|=(1<<PC0);                    // PC0 als Ausgang


    sei();                        //Interrupts freigeben

//Interrupt initialisieren
       EIMSK|=(1<<INT0)|(1<<INT1);
       EICRA|=(1<<ISC01)|(1<<ISC11);  







    
    while(1)                    //Endlosschleife
    {

   
    }
    
    return 0;
}

ich bekomme diese Fehlern,
../M_XY_vers2.c:133: error: static declaration of '__vector_2' follows non-static declaration
../M_XY_vers2.c:133: error: previous declaration of '__vector_2' was here


weißt bitte jemand wo der Fehler ist?
danke

[markusj]

Kann es sein, dass dir in INT0 eine schließende Klammer (die von der äußeren for-Schleife) abhanden gekommen ist? Bei sowas sind die nachfolgenden Fehlermeldungen oft nur bedingt Aussagekräftig

mfG
Markus

[Hubert.G]

Die Angabe welcher Kontroller und welcher Kompiler verwendet wird, erleichtert die Hilfestellung.
Diese Schreibweise im main führt möglicherweise auch zu einem Fehler: DDRB=0xb11111111; // PortB als Ausgang definieren

[PicNick]

möglicherweise verwirrt ihn das:

volatile uint16_t zx,zy,D;

void pause(D)
{


das ist "D" statisch definiert, aber auch als argument f. "pause"

[Hubert.G]

Das "D" ist als globale Variable definiert, wird auch nicht verwendet und gehört entfernt.

Da im Programm ein Wert an die Funktion übergeben wird, muss diese Variable in Funktionsaufruf definiert sein.
Das " void pause(D) " ergibt ein Warning, es soll so aussehen "void pause(unsigned int D)"

[amal]

Die Angabe welcher Kontroller und welcher Kompiler verwendet wird, erleichtert die Hilfestellung.
Diese Schreibweise im main führt möglicherweise auch zu einem Fehler: DDRB=0xb11111111; // PortB als Ausgang definieren

das war einen schreibfehler, eigentlich nur 0b11111111.
danke

[021aet04]

Statt 0b... kann man auch die Dezimalschreibweise oder Hexadezimalschreibweise nehmen.

Ich habe es am Anfang so gemacht, dass ich wenn ich einen ganzen Port o.Ä. setze zuerst die Binärschreibweise genommen habe (so wie du). Dann habe ich es in die Hex Schreibweise gewandelt und diese geschrieben. Die Umwandlung habe ich auf einem Zettel, den ich daneben gehabt habe gemacht.

Wenn du deine Binärschreibweise nimmst. Die einzelnen Schritte:
Ausgangskonfiguration hinschreiben (eventuell noch darüberschreiben, welcher Port gemeint ist (z.B. PB0-PB7)
Aufteilen in 2 Nibbles (4bit)
Zusammenrechnen
Als Hex Format hinschreiben

0b11111111
0b 1111  1111
ergibt
  8+4+2+1    8+4+2+1
     15        15
0x   f          f
 Ergebnis 0xff

Du kannst aber auch so schreiben. So kannst du leichter einzelne Pins zuweisen ohne herumrechnen.

DDRB |= (1<<PB0) | (1<<PB1)....(1<<PB7);

Einen Tipp habe ich noch (man sollte aber das händisch umrechnen können). Der Rechner von Windows kann Standartmäßig Hex- (0-15), Binär- (0,1), Oktal- (0-7) Dezimalzahlen (0-9) umrechnen. Bei älteren Versionen (z.B. 2k, XP,..) muss man Wissenschaftlich wählen. Bei z.B. Win 7 muss man Programmierer wählen. Bei Vista bin ich mir nicht sicher wie es da ist, müsste nachschauen.

MfG Hannes

[radbruch]

Hallo

Gelegentlich ist die binäre Schreibweise vorteilhafter. Um trotzdem den Überblick zu behalten verwende ich eine zusätzliche Kommentarzeile:

//     76543210
DDRB=0b11111111;

Gruß

mic

[amal]

stimmt, in INT0 fehlt ein Klammer,
danke