Wo liegt mein Denkfehler

[Anfänger (Gast)]

Hi,

ich habe folgendes kleines Programm geschrieben:

void Schalter1 (void)
{

  PORTC &= ~(1<<PORTC0); // Bit 0 loeschen, LED an
  _delay_ms(2000);
  PORTC |= (1 << PORTC0);  // Bit 0 setzen, LED aus
}
void Schalter2 (void)
{
  PORTC &= ~(1<<PORTC1); // Bit 0 loeschen, LED an
  _delay_ms(2000);
  PORTC |= (1 << PORTC1);  // Bit 0 setzen, LED aus} 
}


int main(void)
{
  
  // Datenrichtung festsetzen
  DDRB = 0x00;  // alles Eingänge
  PORTB = 0xFF; // alle PullUps einschalten
  
  DDRC = 0x3F;  // Pin 0-5 Ausgänge!
  
  // Interrupts einschalten
  // sei ();
  
  PORTC = 0xFF;  // alle LEDs aus
    
  while (1)
  {
    if (PINB & (0<<PINB0)) 
    {
      Schalter1();
    }
    
    if (PINB & (0<<PINB1))
    {
      Schalter2();
    }    
  } // Ende while
}

Wenn ich PINB0 mit Masse verbinde, dann müsste die LED an PINC0 für 2 
Sekunden aufleuchten und dann wieder ausgehen.

Ich hab mich dabei an die Angaben aus dem Tutorial gehalten:

DDRB = 0x00, alles Eingänge
PORTB = 0xFF, alle internen PULL-Ups an

Leider blinkt die LED an PORTC0 nicht auf, wenn ich PINB0 mit Masse 
verbinde.

[Karl H. (kbuchegg)]

Anfänger schrieb:

>     if (PINB & (0<<PINB0))

Eine 0 kannst du links/rechts schieben sooft du willst, das bleibt immer 
0.

Das steht also mehr oder weniger

    if( PINB & 0 )

was wiederrum zu

    if( 0 )

zusammenkürzt.


   if( PINB & ( 1 << PB0 ) )
um abzufragen, ob ein Pin 1 ist

oder eben

   if( !(PINB & ( 1 << PB0 ) )
um abzufragen, ob der Pin 0 ist.

Aber wie du es auch drehst und wendest, es heisst immer 1<<irgendwas. 
Denn 0<<irgendwas ist nichts anderes als eine komplizierte Art und Weise 
um 0 zu schreiben.

[mr. mo (Gast)]

Anfänger schrieb:
> void Schalter1 (void)
> {
>
>   PORTC &= ~(1<<PORTC0); // Bit 0 loeschen, LED an
>   _delay_ms(2000);
>   PORTC |= (1 << PORTC0);  // Bit 0 setzen, LED aus
> }

Ist das nicht PC0 und nicht PORTC0? Bekommst du Warnungen oder sonst was 
ausgegeben?

Anfänger schrieb:
> if (PINB & (0<<PINB0))
>     {
>       Schalter1();
>     }

Hab das ein bisschen anders in Erinnerung:

#include <avr/io.h>
...
/* Fuehre Aktion aus, wenn Bit Nr. 1 (das "zweite" Bit) in PINC gesetzt (1) ist */
if ( PINC & (1<<PINC1) ) {
  /* Aktion */
}
 
/* Fuehre Aktion aus, wenn Bit Nr. 2 (das "dritte" Bit) in PINB geloescht (0) ist */
if ( !(PINB & (1<<PINB2)) ) {
  /* Aktion */
}
...

[Anfänger (Gast)]

schreib ich in die Schleife nur rein

PORTC = PINB;

dann geht die Lampe0 an, wenn ich an PORTB mit Masse verbinde.

[Anfänger (Gast)]

mr. mo schrieb:
> if ( !(PINB & (1<<PINB2)) )

Ja stimmt, jetzt wo Du es mir schreibst, wird es mir klar. Das erste mit 
dem Bit verschieben mit << in der Klammer und da eine Null rein, dass 
ergibt keinen Sinn. Jetzt mit dem NOT-Operator in der If-Abfrage klappt 
es :-) Vielen Dank für Deine Hilfe.