Discussion :

[PhilLU]

Bonjour,
J'utilise avec succès ESP NOW pour recevoir des messages de plusieurs modules. (ESP32 vers ESP8866)
Voici une des tentatives pour générer 3 beep sur un buzzer lors de la réception des ces messages (le buzzer et le code associé est bien fonctionnel puisque j'en ai forcé un cycle au démarrage du programme)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
void OnDataRecv(uint8_t * mac, uint8_t *incomingData, uint8_t len) {
  memcpy(&myData, incomingData, sizeof(myData));
  for (int i = 0; i < 3; i++) 
  {
    digitalWrite(buzzerPin, HIGH); // Set to HIGH to make the buzzer sound
    delay(40); 
    digitalWrite(buzzerPin, LOW); // LOW to turn off the buzzer
    delay(60);  
  }
delay(1500);
 Serial.println(myData.a);

Je reçois parfaitement les messages dans le port serie, mais pas les beep d'avertissement attendus :-(
Information importante: cette partie de code (beep à l'arrivée) fonctionne parfaitement avec un ESP32 en destination?!?
J'ai décidé de remplacer l'ESP32 par un ESP8266 car je recevais des messages parasites dans la trace sérielle que je n'ai jamais réussi à régler :-(

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

dhcps: send_nak>>udp_sendto result 0

Si qq à une idée...
Merci d'avance

[Jay M]

Sans l'intégralité du code difficile de se faire une opinion, mais si OnDataRecv() est exécutée dans un contexte d'interruption (lié à ESP-NOW) vous ne devez pas utiliser delay() dans cette fonction.

Levez un drapeau disant que le message est arrivé et traité le "beep" dans la loop.

[jpbbricole]

Bonjour PhilLU

Quels sont les numéros de ports du beep utilisés dans l'ESP32 et l'ESP8266?

Cordialement
jpbbricole

[PhilLU]

Bonjour,
Dans les deux cas, le port est 12.
Vu que je demande 3 beep au démarrage, et que ceux-ci fonctionnent, je m'assure que le port et la config et bonne.
Merci pour vos réponses...
Phil.

[PhilLU]

Bonjour,
Voici ce que j'ai trouvé comme solution...
Lors de la réception d'un message je met la variable booléene à true

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
void OnDataRecv(uint8_t * mac, uint8_t *incomingData, uint8_t len) {
  memcpy(&myData, incomingData, sizeof(myData));
 
  arrived = true;
 
delay(10); 
 Serial.println(myData.a);
}

... ensuite dans le loop, je génère les beep et je remet la variable à false...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void loop(){
 
  if (arrived == true){
    for (int i = 0; i < 3; i++) 
  {
    digitalWrite(buzzerPin, HIGH); // Set to HIGH to make the buzzer sound
    delay(40); 
    digitalWrite(buzzerPin, LOW); // LOW to turn off the buzzer
    delay(60);  
  }
delay(100);
 
}
arrived = false;
}

... reste à optimiser les delay pour ne pas risquer de rater un autre message (à moi que la tâche OnDataRecv soit prioritaire; à vrai dire je n'en sais rien :-(

Merci pour vos suggestions...

[Jay M]

le drapeau c'est bien mais si OnDataRecv() est celui d'ESP-NOW, vous êtes dans un contexte d'interruption et donc il ne fait pas faire de delay() du tout et le drapeau doit être une variable déclarée volatile.

idéalement il faudrait aussi traiter sa lecture et remise à false dans une section critique (avec un sémaphore / mutex) dans la loop et ne mettre le drapeau false qu'une fois le message traité.

[edgarjacobs]

Hello,

Pour répondre à la question de @Jay M: comment le programme entre-t-il dans la fonction OnDataRecv() ? Montrer le setup() serait une bonne chose.

Ensuite, il faudrait faire à l'indentation du code: par exemple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
void loop(void) {
    if (arrived == true) {
        for (int i = 0; i < 3; i++) {
            digitalWrite(buzzerPin, HIGH); // Set to HIGH to make the buzzer sound
            delay(40);
            digitalWrite(buzzerPin, LOW);  // LOW to turn off the buzzer
            delay(60);
        }
        delay(100);
    }
    arrived = false;
}

Là, on voit clairement les blocs.