diff --git a/components/usb/ext_hub.c b/components/usb/ext_hub.c index d54feb7fba..fb20242f70 100644 --- a/components/usb/ext_hub.c +++ b/components/usb/ext_hub.c @@ -1309,7 +1309,6 @@ esp_err_t ext_hub_new_dev(uint8_t dev_addr) ret = find_first_intf_desc(config_desc, &hub_config); if (ret != ESP_OK) { - ESP_LOGE(EXT_HUB_TAG, "Finding HUB interface error %s", esp_err_to_name(ret)); goto exit; } diff --git a/components/usb/private_include/ext_hub.h b/components/usb/private_include/ext_hub.h index 6017daf74e..396d838be1 100644 --- a/components/usb/private_include/ext_hub.h +++ b/components/usb/private_include/ext_hub.h @@ -97,8 +97,6 @@ esp_err_t ext_hub_uninstall(void); * Entry: * - should be called within Hub Driver * - * @param[in] config External Hub driver configuration - * * @return * - Unique pointer to identify the External Hub as a USB Host client */ @@ -157,7 +155,7 @@ esp_err_t ext_hub_dev_gone(uint8_t dev_addr); * * @return * - ESP_OK: All devices freed - * - ESP_ERR_INVALID_STATE: External Hub driver is not installed + * - ESP_ERR_NOT_FINISHED: Operation not finished: devices waiting children to be freed. */ esp_err_t ext_hub_all_free(void); diff --git a/components/usb/usb_host.c b/components/usb/usb_host.c index 13492aa3b4..3be490f0e9 100644 --- a/components/usb/usb_host.c +++ b/components/usb/usb_host.c @@ -1018,16 +1018,10 @@ esp_err_t usb_host_device_free_all(void) HOST_EXIT_CRITICAL(); esp_err_t ret; #if ENABLE_USB_HUBS - ret = hub_notify_all_free(); - if (ret != ESP_OK) { - ESP_LOGE(USB_HOST_TAG, "Marking all devices as free in HUB error: %s", esp_err_to_name(ret)); - } + hub_notify_all_free(); #endif // ENABLE_USB_HUBS ret = usbh_devs_mark_all_free(); // If ESP_ERR_NOT_FINISHED is returned, caller must wait for USB_HOST_LIB_EVENT_FLAGS_ALL_FREE to confirm all devices are free - if (ret != ESP_OK) { - ESP_LOGE(USB_HOST_TAG, "Marking all devices as free in USBH error: %s", esp_err_to_name(ret)); - } return ret; } diff --git a/components/usb/usbh.c b/components/usb/usbh.c index 59dff2fe6a..75188f02d9 100644 --- a/components/usb/usbh.c +++ b/components/usb/usbh.c @@ -574,7 +574,7 @@ static inline void handle_ep0_clear(device_t *dev_obj) static inline void handle_prop_gone_evt(device_t *dev_obj) { // Flush EP0's pipe. Then propagate a USBH_EVENT_DEV_GONE event - ESP_LOGE(USBH_TAG, "Device %d gone", dev_obj->constant.address); + ESP_LOGD(USBH_TAG, "Device %d gone", dev_obj->constant.address); usbh_event_data_t event_data = { .event = USBH_EVENT_DEV_GONE, .dev_gone_data = {