animevietsub
commented
1 year ago Because HandleSendMIDIClock is called from interrupt so you can't MIDI_Serial.sendRealTime(midi::Clock);.
One of the best ways to fix this is to use xSemaphore. With your code that should be some things like this:
#include "Arduino.h"
#include <HardwareSerial.h>
#include <MIDI.h>
#include <uClock.h>
#include <freertos/semphr.h>
#define TXD2 17
#define RXD2 18
#define LED 2
static SemaphoreHandle_t xSemaphore = NULL;
static BaseType_t xHigherPriorityTaskWoken = pdFALSE;
byte clockCounter = 0; // Count the MIDI clock pulses, 24 pulses per quarter note
bool midiClockLedOn = false;
MIDI_CREATE_INSTANCE(HardwareSerial, Serial2, MIDI_Serial);
// The callback function which will be called by Clock each Pulse of 96PPQN clock resolution.
void ClockOut96PPQN(uint32_t tick)
{
// Send MIDI_CLOCK to external gears
HandleSendMIDIClock();
}
void setup()
{
xSemaphore = xSemaphoreCreateBinary();
Serial.begin(115200); // setup HW serial for debug communication
Serial2.begin(31250, SERIAL_8N1, RXD2, TXD2); // setup Serial2 for MIDI control
pinMode(LED, OUTPUT); // set the LED pin as an output
digitalWrite(LED, LOW); // turn off the LED
MIDI_Serial.begin(MIDI_CHANNEL_OMNI); // Initiate MIDI_Serial communications, listen to all channels
MIDI_Serial.turnThruOff();
// Inits the clock
uClock.init();
// Set the callback function for the clock output to send MIDI Sync message.
uClock.setClock96PPQNOutput(ClockOut96PPQN);
// Set the clock BPM to 126 BPM
uClock.setTempo(126);
// Starts the clock, tick-tac-tick-tac...
uClock.start();
};
void loop()
{
// Empty loop, no need to execute anything repeatedly.
if (xSemaphoreTake(xSemaphore, portMAX_DELAY))
{
MIDI_Serial.sendRealTime(midi::Clock);
xHigherPriorityTaskWoken = pdFALSE;
}
};
void HandleSendMIDIClock()
{
xSemaphoreGiveFromISR(xSemaphore, &xHigherPriorityTaskWoken);
// Increment the clock counter, wrap around after 24 pulses
clockCounter = (clockCounter + 1) & 0x0F;
handle_bpm_led(clockCounter); // leds work fine
portYIELD_FROM_ISR(xHigherPriorityTaskWoken);
}
void handle_bpm_led(uint32_t tick)
{
// BPM led indicator
if (tick == 0 || tick % 24 == 0)
{ // Each quarter note
midiClockLedOn = true;
digitalWrite(LED, HIGH);
}
else if (midiClockLedOn && tick % 24 == 6)
{ // Turn off the LED after 6 ticks (1/16th note duration)
midiClockLedOn = false;
digitalWrite(LED, LOW);
}
}
cyberdevnet
midilab
Hi,
I'm facing a Guru Meditation Error on my ESP32 Dev Module and ESP32-S3 Dev Module boards when trying to implement a basic code. The error occurs when sending the MIDI clock message, and it leads to a reboot with the following message:
Guru Meditation Error: Core 1 panic'ed (Interrupt wdt timeout on CPU1).Additionally, the backtrace shows corruption in the core 1 and core 0 register dumps.
Code:
Error:
Bord config (vscode):
Additional Information:
I was able once or twice to let the code run for 10 seconds on ESP32-S3 and the clock accuracy is perfect, 126 sent, 126 get. but suddenly it crashed over and over...
Has the library already been tested with these platforms?
Something that I have missed out?