Driver breakdown improvements

Drivers/cst816s-module/source/cst816s.cpp

@@ -105,22 +105,27 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Cst816sInternal*>(device_get_driver_data(device));
 
-    bool ok = true;
-
     // esp_lcd_touch_del() only releases the touch-side resources; the panel IO handle is owned
     // separately and needs its own deletion.
+    if (internal->touch_handle != nullptr) {
         if (esp_lcd_touch_del(internal->touch_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete touch handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->touch_handle = nullptr;
     }
 
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     free(internal);
-    return ok ? ERROR_NONE : ERROR_RESOURCE;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/ft5x06-module/source/ft5x06.cpp

@@ -105,22 +105,27 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Ft5x06Internal*>(device_get_driver_data(device));
 
-    bool ok = true;
-
     // esp_lcd_touch_del() only releases the touch-side resources; the panel IO handle is owned
     // separately and needs its own deletion.
+    if (internal->touch_handle != nullptr) {
         if (esp_lcd_touch_del(internal->touch_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete touch handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->touch_handle = nullptr;
     }
 
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     free(internal);
-    return ok ? ERROR_NONE : ERROR_RESOURCE;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/ft6x36-module/source/ft6x36.cpp

@@ -105,22 +105,27 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Ft6x36Internal*>(device_get_driver_data(device));
 
-    bool ok = true;
-
     // esp_lcd_touch_del() only releases the touch-side resources; the panel IO handle is owned
     // separately and needs its own deletion.
+    if (internal->touch_handle != nullptr) {
         if (esp_lcd_touch_del(internal->touch_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete touch handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->touch_handle = nullptr;
     }
 
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     free(internal);
-    return ok ? ERROR_NONE : ERROR_RESOURCE;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/gt911-module/source/gt911.cpp

@@ -114,22 +114,27 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Gt911Internal*>(device_get_driver_data(device));
 
-    bool ok = true;
-
     // esp_lcd_touch_del() only releases the touch-side resources; the panel IO handle is owned
     // separately and needs its own deletion.
+    if (internal->touch_handle != nullptr) {
         if (esp_lcd_touch_del(internal->touch_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete touch handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->touch_handle = nullptr;
     }
 
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     free(internal);
-    return ok ? ERROR_NONE : ERROR_RESOURCE;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/ili9341-module/source/ili9341.cpp

@@ -165,20 +165,26 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Ili9341Internal*>(device_get_driver_data(device));
 
-    error_t result = ERROR_NONE;
+    if (internal->panel_handle != nullptr) {
-
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
+
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     vSemaphoreDelete(internal->draw_done_semaphore);
     free(internal);
-    return result;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/ili9488-module/source/ili9488.cpp

@@ -161,17 +161,25 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<Ili9488Internal*>(device_get_driver_data(device));
 
+    if (internal->panel_handle != nullptr) {
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
             return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
+
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO");
             return ERROR_RESOURCE;
         }
+        internal->io_handle = nullptr;
+    }
 
     vSemaphoreDelete(internal->draw_done_semaphore);
     free(internal);
+    device_set_driver_data(device, nullptr);
     return ERROR_NONE;
 }
 

Drivers/rgb-display-module/source/rgb_display.cpp

@@ -256,15 +256,17 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<RgbDisplayInternal*>(device_get_driver_data(device));
 
+    if (internal->panel_handle != nullptr) {
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
-        vSemaphoreDelete(internal->frame_complete_semaphore);
-        free(internal);
             return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
 
     vSemaphoreDelete(internal->frame_complete_semaphore);
     free(internal);
+    device_set_driver_data(device, nullptr);
     return ERROR_NONE;
 }
 

Drivers/st7789-i8080-module/source/st7789_i8080.cpp

@@ -204,20 +204,26 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<St7789I8080Internal*>(device_get_driver_data(device));
 
-    error_t result = ERROR_NONE;
+    if (internal->panel_handle != nullptr) {
-
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
+
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     vSemaphoreDelete(internal->draw_done_semaphore);
     free(internal);
-    return result;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/st7789-module/source/st7789.cpp

@@ -165,17 +165,25 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<St7789Internal*>(device_get_driver_data(device));
 
+    if (internal->panel_handle != nullptr) {
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
             return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
+
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO");
             return ERROR_RESOURCE;
         }
+        internal->io_handle = nullptr;
+    }
 
     vSemaphoreDelete(internal->draw_done_semaphore);
     free(internal);
+    device_set_driver_data(device, nullptr);
     return ERROR_NONE;
 }
 

Drivers/st7796-module/source/st7796.cpp

@@ -165,20 +165,26 @@ static error_t start(Device* device) {
 static error_t stop(Device* device) {
     auto* internal = static_cast<St7796Internal*>(device_get_driver_data(device));
 
-    error_t result = ERROR_NONE;
+    if (internal->panel_handle != nullptr) {
-
         if (esp_lcd_panel_del(internal->panel_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
         }
+        internal->panel_handle = nullptr;
+    }
+
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO");
-        result = ERROR_RESOURCE;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     vSemaphoreDelete(internal->draw_done_semaphore);
     free(internal);
-    return result;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion

Drivers/xpt2046-module/source/xpt2046.cpp

@@ -239,24 +239,30 @@ static error_t stop(Device* device) {
 
     if (internal->power_supply_device != nullptr) {
         destroy_power_supply_child(internal->power_supply_device);
+        internal->power_supply_device = nullptr;
     }
 
-    bool ok = true;
-
     // esp_lcd_touch_del() only releases the touch-side resources; the panel IO handle is owned
     // separately and needs its own deletion.
+    if (internal->touch_handle != nullptr) {
         if (esp_lcd_touch_del(internal->touch_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete touch handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->touch_handle = nullptr;
     }
 
+    if (internal->io_handle != nullptr) {
         if (esp_lcd_panel_io_del(internal->io_handle) != ESP_OK) {
             LOG_E(TAG, "Failed to delete panel IO handle");
-        ok = false;
+            return ERROR_RESOURCE;
+        }
+        internal->io_handle = nullptr;
     }
 
     free(internal);
-    return ok ? ERROR_NONE : ERROR_RESOURCE;
+    device_set_driver_data(device, nullptr);
+    return ERROR_NONE;
 }
 
 // endregion