usb: device_next: usbd_cdc_ncm: correct net_if_carrier use

net_if_carrier is to be used independently of the
administrative state (start and stop of the ethernet_api).

This seem to work.

Signed-off-by: Fin Maaß <[email protected]>

Author: maass-hamburg

subsys/usb/device_next/class/usbd_cdc_ncm.c

@@ -635,6 +635,7 @@ static void ncm_handle_notifications(const struct device *dev, const int err)
 	if (data->if_state == IF_STATE_CONNECTION_STATUS_SUBMITTED) {
 		data->if_state = IF_STATE_CONNECTION_STATUS_SENT;
 		LOG_INF("Connection status sent");
+		net_if_carrier_on(data->iface);
 	}
 }
 
@@ -840,6 +841,7 @@ static void usbd_cdc_ncm_update(struct usbd_class_data *const c_data,
 
 	if (data_iface == iface && alternate == 0) {
 		atomic_clear_bit(&data->state, CDC_NCM_DATA_IFACE_ENABLED);
+		net_if_carrier_off(data->iface);
 		data->tx_seq = 0;
 		data->rx_seq = 0;
 	}
@@ -868,6 +870,8 @@ static void usbd_cdc_ncm_disable(struct usbd_class_data *const c_data)
 	atomic_clear_bit(&data->state, CDC_NCM_DATA_IFACE_ENABLED);
 	atomic_clear_bit(&data->state, CDC_NCM_CLASS_SUSPENDED);
 
+	net_if_carrier_off(data->iface);
+
 	LOG_INF("Disabled %s", c_data->name);
 }
 
@@ -1123,7 +1127,6 @@ static int cdc_ncm_iface_start(const struct device *dev)
 	LOG_DBG("Start interface %d", net_if_get_by_iface(data->iface));
 
 	atomic_set_bit(&data->state, CDC_NCM_IFACE_UP);
-	net_if_carrier_on(data->iface);
 
 	if (atomic_test_bit(&data->state, CDC_NCM_DATA_IFACE_ENABLED)) {
 		(void)k_work_reschedule(&data->notif_work, K_MSEC(1));