UHR- if-bedingung

[Azubi (Gast)]

Hallo

Kann mir mal einer sagen wieso diese if-bedingung bei meiner Schaltung
plotzlich wahr wird :(

if ( !(PINC & ( 1 << PINC5)))
    {
      while (1)
      {
        PORTD = 0xFF;
      }
    }

[hilmar (Gast)]

Weil an PINC5 Low angelegt wird?

[Peter II (Gast)]

Wiel wichtiger ist doch die Frage, warum du uns eine C# Projekt 
angehangen hast.

[Azubi (Gast)]

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

int sekunde = 0;      //globale Variable sekunde
int stunde_a = 1;
int stunde_b = 3;

int minute_c = 4;
int minute_d = 3;



int main (void)
{
  int ausgabe[10];
  ausgabe [0] = 0x01;
  ausgabe [1]  = 0x37;
  ausgabe [2] = 0x42;
  ausgabe [3] = 0x12;
  ausgabe [4] = 0x34;
  ausgabe [5] = 0x18;
  ausgabe [6] = 0x08;
  ausgabe [7] = 0x31;
  ausgabe [8] = 0x00;
  ausgabe [9] = 0x10;
  
  DDRD = 0xFF;
  PORTD = 0xFF;
  
  DDRC = 0x0F;
  PORTC = ( 1 << PC4)|( 1 << PC5);    
  
  

  TCCR1B |= (1 << WGM12); 

  TIMSK |= (1 << OCIE1A);

  sei(); //  

  OCR1A   = 15624; 

  TCCR1B |= ((1 << CS10) | (1 << CS11)); 

  for (;;)
  {
    if ( !(PINC & ( 1 << PINC5)))
    {
      while (1)
      {
        PORTD = 0xFF;
      }
    }
    
    
    
    
    PORTC = ( 1<<PC0);
    PORTD = ausgabe[stunde_a];
    _delay_ms(5);
    
    PORTC = (1<<PC1);
    PORTD = ausgabe[stunde_b];
    _delay_ms(5);
    
    
    
    
    PORTC = ( 1<< PC2);
    PORTD = ausgabe[minute_c];
    _delay_ms(5);
    
    PORTC = (1 << PC3);
    PORTD = ausgabe[minute_d];
    _delay_ms(5);    
  }
}




ISR(TIMER1_COMPA_vect)
{
  sekunde++;
  
  if (sekunde == 60)
  {
    minute_d++;
    sekunde = 0;
    
    if (minute_d == 10)
    {
      minute_d = 0;
      minute_c++;
      
      if (minute_c == 6)
      {
        minute_c = 0;
        stunde_b++;
        if (stunde_b == 10)
        {
          stunde_b = 0;
          stunde_a++;
          
          if (stunde_a == 3)
          {
            stunde_a = 0;
          }
        }
      }
      
    }
    
  }

[Azubi (Gast)]

>Wiel wichtiger ist doch die Frage, warum du uns eine C# Projekt
>angehangen hast.


:(

[Peter II (Gast)]

> PINC & ( 1 << PINC5)

und

> PORTC = (1 << PC3);

passt nicht wirklich zusammen. Du darst nur die Ausgänge manipulieren.


Außerdem fehler noch die volatiles bei den ganzen ISR variablen.

[hilmar (Gast)]

Wenn C.5 auf GND (oder LOW) gelegt wird, schaltest du deinen PORTD auf 
0xFF (alles HIGH) und das machst du so lange, bis du dem µC den Strom 
wegnimmst.
Alles, was darunter steht, wird niemals erreicht.
Die ISR wird zwar arbeiten, aber die verändert ja nur stunde, minute, 
sekunden. Die Ausgabe erfolgt nie.

[amateur (Gast)]

PINC         | lese Port C | ??0????? oder ??1?????
( 1 << PINC5)| 00100000    | 00100000      00100000
&            |             | 00000000      00100000
             |             | False         True
!            |             | True          False

Ich würde es mal mit 0-Pegel an Pin C5 Probieren - oder auch nicht.

[Azubi (Gast)]

>Die ISR wird zwar arbeiten, aber die verändert ja nur stunde, minute,
>sekunden. Die Ausgabe erfolgt nie.

Wenn die Taste gedrückt wird soll keine ausgabe mehr erfolgen.

[Azubi (Gast)]

> PINC & ( 1 << PINC5)

>und

> PORTC = (1 << PC3);

>passt nicht wirklich zusammen. Du darst nur die Ausgänge manipulieren.

kannst du mir das bitte genauer erklären?

[Peter II (Gast)]

Azubi schrieb:
>> PORTC = (1 << PC3);
>
>>passt nicht wirklich zusammen. Du darst nur die Ausgänge manipulieren.
>
> kannst du mir das bitte genauer erklären?

wenn ein PullUP eingeschaltet ist, dann wird diese mit PORTC = (1 << 
PC3); wieder abgeschaltet.

Man sollte einfach nur das Bit ändern, was man braucht. Und nicht den 
kompletten port zuweisen.

[hilmar (Gast)]

>  DDRC = 0x0F;
>  PORTC = ( 1 << PC4)|( 1 << PC5);

Die Datenrichtung ist: Ausgang für C0 bis C3 und Eingang für C4 bis C7
C4 und C5 erhalten einen Pullup.

> wenn ein PullUP eingeschaltet ist, dann wird diese mit PORTC = (1 <<
> PC3); wieder abgeschaltet.

Ein Ausgang hat keinen Pullup!
C3 ist ein Ausgang, er wird durch "PORTC = (1 << PC3);" auf HIGH 
gesetzt.

[Azubi (Gast)]

>wenn ein PullUP eingeschaltet ist, dann wird diese mit PORTC = (1 <<
>PC3); wieder abgeschaltet.

Nur zum Verständnis:

PORTC = ( 1 << PC0); entspricht PORTC = b'0000 0001'

dadurch schalte ich mein Pull-up Widerstand ab, also muss ich
PORTC = ( 1 << PC0); durch PORTC = b'00100001' ersetzen um meinen 
Pull-up Widestand eben nicht abzuschalten.

Ich hoffe ich habe es jetzt richtig verstanden

[hilmar (Gast)]

DDRC ist das Datenrichtungs-Register. Dort, wo eine 1 steht, ist ein 
Ausgang, bei einer 0 ist das Bit ein Eingang.

Für die Eingangsbits: Wenn du auf PORTC schreibst, bedeutet ein 1-Bit, 
daß der Pullup eingeschaltet wird und ein 0-Bit, daß der Pullup aus ist.

Für die Ausgangsbits: Ein 1-Bit bedeutet, daß der Ausgang auf 1 (also 
HIGH) geht, ein 0-Bit, daß der Ausgang auf LOW geht.

Also erstmal sagen, was Eingang und was Ausgang ist mit DDRC.
Und dann bei Eingängen die Pullups und bei Ausgängen die Ausgangspegel 
setzen, ok?

[Karl H. (kbuchegg)]

hilmar schrieb:
>>  DDRC = 0x0F;
>>  PORTC = ( 1 << PC4)|( 1 << PC5);
>
> Die Datenrichtung ist: Ausgang für C0 bis C3 und Eingang für C4 bis C7
> C4 und C5 erhalten einen Pullup.
>
>> wenn ein PullUP eingeschaltet ist, dann wird diese mit PORTC = (1 <<
>> PC3); wieder abgeschaltet.
>
> Ein Ausgang hat keinen Pullup!
> C3 ist ein Ausgang, er wird durch "PORTC = (1 << PC3);" auf HIGH
> gesetzt.

Schon.
Was er sagen wollte.
Durch die Zuweisung wird nicht nur PC3 auf High gesetzt.
Als Nebeneffekt werden alle anderen BIts in PORTC auf 0 gesetzt. Und 
damit ist dann der schöne Pullup, der hier eingeschaltet wurde
>>  PORTC = ( 1 << PC4)|( 1 << PC5);
nur noch Geschichte.

[Azubi (Gast)]

>ok?

Danke, das habe ich schon verstanden

[Karl H. (kbuchegg)]

Azubi schrieb:
>>wenn ein PullUP eingeschaltet ist, dann wird diese mit PORTC = (1 <<
>>PC3); wieder abgeschaltet.
>
> Nur zum Verständnis:
>
> PORTC = ( 1 << PC0); entspricht PORTC = b'0000 0001'
>
> dadurch schalte ich mein Pull-up Widerstand ab, also muss ich
> PORTC = ( 1 << PC0); durch PORTC = b'00100001' ersetzen um meinen
> Pull-up Widestand eben nicht abzuschalten.
>
> Ich hoffe ich habe es jetzt richtig verstanden

Im Prinzip ja.
Nur mahct man das nicht so.
Du willst dir doch nicht ewig und immer und überall merken, welche 
anderen Pins du in welchem Portregister du zusätzlich auch noch mit 
berücksichtigen musst!

Hier

  TIMSK |= (1 << OCIE1A);

hast du es doch so schön geschrieben:
In TIMSK ausschliesslich und nur das OCIE1A Bit setzen. Alle anderen 
Bits bleiben so wie sie sind.

[Peter II (Gast)]

Azubi schrieb:
> dadurch schalte ich mein Pull-up Widerstand ab, also muss ich
> PORTC = ( 1 << PC0); durch PORTC = b'00100001' ersetzen um meinen
> Pull-up Widestand eben nicht abzuschalten.

ja aber nicht sehr komfortabel. du sollte einfach nur ein Bit setzen und 
die andere einfach nicht anfassen!

PORTC  |= ( 1 << PC0)

http://www.mikrocontroller.net/articles/Bitmanipulation

[Karl H. (kbuchegg)]

Ein Bit setzen

    Register |= ( 1 << Bitname );

Ein Bit löschen

    Register &= ~( 1 << Bitname );

ein Bit auf 1 abfragen

    if( Register & ( 1 << Bitname ) )

ein Bit auf 0 abfragen

    if( !(Register & ( 1 << Bitname )) )

Bitmanipulation

[Azubi (Gast)]

>PORTC  |= ( 1 << PC0)

jetzt dämmerts :D

[Azubi (Gast)]

danke euch allen fÜr die Hilfe