PIC XC8 Mehrere Registerbits auf einmal setzen

[Alex M. (Gast)]

Als AVR Umsteiger bin ich es gewohnt die Bits in den Registern 
folgendermaßen zu setzen:

PORTA=1<<PA1|1<<PA2;

Der Compiler optimiert es dann zur PORTA=6 und alle Bits werden 
gleichzeitig gesetzt.

Wenn ich mir die Tutorials zu PICs angucke treffe ich aber auf folgende 
Variante:

LATAbits.LATA1=1;
LATAbits.LATA2=1;

Dies wird zu einem Haufen Bit sets kompiliert, was Ausführungszeit und 
Platz kostet.

Im Moment behelfe ich mir so:

#define my_LATA0_BIT 0
#define my_LATA1_BIT 1
#define my_LATA2_BIT 2

LATA=1<<myLATA1bit|1<<myLATA2_BIT;

Was zwar zum gewünschtem Ergebnis führt, auf Dauer aber Zeitaufwändig 
und nervig ist.

Gibt es da vielleicht einen besseren Weg?

[Alex M. (Gast)]

Ich glaube mein Problem gelöst zu haben.

Ich habe folgende Makros erstellt:

#define BV(abit) BV2(abit)
#define BV2(addr,pos)   1<<pos

Damit wird aus der Definition aus dem Deviceheader

LATA2_bit           BANKMASK(LATA), 2

1<<2

Die Werte lassen sich dann mit gewünschten logischen Operationen 
verknüpfen und auf einmal in den Register schreiben.

[Jürgen D. (poster)]

Alex M. schrieb:
> PORTA=1<<PA1|1<<PA2;

Was spricht eigendlich dagegen gleich

PORTA=0x06;

zu schreiben?

[Alex M. (Gast)]

Jürgen D. schrieb:
> Alex M. schrieb:
>> PORTA=1<<PA1|1<<PA2;
>
> Was spricht eigendlich dagegen gleich
>
> PORTA=0x06;
>
> zu schreiben?

Ich mag es wenn es aus dem Code ersichtlich ist was die Bedeutung der 
Bits ist, die gesetzt werden.

[B. S. (bestucki)]

Alex M. schrieb:
> Ich mag es wenn es aus dem Code ersichtlich ist was die Bedeutung der
> Bits ist, die gesetzt werden.

Dann besser gleich Plattformunabhängig:

#define high 1
#define low 0
#define setLED0(x) (PORTAbits.RA0=(x))
#define setLED1(x) (PORTAbits.RA1=(x))

So musst du den Code nur an einer Stelle anpassen, wenn sich die 
Pinbelegung ändert und es ist ersichtlich, was du tust.

Hast du folgenden Codeschnipsel:

setLED0(high);
setLED1(low);

so wird der Compiler keine Optimierungen vornehmen und die Ausdrücke 
nicht zusammenfassen, da alle internen Register als volatile deklariert 
sind. Ansonsten müsstest du auf Schattenregister ausweichen.