Zeitkritische Interrupt-Routine bei XC886

Muß die Funktion Set_CS_TDA5150(NOT_ACTIVE) unbedingt extern sein?
Damit zwingst Du den Compiler, alle 12 Register zu push/popen.
Besser also als Macro schreiben.

Was soll denn die Funktion überhaupt machen?
Wenn Du nur einen Pin setzen mußt, nimm besser Bitbefehle (Pin als 
sfr8).


Peter

Dein Kommentar

1

// set the data to pin 3 ...

legt eine Bitmanipulation des einen Pins nahe.
Warum beackerst du den ganzen Port?
Mach das doch im Stil des
> P1_DATA &= ~0x40;
also etwa so:

1

if( (TXFifo0[TXiterations] << BitsPerByte)& 0x80 ) 

2

   P1_DATA |=  0x8;

3

else 

4

   P1_DATA &= ~0x8;

EDIT:
Noch besser wäre es, nicht einen Zähler zu verwenden, sondern die Maske 
zu schieben, dann bleibt es bei einer Abfrage:

1

if (TXFifo0[TXiterations] & BitMask) 

2

   P1_DATA |=  0x8;

3

else 

4

   P1_DATA &= ~0x8;

5

6

if(BitMask>>=1) {   // after 1 byte is shifted out,      

7

   TXiterations++;  // increment TXiterations which is an index of an array and 

8

   BitMask = 0x80;  // reset the bit-mask which is necessary for the mask operation above

9

}

Du solltest mal versuchen mit Bitvariablen zu arbeiten, das macht vieles 
leichter (und schneller).

Ganz aufwendig ist ein variables Shift, nimm besser ne Maske, die man 
immer nur einmal schiebt.

Definier dochmal alle Variablen, damit man den Code compilieren kann, 
dann ist es viel einfacher zu optimieren.


Peter