No estoy seguro si este es tu problema, pero setPinValuesme parece raro:

void setPinValues(String message) {
  for(int i = 1; i <= sizeof(pins) / sizeof(pins[0]); i++ ) {
    String pinNumber = String(i);

    if( message == pinNumber + "H\n" ) {
      pinValues[i] = HIGH;
    }
    if( message == pinNumber + "L\n" ) {
      pinValues[i] = LOW;
    }
  }
}

Saliendo de los límites

Cada vez que llame setPinValues, ise extenderá de 1a length(pins), por lo que si se pinsdeclaró como pins[5], iserá 1,,, y . Suponiendo que esto es un problema en sí mismo, ¡porque ni siquiera existiría!2345length(pins) == length(pinValues)pinValues[5]

pinValues[5]solo crea pinValues[0]para (sí, sé que es extraño, está declarando 5 elementos pero, dado que C está indexado en 0 , pinValues[4]son 0, , , y ), por lo que NUNCA debe llegar a 5 o estará fuera de los límites, lo cual es comportamiento indefinido (y bien podría ser la fuente de sus problemas) porque está escribiendo en la ubicación de memoria incorrecta.1234i

La forma correcta de hacer esto es recorrer desde 0hasta i < sizeof(pins) / sizeof(pins[0])(observe el estricto operador menor que <) que va desde 0hasta length(pins) - 1y luego solo String pinNumber = String(i + 1);. De esa manera, 1Hcambiará el valor de pinValues[0], asignando sus mensajes en serie indexados en 1 a matrices C indexadas en 0.

El bucle es innecesario

No estoy seguro de por qué estás dando vueltas pins. Está haciendo un trabajo innecesario y probablemente acaparando el uC (especialmente porque, como señala Dave Tweed en la otra respuesta, está usando funciones costosas como Stringy comparación de cadenas con ==).

messagetiene una sola instancia, por ejemplo, 1H\npor lo que no se necesita un bucle. char pinNumber = message[0] - '0';almacenará el valor entero del primer carácter en pinNumber, con el que podrá indexar la matriz.

¿Cómo funciona?

• message[0]sería un chartipo (por ejemplo '1')
• '0'es también un char, que corresponde al valor ASCII de 0 (48 en decimal).
• Si message[0]contiene el char '0', entonces el resultado es'0' - '0' == 48 - 48 == 0
• Dado que los números son contiguos y aumentan en el conjunto ASCII, el resultado de la resta es el número real como un entero de un solo byte.
• '1' - '0' == 49 - 48 == 1y así sucesivamente.

También es posible que desee rechazar entradas falsas:

if (pinNumber >= sizeof(pins) / sizeof(pins[0]))
  return;

Más trabajo espurio

En cada loop()iteración haces esto:

// Write to pins        
for (int i = 1; i <= sizeof(pins) / sizeof(pins[0]); i++) {
  digitalWrite(pins[i], pinValues[i]);
}

Que tiene el mismo problema con los fuera de límites que antes.

También hace un trabajo innecesario, ya que está escribiendo TODOS los valores de los pines, ya sea que hayan cambiado o no. ¿Por qué no digitalWrite(pins[pinNumber], HIGH or LOW);entras directamente setPinValuesy evitas este bucle? Incluso podría deshacerse de la pinValuesmatriz si realmente no necesita almacenarla (o puede conservarla si la necesita por algún motivo).

Mi alternativa setPinValues

void setPinValue(char pin, char value) {
  char pinNumber = pin - '0';  // Get the integer value of the ASCII char

  if (pinNumber >= sizeof(pins) / sizeof(pins[0]))  // Reject out-of-bounds input
    return;

  switch (value) {
    case 'H':
      pinValues[pinNumber] = HIGH;  // Not necessary if you don't want to store the pin values
      digitalWrite(pins[pinNumber], HIGH);
      break;
    case 'L':
      pinValues[pinNumber] = LOW;  // Not necessary if you don't want to store the pin values
      digitalWrite(pins[pinNumber], LOW);
  }
}

Y luego simplemente llámalo desde loop()comosetPinValue(message[0], message[1]);

Déjame saber si esto resuelve tu problema.

Es un problema de rendimiento. La primera versión de setPinValues()está haciendo muchas llamadas a String()cada vez que se llama, y ​​esta es una función no trivial. Esto hace setPinValues()que tarde mucho más de lo que piensas en terminar, por lo que no es demasiado sorprendente que el Arduino finalmente no pueda seguir el ritmo de los mensajes entrantes a 57600 bps.

En la segunda versión de setPinValues(), los argumentos de las String()llamadas son todos constantes, por lo que se evalúan en tiempo de compilación. Esta versión se ejecuta mucho más rápido y puede mantenerse al día en 57600.