Arduino com contador

Não consigo entender o que tem de errado nesse código, era para eu apertar s1 uma vez e m1 ficar ligado porém ele desliga depois de um tempo. O programa também não conta como deveria.

A questão é essa:
No sistema abaixo quando uma peça é colocada na posição dada pelo sensor s1 o motor m1 é ligado, levando a peça até o sensor s2 e caindo na caixa de saída. Sabe-se que a caixa suporta até 20 peças. Implemente um contador (cont) para que e somente quando 20 peças caírem na caixa o motor m1 é desligado e a lâmpada l1 é ligada, alertando ao operador da necessidade de trocar a caixa. Implemente um algoritmo que atenda o solicitado.

const int s1 = 6;

const int s2 = 7;

const int m1 = 8;

const int l1 = 11;

int cont = 0;

void setup() {

// put your setup code here, to run once:

  pinMode(s1, INPUT_PULLUP);

  pinMode(s2, INPUT_PULLUP);

  pinMode(l1, OUTPUT);

  pinMode(m1, OUTPUT);

  Serial.begin(9600);

}

   void loop(){

  while(digitalRead(s1) == HIGH){

    Serial.print("Valor de S1 = ");

    Serial.println(s1);

 }

  digitalWrite(m1, HIGH);

  if((digitalRead (s2) == LOW) && (cont < 20)){

    cont ++;

    Serial.print("número de vezes em que o botão foi pressionado:");

    Serial.println(cont);

    }

  else{

  digitalWrite (m1, LOW);

digitalWrite (l1, HIGH);

 delay(5000); //tempo para o LED permanecer acesso

 digitalWrite (l1, LOW);

 }

 }