Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755558AbbLKL3N (ORCPT ); Fri, 11 Dec 2015 06:29:13 -0500 Received: from mailout4.samsung.com ([203.254.224.34]:39881 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755449AbbLKL1T (ORCPT ); Fri, 11 Dec 2015 06:27:19 -0500 X-AuditID: cbfee61a-f79266d000003652-36-566ab30b9098 From: Robert Baldyga To: balbi@ti.com Cc: gregkh@linuxfoundation.org, andrzej.p@samsung.com, m.szyprowski@samsung.com, b.zolnierkie@samsung.com, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Robert Baldyga Subject: [PATCH v3 33/36] usb: gadget: f_subset: conversion to new API Date: Fri, 11 Dec 2015 12:25:12 +0100 Message-id: <1449833115-24065-34-git-send-email-r.baldyga@samsung.com> X-Mailer: git-send-email 1.9.1 In-reply-to: <1449833115-24065-1-git-send-email-r.baldyga@samsung.com> References: <1449833115-24065-1-git-send-email-r.baldyga@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprOLMWRmVeSWpSXmKPExsVy+t9jQV3uzVlhBg/2q1vMetnOYrFxxnpW i4P36y2aF69ns7i8aw6bxaJlrcwWa4/cZbd4cHgnuwOHx/65a9g9+rasYvQ4fmM7k8fnTXIB LFFcNimpOZllqUX6dglcGRt+XmUq+OFU8fX+UaYGxusWXYycHBICJhKXmw6wQNhiEhfurWfr YuTiEBKYxSixr/UFlPOTUeL33qWMIFVsAjoSW75PALNFBAQk1r+4xA5SxCxwjlHi4Z02sISw gJvEhP7NbCA2i4CqxJ0Db9hBbF6g+O7zp1kh1slJnDw2GczmBIpPPnmJGcQWEnCVePN6HeME Rt4FjAyrGCVSC5ILipPScw3zUsv1ihNzi0vz0vWS83M3MYKD65nUDsaDu9wPMQpwMCrx8C7g yAoTYk0sK67MPcQowcGsJML7awNQiDclsbIqtSg/vqg0J7X4EKM0B4uSOG/tpcgwIYH0xJLU 7NTUgtQimCwTB6dUA6O/zU7VL55vK0T2pjw3Ysk02Va7TGzyjj1X95tv8+1uEBF6drjj3h7n /g15m9v+31TSktPW/TvXtUxLbGNDcHOT0dZ52d0avFP3Xfn4ueiliOsCvfnPNefODtvoMzNq pmgog6TS43oh13eCNcXVNx29k581GTk9vr92c6lOaQ2fW+ub11OqOZRYijMSDbWYi4oTAbNo r94qAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8564 Lines: 254 Generate descriptors in new format and attach them to USB function in prep_descs(). Implement prep_vendor_descs() to supply class specific descriptors. Change set_alt() implementation and implement clear_alt() operation. Remove boilerplate code. Signed-off-by: Robert Baldyga --- drivers/usb/gadget/function/f_subset.c | 165 ++++++++++----------------------- 1 file changed, 47 insertions(+), 118 deletions(-) diff --git a/drivers/usb/gadget/function/f_subset.c b/drivers/usb/gadget/function/f_subset.c index 829c78d..98324ca 100644 --- a/drivers/usb/gadget/function/f_subset.c +++ b/drivers/usb/gadget/function/f_subset.c @@ -154,17 +154,6 @@ static struct usb_endpoint_descriptor fs_subset_out_desc = { .bmAttributes = USB_ENDPOINT_XFER_BULK, }; -static struct usb_descriptor_header *fs_eth_function[] = { - (struct usb_descriptor_header *) &subset_data_intf, - (struct usb_descriptor_header *) &mdlm_header_desc, - (struct usb_descriptor_header *) &mdlm_desc, - (struct usb_descriptor_header *) &mdlm_detail_desc, - (struct usb_descriptor_header *) ðer_desc, - (struct usb_descriptor_header *) &fs_subset_in_desc, - (struct usb_descriptor_header *) &fs_subset_out_desc, - NULL, -}; - /* high speed support: */ static struct usb_endpoint_descriptor hs_subset_in_desc = { @@ -183,17 +172,6 @@ static struct usb_endpoint_descriptor hs_subset_out_desc = { .wMaxPacketSize = cpu_to_le16(512), }; -static struct usb_descriptor_header *hs_eth_function[] = { - (struct usb_descriptor_header *) &subset_data_intf, - (struct usb_descriptor_header *) &mdlm_header_desc, - (struct usb_descriptor_header *) &mdlm_desc, - (struct usb_descriptor_header *) &mdlm_detail_desc, - (struct usb_descriptor_header *) ðer_desc, - (struct usb_descriptor_header *) &hs_subset_in_desc, - (struct usb_descriptor_header *) &hs_subset_out_desc, - NULL, -}; - /* super speed support: */ static struct usb_endpoint_descriptor ss_subset_in_desc = { @@ -221,18 +199,16 @@ static struct usb_ss_ep_comp_descriptor ss_subset_bulk_comp_desc = { /* .bmAttributes = 0, */ }; -static struct usb_descriptor_header *ss_eth_function[] = { - (struct usb_descriptor_header *) &subset_data_intf, - (struct usb_descriptor_header *) &mdlm_header_desc, - (struct usb_descriptor_header *) &mdlm_desc, - (struct usb_descriptor_header *) &mdlm_detail_desc, - (struct usb_descriptor_header *) ðer_desc, - (struct usb_descriptor_header *) &ss_subset_in_desc, - (struct usb_descriptor_header *) &ss_subset_bulk_comp_desc, - (struct usb_descriptor_header *) &ss_subset_out_desc, - (struct usb_descriptor_header *) &ss_subset_bulk_comp_desc, - NULL, -}; +USB_COMPOSITE_ENDPOINT(ep_in, &fs_subset_in_desc, &hs_subset_in_desc, + &ss_subset_in_desc, &ss_subset_bulk_comp_desc); +USB_COMPOSITE_ENDPOINT(ep_out, &fs_subset_out_desc, &hs_subset_out_desc, + &ss_subset_out_desc, &ss_subset_bulk_comp_desc); + +USB_COMPOSITE_ALTSETTING(intf0alt0, &subset_data_intf, &ep_in, &ep_out); + +USB_COMPOSITE_INTERFACE(intf0, &intf0alt0); + +USB_COMPOSITE_DESCRIPTORS(subset_descs, &intf0); /* string descriptors: */ @@ -260,26 +236,20 @@ static int geth_set_alt(struct usb_function *f, unsigned intf, unsigned alt) struct usb_composite_dev *cdev = f->config->cdev; struct net_device *net; - /* we know alt == 0, so this is an activation or a reset */ - - if (geth->port.in_ep->enabled) { - DBG(cdev, "reset cdc subset\n"); - gether_disconnect(&geth->port); - } - DBG(cdev, "init + activate cdc subset\n"); - if (config_ep_by_speed(cdev->gadget, f, geth->port.in_ep) || - config_ep_by_speed(cdev->gadget, f, geth->port.out_ep)) { - geth->port.in_ep->desc = NULL; - geth->port.out_ep->desc = NULL; - return -EINVAL; - } + + geth->port.in_ep = usb_function_get_ep(f, intf, 0); + if (!geth->port.in_ep) + return -ENODEV; + geth->port.out_ep = usb_function_get_ep(f, intf, 1); + if (!geth->port.out_ep) + return -ENODEV; net = gether_connect(&geth->port); return PTR_ERR_OR_ZERO(net); } -static void geth_disable(struct usb_function *f) +static void geth_clear_alt(struct usb_function *f, unsigned intf, unsigned alt) { struct f_gether *geth = func_to_geth(f); struct usb_composite_dev *cdev = f->config->cdev; @@ -292,14 +262,26 @@ static void geth_disable(struct usb_function *f) /* serial function driver setup/binding */ -static int -geth_bind(struct usb_configuration *c, struct usb_function *f) +static int geth_prep_descs(struct usb_function *f) { - struct usb_composite_dev *cdev = c->cdev; - struct f_gether *geth = func_to_geth(f); + struct usb_composite_dev *cdev = f->config->cdev; struct usb_string *us; + + us = usb_gstrings_attach(cdev, geth_strings, + ARRAY_SIZE(geth_string_defs)); + if (IS_ERR(us)) + return PTR_ERR(us); + + subset_data_intf.iInterface = us[0].id; + ether_desc.iMACAddress = us[1].id; + + return usb_function_set_descs(f, &subset_descs); +} + +static int geth_prep_vendor_descs(struct usb_function *f) +{ + struct usb_composite_dev *cdev = f->config->cdev; int status; - struct usb_ep *ep; struct f_gether_opts *gether_opts; @@ -322,63 +304,16 @@ geth_bind(struct usb_configuration *c, struct usb_function *f) gether_opts->bound = true; } - us = usb_gstrings_attach(cdev, geth_strings, - ARRAY_SIZE(geth_string_defs)); - if (IS_ERR(us)) - return PTR_ERR(us); - - subset_data_intf.iInterface = us[0].id; - ether_desc.iMACAddress = us[1].id; - - /* allocate instance-specific interface IDs */ - status = usb_interface_id(c, f); - if (status < 0) - goto fail; - subset_data_intf.bInterfaceNumber = status; - - status = -ENODEV; - - /* allocate instance-specific endpoints */ - ep = usb_ep_autoconfig(cdev->gadget, &fs_subset_in_desc); - if (!ep) - goto fail; - geth->port.in_ep = ep; - - ep = usb_ep_autoconfig(cdev->gadget, &fs_subset_out_desc); - if (!ep) - goto fail; - geth->port.out_ep = ep; - - /* support all relevant hardware speeds... we expect that when - * hardware is dual speed, all bulk-capable endpoints work at - * both speeds - */ - hs_subset_in_desc.bEndpointAddress = fs_subset_in_desc.bEndpointAddress; - hs_subset_out_desc.bEndpointAddress = - fs_subset_out_desc.bEndpointAddress; - - ss_subset_in_desc.bEndpointAddress = fs_subset_in_desc.bEndpointAddress; - ss_subset_out_desc.bEndpointAddress = - fs_subset_out_desc.bEndpointAddress; - - status = usb_assign_descriptors(f, fs_eth_function, hs_eth_function, - ss_eth_function); - if (status) - goto fail; - - /* NOTE: all that is done without knowing or caring about - * the network link ... which is unavailable to this code - * until we're activated via set_alt(). - */ - - DBG(cdev, "CDC Subset: %s speed IN/%s OUT/%s\n", - gadget_is_superspeed(c->cdev->gadget) ? "super" : - gadget_is_dualspeed(c->cdev->gadget) ? "dual" : "full", - geth->port.in_ep->name, geth->port.out_ep->name); - return 0; + subset_data_intf.bInterfaceNumber = usb_get_interface_id(f, 0); -fail: - ERROR(cdev, "%s: can't bind, err %d\n", f->name, status); + usb_altset_add_vendor_desc(f, 0, 0, + (struct usb_descriptor_header *)&mdlm_header_desc); + usb_altset_add_vendor_desc(f, 0, 0, + (struct usb_descriptor_header *)&mdlm_desc); + usb_altset_add_vendor_desc(f, 0, 0, + (struct usb_descriptor_header *)&mdlm_detail_desc); + usb_altset_add_vendor_desc(f, 0, 0, + (struct usb_descriptor_header *)ðer_desc); return status; } @@ -460,12 +395,6 @@ static void geth_free(struct usb_function *f) kfree(eth); } -static void geth_unbind(struct usb_configuration *c, struct usb_function *f) -{ - geth_string_defs[0].id = 0; - usb_free_all_descriptors(f); -} - static struct usb_function *geth_alloc(struct usb_function_instance *fi) { struct f_gether *geth; @@ -496,10 +425,10 @@ static struct usb_function *geth_alloc(struct usb_function_instance *fi) geth->port.cdc_filter = DEFAULT_FILTER; geth->port.func.name = "cdc_subset"; - geth->port.func.bind = geth_bind; - geth->port.func.unbind = geth_unbind; + geth->port.func.prep_descs = geth_prep_descs; + geth->port.func.prep_vendor_descs = geth_prep_vendor_descs; geth->port.func.set_alt = geth_set_alt; - geth->port.func.disable = geth_disable; + geth->port.func.clear_alt = geth_clear_alt; geth->port.func.free_func = geth_free; return &geth->port.func; -- 1.9.1 -- 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/