Merge pull request #54 from someweisguy:bugfix/gptimer-fail

allow gptimer to fail gracefully
someweisguy · Mar 17, 2023 · 18e52fa · 18e52fa
2 parents 7d028ca + 945675e
commit 18e52fa
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/esp_dmx.c b/src/esp_dmx.c
@@ -488,7 +488,12 @@ esp_err_t dmx_driver_install(dmx_port_t dmx_num, int intr_flags) {
       .direction = GPTIMER_COUNT_UP,
       .resolution_hz = 1000000,  // 1MHz resolution timer
   };
-  gptimer_new_timer(&timer_config, &driver->gptimer_handle);  // TODO: err check
+  esp_err_t err = gptimer_new_timer(&timer_config, &driver->gptimer_handle);
+  if (err) {
+    ESP_LOGE(TAG, "DMX driver gptimer error");
+    dmx_driver_delete(dmx_num);
+    return err;
+  }
   const gptimer_event_callbacks_t gptimer_cb = {.on_alarm = dmx_timer_isr};
   gptimer_register_event_callbacks(driver->gptimer_handle, &gptimer_cb, driver);
   gptimer_enable(driver->gptimer_handle);