Pls alle rein

[µC Looper (Gast)]

Hallo zusammen,
ich bin ganz neu in C Sachen
ich möchte PORTC PC0 auf 1 für 15 sek dann wieder auf 0 danach wieder 
auf 1 dann auf 0. Dazwischen soll eine dauert von 10 sec. Ich verwende 
Timer und Prescaler 64

while(1)
  {
    
    if (TIFR & (1 << OCF1A))
    {
      PORTB ^= ( 1 << PB0 ) | ( 1 << PB1 );
    
      ElapsedIntervall++;
      if (ElapsedIntervall >= (60 * TIMEINTERVAL))
      {
         //PORTB |= ( 1 << PB7 );
         //LongWait(100);
         //PORTB &= ~( 1 << PB7 );
         ElapsedDays++;
         
         PORTC |= ( 1 << PC0 ); 
         LongWait(ElapsedPressedButton);
         PORTC &= ~( 1 << PC0 ); 
         //LongWait(ElapsedPressedButton);
         
         LongWait(10000);
         
         PORTC |= ( 1 << PC0 ); 
         LongWait(ElapsedPressedButton);
         PORTC &= ~( 1 << PC0 ); 
         LongWait(ElapsedPressedButton);
         
         ElapsedIntervall = 0;
      }
    
      TIFR = (1 << OCF1A);
      
    }
    //if (ElapsedDays >= ((RUNDAYS * 86400) / (TIMEINTERVAL * 60)))
    //{
       //Sleep_Maker();
       //PORTB &= ~( 1 << PB0 ) | ~( 1 << PB1 );
       //PORTB ^= ( 1 << PB7 );
       //LongWait(100);
       //ElapsedDays = 0;
       
    //}
    
  }

[µC Looper (Gast)]

Ich bitte euch um Hilfe. Es funktioniert leider nicht
PC0 wirs auf 1 für 15 gesetzt dann nicht mehr

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

µC Looper schrieb:
> PORTC |= ( 1 << PC0 );
>          LongWait(ElapsedPressedButton);
>          PORTC &= ~( 1 << PC0 );

Wenn PC0 gesetzt wird und nie wieder gelöscht, dann würde ich mal diese 
Unterroutine mitposten. Die von gcc gelieferte ms_delay() Routine z.B. 
schluckt nur Konstanten und keine Variablen.
Wenn das übrigens wirklich so ein langes Warten ist, wird das Löschen 
von OCF1A mit Sicherheit zu spät kommen - aber das ist Raterei, wenn du 
die entscheidende Routine LongWait() nicht zeigst.

[mh (Gast)]

Die fehlenden Antworten liegen vermutlich daran, dass da nur ein 
Bruchteil des Codes steht und bergeweise Informationen fehlen. Die 
Aussage "Ich verwende Timer und Prescaler 64" bringt genauso viel, wie 
"Ich habe ein Auto".
Wie genau hast Du Deinen Timer konfiguriert. In welchen Intervall 
erwartest Du, dass er triggert, ...

Ausserdem steht da "LongWait" - kenn' ich nicht. Klingt aber nach 
busy-waiting, was per definition "böse" ist.

Ein paar ganz allgemeine Tips:
- Nie busy-waiting, immer StateMachine, ausser Du weisst ganz genau was 
Du tust.
- Lass Deinen Timer im Interrupt einen globalen Zähler hochzählen (z.B. 
im Millisekundentakt) und verwende diesen Zähler als globale Zeitbasis 
(manche Architekturen haben dafür einen SysTickCounter). Je nach Bedarf 
16 oder 32 bit Integer.
- Lass in der Main-Loop eine LED durchgehend blinken. z.B. mit

if ((sysTickCounter & 0xFF) == 0) toggleLed();

 Damit weißt Du dass Dein Timer läuft und Dein Programm nicht in 
irgendwelchen Interrupts oder Busy-Waiting-Kram (den Du ja eh nicht mehr 
machst) hängt.

Du wirst sehen, wenn Du diese Ratschläge befolgst, wirst Du Dein Problem 
nicht mehr haben...

[µC Looper (Gast)]

Der Code:

int main(void)
{
  unsigned long int ElapsedIntervall = 0;
  unsigned long int ElapsedDays = 0;
  unsigned long int ElapsedPressedButton = TIMEPRESSBUTTON;
  unsigned long int ElapsedSleepPressed= SLEEPTIME;
  //unsigned long int PulsCounter = 0;
  
  DDRB = 0x03;
  DDRC = 0x01;
  

  TCCR1B |= ( 1 << WGM12 );
  OCR1A   = 15624;
  TCCR1B |= (( 1 << CS10 ) | ( 1 << CS11 ));
  
  PORTC &= ~( 1 << PC0 );

  while(1)
  {
    
    if (TIFR & (1 << OCF1A))
    {
      PORTB ^= ( 1 << PB0 ) | ( 1 << PB1 );
    
      ElapsedIntervall++;
      if (ElapsedIntervall >= (60 * TIMEINTERVAL))
      {
         PORTC |= ( 1 << PC0 );
         LongWait(ElapsedPressedButton);
         PORTC &= ~( 1 << PC0 ); 
         //LongWait(ElapsedPressedButton);
         
         LongWait(1000);
         
         PORTC |= ( 1 << PC0 ); 
         LongWait(ElapsedPressedButton);
         PORTC &= ~( 1 << PC0 ); 
         
         ElapsedDays++;
         ElapsedIntervall = 0;
      }
    
      TIFR = (1 << OCF1A);
      //PORTC &= ~( 1 << PC0 );
      
    }
    //if (ElapsedDays >= ((RUNDAYS * 86400) / (TIMEINTERVAL * 60)))
    //{
       //Sleep_Maker();
       //PORTB &= ~( 1 << PB0 ) | ~( 1 << PB1 );
       //PORTB ^= ( 1 << PB7 );
       //LongWait(100);
       //ElapsedDays = 0;
       
    //}
    
  }
  
  return 0;
}

[Prophet (Gast)]

Bestümmpt ein PIC wo auf PC0 und PC1 ein Quarz erwartet wird.

Das muss man natürlich ausschalten. Dazu muss Man[n] dann
halt einmal ins Datenblatt schauen.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

µC Looper schrieb:
> DDRB = 0x03;
>   DDRC = 0x01;
>
>
>   TCCR1B |= ( 1 << WGM12 );
>   OCR1A   = 15624;
>   TCCR1B |= (( 1 << CS10 ) | ( 1 << CS11 ));

Naja, das riecht schon nach AVR. Aber ich sehe da nach wie vor nur ein 
main() und keinerlei Unterroutinen. Ausserdem wird PC0 ja gesetzt, aber 
nie wieder gelöscht, was die Theorie mit dem PIC unwahrscheinlich werden 
lässt.