Probleme mit Tasterschaltung

[Tuxist]

Es geht die Taster waren es unglaublich

[Tuxist]

so den code habe ich auch dar wo ich ihn hin haben möchte:

#define F_CPU 10000000UL
#include <avr/io.h>
#include <util/delay.h>


int channel0;
int channel1;


void PowerOn(){
/**********Start Powerleds*******************/
  DDRB |= 1<<PB0;      /* set PB0 to output */
  DDRB |= 1<<PB1;      /* set PB1 to output */
  PORTB |= (1 << PB0); /* LED0 on           */
  PORTB |= (1 << PB1); /* LED1 on           */
/********************************************/ 
}


void setPins(){
/**************Input*************************/
  DDRB &= ~(1 << PB2); /* set PB2 to input*/  
  DDRB &= ~(1 << PB3); /* set PB3 to input*/
/********************************************/
  
/**************Ouput*************************/  
  DDRD |= 1<<PD6;      /* set PB6 to output */
  DDRD |= 1<<PD7;      /* set PB7 to output */
  PORTD &= ~(1<<PD6);  /* PD6  off          */                     
  PORTD &= ~(1<<PD7);  /* PD7  off          */
/********************************************/
}


void eventhandler(){
/**********Read Input PB2 and PB3*************/ 
  if((PINB & (1 << PB2)) ) {
    channel0=1;
    channel1=0;   
  }
  
  if((PINB & (1 << PB3)) ){
    channel1=1;
    channel0=0;    
  }
  
  if(channel0==1){
    PORTD |= 1<<PD6;               /* LED on */
    PORTD &= ~(1<<PD7);            /* LED off */     
  }else if(channel1==1){
    PORTD |= 1<<PD7;               /* LED on */
    PORTD &= ~(1<<PD6);            /* LED off */
  }
}


int main(void) {
  setPins();
  PowerOn();
  channel0=1;
  while(1) {
    eventhandler();
  }
  return 0;
}

[sast]

ich würde mal sagen, nicht die Taster sind schuld, wenn sie nicht richtig angeschlossen sind.
Aber trotzdem schön, wenn es jetzt klappt.

Gute Nacht
sast

[MisterMou]

Es ist schön festzustellen, dass sich immer mehr für AVRs begeistern können.
Das wird ja fast zum Volkssport

Du machst einige unnötige Dinge in Deinem Code.

Dein Programm startet ja in der main(), zu diesem Zeitpunkt ist DDRB= 0000.0000 (binär).
In der setPins() setzt du dann die B2 und B3 0, obwohl sie es schon sind, tut also nicht not
Das selbe Spiel am Ende der setPins(), PORTD ist bereits 0000.0000 und muss nicht extra genullt werden.

Außerdem kannst Du Dir die beiden Variablen sparen

void eventhandler()
{
  if((PINB & (1 << PB2)) ) {
    PORTD |= 1<<PD6;               /* LED on */
    PORTD &= ~(1<<PD7);            /* LED off */    
  }
  
  if((PINB & (1 << PB3)) ){
    PORTD &= ~(1<<PD6);            /* LED off */     
    PORTD |= 1<<PD7;               /* LED on */
  
  }
  
}

Ein kleiner Tipp am Rande, diese schönen Kästchen im Code sind gruselig.
Fang am besten jetzt schon mit einem sauberen Schreibstiel an

[Calis007]

Ein paar ueberfluessige Klammern gabs da noch, und ich empfehle auch immer, die geschwungenen Block-Klammern genauso einzuruecken wie den Block selbst. Findet man dann viel leichter, wo eine fehlt - ist aber Geschmackssache

void eventhandler()
  {
  if (PINB & (1 << PB2))
    {
    PORTD |= 1<<PD6;             /* LED on */
    PORTD &= ~(1<<PD7);          /* LED off */    
    }
  
  if (PINB & (1 << PB3))
    {
    PORTD &= ~(1<<PD6);          /* LED off */     
    PORTD |= 1<<PD7;             /* LED on */
    }
  }