Interrupt löst dauernd aus

Ein Bit bzw. Flag bekomme ich nicht gelöscht.

"This bit is set when there is room in the transmit data buffer. It is 
cleared byreading SPI1S with SPTEF set, followed by writing a data value 
to the transmit buffer at SPI1D. SPI1S must be read with SPTEF = 1 
before writing data to SPI1D or the SPI1D write will be ignored. SPTEF 
generates an SPTEF CPU interrupt request if the SPTIE bit in the SPI1C1 
is also set. SPTEF is automatically set when a data byte transfers from 
the transmit buffer into the transmit shift register. For an idle SPI 
(no data in the transmit buffer or the shift register and no transfer in 
progress), data written to SPI1D is transferred to the shifter almost
immediately so SPTEF is set within two bus cycles allowing a second 
8-bit data value to be queued into the transmit buffer. After completion 
of the transfer of the value in the shift register, the queued value 
from the
transmit buffer will automatically move to the shifter and SPTEF will be 
set to indicate there is room for new data in the transmit buffer. If no 
new data is waiting in the transmit buffer, SPTEF simply remains set and 
no data moves from the buffer to the shifter.
0 SPI transmit buffer not empty
1 SPI transmit buffer empty"

Hier ein auszug aus dem Datenblatt.
Sobald ich keine neuen Daten mehr schreiben will, und das SPTEF Flag 
gesetzt bleibt, löst die ganze Zeit das Interrupt aus.

"SPTEF simply remains set..." Genau das versuche ich eigentlich...

Jemand eine Idee?


#Nachtrag:

1

__interrupt VectorNumber_Vspi1 void ISR_SPI() {

2

3

    uint8 Temp;

4

    uint8 Tempo1;

5

    uint8 Tempo2;

6

7

8

9

  //Flags speichern

10

  Tempo1 = SPI1S_SPRF;

11

  Tempo2 = SPI1S_SPTEF;

12

13

  //Flags löschen

14

  SPI1S_SPRF = 0;

15

  SPI1S_SPTEF = 0;

16

17

18

   if(Tempo1 == 1) {

19

20

      Temp = spi_receiveData();

21

   } 

22

   else if(Tempo2 == 1) 

23

   {

24

      //schreibbereit    

25

   } 

26

   else{

27

      //Fehler

28

   }

29

30

}

So versuche ich es mittlerweile, leider auch ohne Erfolg. Einfach 0 
setzen geht nicht.

1

void spi_enableInterrupt(bool b_Option)

2

{

3

  if(b_Option == true)

4

  {

5

    // SPI Interrupt einschalten

6

    SPI1C1 = SPI1C1|(SPI1C1_SPIE_MASK);

7

    SPI1C1 = SPI1C1|(SPI1C1_SPTIE_MASK);

8

9

    // Interrupts freigeben

10

    EnableInterrupts;

11

  }

12

  else

13

  {

14

    // SPI Interrupt abschalten

15

    SPI1C1 = SPI1C1&~(SPI1C1_SPIE_MASK);

16

  }

17

}

EnableInterrupts steht für sei().

Irgendjemand noch eine Idee?

Zur Info:
Controller MC9S08AW60 (freescale)

Amateur schrieb:
> Stichwort: volatile.

IO-Register werden in der Regel im *.h File als volatile gecastet.
Zum Lesezugriff reicht aus:

1

SPI1D;

M. N. schrieb:
> Das ist ja auch richtig, damit die Übertragung überhaupt gestartet wird.
> Nachdem das letzte Byte übertragen wurde, mußt Du den Interrupt
> abschalten.
>
> Das könnte so aussehen:
> spi_enableInterrupt(false);

Ok, dass wusste ich so nicht. Dann muss ich das Interrupt einfach 
ausschalten.

Bei der jetztigen Codeversion habe ich das Interrupt noch nicht 
ausgeschaltet, aber es funktioniert in dem Sinne von es schreibt die 
ganze Zeit.

1

void main(void) {

2

3

    spi_init();

4

    spi_setMode(Master, FallingEdge, LeadingEdge, MSBfirst);

5

    spi_selectChip(true);

6

    spi_enableInterrupt(true);

7

8

      for(;;)

9

      {

10

        __RESET_WATCHDOG();

11

12

          if( spi_checkComplete()==true) {

13

14

          SendeTakt(27);

15

          }

16

      }  

17

   } 

18

19

__interrupt VectorNumber_Vspi1 void ISR_SPI() {

20

21

    __RESET_WATCHDOG();

22

23

    if(sbufferpos<32) {

24

        sbufferpos = sbufferpos + 1;

25

26

           if((SPI1S&(SPI1S_SPRF_MASK)) == 0) {

27

           } 

28

           else{

29

              SPI1S;

30

              SPI1D;

31

           }

32

           if((SPI1S&(SPI1S_SPTEF_MASK)) == 0){  

33

           } 

34

            else{

35

              SPI1S;

36

              SendeTakt(27);

37

            }

38

    }

39

    else{

40

       sbufferpos = 0;

41

    }  

42

}

Hier die spi_checkComplete:

1

bool spi_checkComplete()

2

{

3

  uint8 Temp;

4

5

  if((SPI1S&(SPI1S_SPTEF_MASK)) == 0)

6

  {

7

    return false;

8

  }

9

  else

10

  {

11

    // SPI Data Buffer lesen um Interrupt Bit zu leeren

12

    Temp = SPI1D;

13

    return true;

14

  }

15

}

Den Watchdog sollte man ja eigentlich nciht im Interrupt feeden. Wenn 
ich es allerdings nicht mache, wird das Watchdoginterrupt nach etwa 20 
Durchläufen des Sendevorgangs ausgelöst. Ziemlich komisch, es werden 
keine Fehler angezeigt -> COP Watchdog caused a Reset

Hat hier noch jemand eine Idee?

Controller: MC9S08AW60 (freescale)
http://www.freescale.com/files/microcontrollers/doc/data_sheet/MC9S08AW60.pdf


Auf jeden Fall schon mal vielen Dank für die hilfreichen Antworten!

Ok, nur es ist ja ein SPI Interrupt.
Und wieso kommt der Fehler beim ca. 20. Durchlauf und nicht schon beim 
ersten?

Ich kann euch mitteilen, dass es so ist. Wieso wird bei mir leider 
schwer...

Aber sehe ich das richtig, dass man zwangsläufig in der ISR den 
Schreibvorgang starten muss.

Ein solches Konstrukt funktioniert nicht:

1

main(){

2

    while(sbufferpos < buffergöße)

3

    {

4

    SendeData(bufferpos);

5

    }

6

}

7

8

ISR()

9

{

10

    if(Transmissionbuffer == empty)

11

    {

12

    sbufferpos++; 

13

    } 

14

}

Wenn man es so machen würde, würde das Interrupt ja permanent ausgelöst 
werden, da das Transmissionbufferemptyflag nicht acknowledged wird. Sehe 
ich das richtig?