Anhand eines Tasters entweder I²C oder SPI Steuerung beginnen

So habe das jetzt folgendermassen gemacht:

 pinMode(kipp_left, INPUT_PULLUP); //kippstellung links in pin30 INPUT_PULLUP->interner pullup, somit pullup aktivieren
 pinMode(kipp_right, INPUT_PULLUP);//kippstellung rechts in pin31 und wird input-widerstand aktiviert (intern)

void loop()
{

left = digitalRead(kipp_left);
     right = digitalRead(kipp_right);
     if(left == LOW)
     {
       control_i2c(); //Aufruf der Funktion 
     }//endif left (i2c)
     /*Eventuell eine LED, falls control-i2c blink, falls nicht leuchtet bzw bleibt aus */
     
     if (right== LOW)
     {
       control_spi();
     }
}

müsste ok sein, oder seht ihr fehler?