Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162378AbbKTI4F (ORCPT ); Fri, 20 Nov 2015 03:56:05 -0500 Received: from mailout4.w1.samsung.com ([210.118.77.14]:25553 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934862AbbKTIyV (ORCPT ); Fri, 20 Nov 2015 03:54:21 -0500 X-AuditID: cbfec7f4-f79026d00000418a-ff-564edfbb156c From: Marek Szyprowski To: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Marek Szyprowski , Ruslan Bilovol , Bartlomiej Zolnierkiewicz Subject: [PATCH v6 2/4] usb: gadget: configfs: pass UDC name via usb_gadget_driver struct Date: Fri, 20 Nov 2015 09:54:10 +0100 Message-id: <1448009652-14716-3-git-send-email-m.szyprowski@samsung.com> X-Mailer: git-send-email 1.9.2 In-reply-to: <1448009652-14716-1-git-send-email-m.szyprowski@samsung.com> References: <1448009652-14716-1-git-send-email-m.szyprowski@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpgluLIzCtJLcpLzFFi42I5/e/4Vd3d9/3CDN6dF7bYOGM9q8XlXXPY LBYta2W2WHvkLrtFz84TjA6sHjtn3WX36NuyitHj8ya5AOYoLpuU1JzMstQifbsEroydZ+ez F3RLVLw8fYCpgbFDpIuRk0NCwETi04z3bBC2mMSFe+uBbC4OIYGljBL/fj1ggXCamCQaOnYz gVSxCRhKdL3tAqri4BARsJboOxgEUsMsMIVR4uftZ2A1wgLREkeOdTOC2CwCqhKnVv4Cs3kF PCSOr21lgtgmJ/H/5Qowm1PAU2L29MvsILYQUM3PUyeZJzDyLmBkWMUomlqaXFCclJ5rqFec mFtcmpeul5yfu4kREi5fdjAuPmZ1iFGAg1GJh7dB3C9MiDWxrLgy9xCjBAezkgjvUj+gEG9K YmVValF+fFFpTmrxIUZpDhYlcd65u96HCAmkJ5akZqemFqQWwWSZODilGhgtvtktD3nnZi+8 b/auuytritIrXi549y+yzH1fsN3ZjexlvXdXB1wVVzrabMHYfyw4QXef0j1vgb9ibNH2RZYn yn+wbF9xQbXhs/Z9tYMTeYOfzN/a8t4+wurNiu0/v+bNT7xZ1bbgWJuuUwlrZ6/shVdlx/v+ LPeWljWK4tEqXXwzxDrxQaoSS3FGoqEWc1FxIgAo1CBAEwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3228 Lines: 105 From: Ruslan Bilovol Now when udc-core supports binding to specific UDC by passing its name via 'udc_name' member of usb_gadget_driver struct, switch to this generic approach. Tested-by: Maxime Ripard Signed-off-by: Ruslan Bilovol Signed-off-by: Marek Szyprowski --- drivers/usb/gadget/configfs.c | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/drivers/usb/gadget/configfs.c b/drivers/usb/gadget/configfs.c index 163d305..0bc6865 100644 --- a/drivers/usb/gadget/configfs.c +++ b/drivers/usb/gadget/configfs.c @@ -56,7 +56,6 @@ struct gadget_info { struct list_head string_list; struct list_head available_func; - const char *udc_name; struct usb_composite_driver composite; struct usb_composite_dev cdev; bool use_os_desc; @@ -233,21 +232,21 @@ static ssize_t gadget_dev_desc_bcdUSB_store(struct config_item *item, static ssize_t gadget_dev_desc_UDC_show(struct config_item *item, char *page) { - return sprintf(page, "%s\n", to_gadget_info(item)->udc_name ?: ""); + return sprintf(page, "%s\n", to_gadget_info(item)->composite.gadget_driver.udc_name ?: ""); } static int unregister_gadget(struct gadget_info *gi) { int ret; - if (!gi->udc_name) + if (!gi->composite.gadget_driver.udc_name) return -ENODEV; ret = usb_gadget_unregister_driver(&gi->composite.gadget_driver); if (ret) return ret; - kfree(gi->udc_name); - gi->udc_name = NULL; + kfree(gi->composite.gadget_driver.udc_name); + gi->composite.gadget_driver.udc_name = NULL; return 0; } @@ -271,14 +270,16 @@ static ssize_t gadget_dev_desc_UDC_store(struct config_item *item, if (ret) goto err; } else { - if (gi->udc_name) { + if (gi->composite.gadget_driver.udc_name) { ret = -EBUSY; goto err; } - ret = usb_udc_attach_driver(name, &gi->composite.gadget_driver); - if (ret) + gi->composite.gadget_driver.udc_name = name; + ret = usb_gadget_probe_driver(&gi->composite.gadget_driver); + if (ret) { + gi->composite.gadget_driver.udc_name = NULL; goto err; - gi->udc_name = name; + } } mutex_unlock(&gi->lock); return len; @@ -427,9 +428,9 @@ static int config_usb_cfg_unlink( * remove the function. */ mutex_lock(&gi->lock); - if (gi->udc_name) + if (gi->composite.gadget_driver.udc_name) unregister_gadget(gi); - WARN_ON(gi->udc_name); + WARN_ON(gi->composite.gadget_driver.udc_name); list_for_each_entry(f, &cfg->func_list, list) { if (f->fi == fi) { @@ -873,10 +874,10 @@ static int os_desc_unlink(struct config_item *os_desc_ci, struct usb_composite_dev *cdev = &gi->cdev; mutex_lock(&gi->lock); - if (gi->udc_name) + if (gi->composite.gadget_driver.udc_name) unregister_gadget(gi); cdev->os_desc_config = NULL; - WARN_ON(gi->udc_name); + WARN_ON(gi->composite.gadget_driver.udc_name); mutex_unlock(&gi->lock); return 0; } -- 1.9.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/