Zwei gleichwertige PIC Anweisungen, aber nur eine funktoniert

[David (Gast)]

Hallo

Von folgenden Funktionen zur Timerinitialisierung funktioniert 
seltsamerweise nur erstere. Ich entwickle mit MplabIDE.

Warum funktioniert die zweite Funktionnicht? Ich komme nicht dahinter.

Liebe Grüsse
David

void InitTimer1(void)
{
  T1CKPS1 = 1;
  T1CKPS0 = 1;
  TMR1ON = 1;

}



void InitTimer1(void)
{
   T1CON |= (T1CKPS1 | T1CKPS0 | TMR1ON);

}

[Marius W. (mw1987)]

Da ich mich nicht mit PIC auskenne: Wie sehen die #defines hinter den 
Bezeichnern aus?

Gruß
Marius

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

David schrieb:
> Warum funktioniert die zweite Funktion nicht?
Wie sind T1CKPS1,  T1CKPS0 und TMR1ON definiert?
Gibt es da evtl. eine Header-Datei, wo das genauer beschreiben ist?

[Hannes S. (herrhannes)]

Du setzt T1CON dauerhaft auf 1. Ist das deine Absicht?

[Steel (Gast)]

Weil die zweite Variante blödsinn ist? Ein gutes C-Buch hilft hier, wie 
so oft. Da wird nichts gesetzt in der 2ten Variante.

[Hannes S. (herrhannes)]

Mein Beitrag war Quatsch. Kann ihn nur leider nicht mehr ändern.
Steel schrieb:
> Weil die zweite Variante blödsinn ist? Ein gutes C-Buch hilft
> hier, wie
> so oft. Da wird nichts gesetzt in der 2ten Variante.

Sicher? Eventuell hat er das ja vorher irgendwo definiert.

[Justus S. (jussa)]

Hannes Stoll schrieb:
> Sicher? Eventuell hat er das ja vorher irgendwo definiert.

dann dürfte die erste Variante aber doch nicht funktionieren...entweder 
die erste oder die zweite, beide gleichzeitig können ja nicht richtig 
sein...

[Simon V. (Gast)]

Probiers mal damit: T1CONbis. T1CKPS1=1;

[Simon V. (Gast)]

Aber ohne Leerzeicheen:[c]T1CONbis.T1CKPS1=1;[c/]
T1CKPS1 ist nur als Zahl zwischen 0 und 7 definiert. Ich weiß die genaue 
Nummer des bits nicht auswendig.

[David (Gast)]

Hier ist die Begriffsdefinition aus der Include

/* Definitions for T1CON register */
static                bit  TMR1ON    @ ((unsigned)&T1CON*8)+0;
static                bit  TMR1CS    @ ((unsigned)&T1CON*8)+1;
static                bit  T1SYNC    @ ((unsigned)&T1CON*8)+2;
static                bit  T1OSCEN    @ ((unsigned)&T1CON*8)+3;
static                bit  T1CKPS0    @ ((unsigned)&T1CON*8)+4;
static                bit  T1CKPS1    @ ((unsigned)&T1CON*8)+5;
static                bit  T1RUN    @ ((unsigned)&T1CON*8)+6;

Ich habe gerade jetzt den Verdacht, dass z.B. T1CKPS0 nicht = 0x10 ist. 
Das müsste es für die nicht funktionierende Varianta ja sein.

Stattdessen ist es (Registeradresse *8)+4. Also die absolute Position im 
Speicher.

Stimmt das? Das könnte die Erklärung sein.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

David schrieb:
> static   bit  T1CKPS0    @ ((unsigned)&T1CON*8)+4;
Das ist ja mal eine wilde Herumcasterei und Rechnerei...   :-o

> die absolute Position im Speicher. Stimmt das?
Muss so sein, denn sonst würde die erste Variante nicht funktionieren.

[Klaus (Gast)]

David schrieb:
> Ich habe gerade jetzt den Verdacht, dass z.B. T1CKPS0 nicht = 0x10 ist.

Das wird wohl so sein. Aus

> T1CKPS0 = 1;

würde der PräProz ja sonst

0x10 = 1;

machen

MfG Klaus

[Flo (Gast)]

Ich weiß jetzt nicht welchen Compiler du verwendest.
Ich habe C18 und da geht das so, wenn man einzelne bits setzen, löschen, 
einlesen will:
Simon  V. schrieb:
> Probiers mal damit: T1CONbis. T1CKPS1=1;

[Bugs (Gast)]

Wie wärs mit:

void InitTimer1(void)
{
   T1CON |= ((1<<T1CKPS1) | (1<<T1CKPS0) | (1<<TMR1ON));

}

MfG Bugs