[kaczakat]

W loop masz wywołane do wykonania funkcje standardowe, a tych swoich nigdzie nie używasz, może w tym jest problem.
Czyli gdzieś tu powinno być wywołanie tych funkcji, np.:
void loop() {
buttonState1 = digitalRead(buttonPin1);
buttonState2 = digitalRead(buttonPin2);
LED1a();
}
I tak samo każda inna funkcja, albo wrzucasz ją do wykonania w SETUP, albo w LOOP, albo w inną funkcję, która jest wykonywana w tych sekcjach. Może też być wywołanie przerwaniem. Natomiast czy twoje funkcje mają jakikolwiek sens też nie wiadomo, program jest nierozłącznie związany ze schematem, którego tu nie pokazujesz. Stan niski na buttonPin może oznaczać wciśnięcie, albo jego brak, zależy co i jak się podłączy, jakie Arduino, jak się ustawi w setup uC. Stan niski na LED może oznaczać brak świecenia lub świecenie, zależy jak jest podłączony LED.
Kolejna sprawa, to szybkość działania, 16MHz oznacza, że uC wykonuje czynności w tempie 16mln/s. Czyli możesz nie zauważyć, że coś zostało włączone na 1us, a wciśnięty przycisk generuje tysiące niskich i wysokich stanów. Eliminuje się to odpowiednio programowo lub znowu schemat i jak masz do wszystko podłączone sprzętowo. Bez tego coś będzie działać, ale raczej nie tak jak się oczekuje.