usb: cdc_acm: update state after RESUME event

Update state after RESUME event. Use LOG_INF for important events and log a warning if the data is discarded. Fixes zephyrproject-rtos#17488 Signed-off-by: Johann Fischer <[email protected]>
mlaz · Aug 23, 2019 · 5fb7c11 · 5fb7c11
1 parent 112ecb7
commit 5fb7c11
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/subsys/usb/class/cdc_acm.c b/subsys/usb/class/cdc_acm.c
@@ -408,17 +408,18 @@ static void cdc_acm_do_cb(struct cdc_acm_dev_data_t *dev_data,
 		dev_data->tx_ready = true;
 		dev_data->tx_irq_ena = true;
 		dev_data->rx_irq_ena = true;
-		LOG_DBG("USB device configured");
+		LOG_INF("USB device configured");
 		break;
 	case USB_DC_DISCONNECTED:
-		LOG_DBG("USB device disconnected");
+		LOG_INF("USB device disconnected");
 		cdc_acm_reset_port(dev_data);
 		break;
 	case USB_DC_SUSPEND:
-		LOG_DBG("USB device suspended");
+		LOG_INF("USB device suspended");
 		break;
 	case USB_DC_RESUME:
-		LOG_DBG("USB device resumed");
+		dev_data->usb_status = USB_DC_CONFIGURED;
+		LOG_INF("USB device resumed");
 		break;
 	case USB_DC_SOF:
 		break;
@@ -548,6 +549,7 @@ static int cdc_acm_fifo_fill(struct device *dev,
 		dev_data, len, ring_buf_space_get(dev_data->tx_ringbuf));
 
 	if (dev_data->usb_status != USB_DC_CONFIGURED) {
+		LOG_WRN("Device not configured, drop %d bytes", len);
 		return 0;
 	}