Hi Felipe, On 07/21/2017 06:31 PM, Felipe Balbi wrote: > Hi, > > Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx> writes: >> +static void xhci_dbc_stop(struct xhci_hcd *xhci) >> +{ >> + struct xhci_dbc *dbc = xhci->dbc; >> + >> + WARN_ON(!dbc); >> + >> + cancel_delayed_work_sync(&dbc->event_work); >> + >> + if (dbc->gs_port_num != GSPORT_INVAL) { >> + gserial_disconnect(&dbc->gs_port); >> + gserial_free_line(dbc->gs_port_num); > why are you tying host stack to the gadget framework? XHCI debug capability is actually a debug device gadget. The hardware and firmware do everything of gadget work and leave the interface to xHCI for enabling/disabling and queuing transfer requests. u_serial.c provides a generic layer between a USB gadget and the TTY layer. I used it to avoid duplicating code. > > With this, you're forcing every single PC in the world to compile the > gadget framework, that's a bit much don't you think? > Yes, you are right. Is it acceptable if I move u_serial.c from the current place to drivers/usb/common? Best regards, Lu Baolu -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html