Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp6763115imu; Mon, 3 Dec 2018 02:20:51 -0800 (PST) X-Google-Smtp-Source: AFSGD/UZWry16OGTpVOS5xuFwUmA/mslJE4EiVPlEfPhcbVkc3Qal9JjPXpz2tPc2Dn+nTHobSLI X-Received: by 2002:a17:902:264:: with SMTP id 91mr14746301plc.108.1543832451150; Mon, 03 Dec 2018 02:20:51 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543832451; cv=none; d=google.com; s=arc-20160816; b=up2Yg1cYivuFFyIUqtXydnotprh2HbWeQRn19GlgG+dMl7TIZioV23I8uUbeLj+4l2 PcZiq8rNCRbQTxiARhUo5nnUBSQXzN899bq5AWgD+HRhAC08FkHy9vS92EizWVU/d7lx R15RRhFFzq0Km+lVFKun3y68GqAF4pE82XwENmec4SEwQPJBHLf1X6vQeZJmcBZImGC5 ad4954w/H3/OhiQzaON/Pu470EqOBufrtAcGzoGUiz3dheMckKuL1ufxzHILdegvZbnC ckCJaGWV5R/qSU4WjK/txFVdPpOA4xXymzuys3CKmN2HYpIWMF7IRFer6IWWOpiCzEM4 t8CA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :spamdiagnosticmetadata:spamdiagnosticoutput:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from:dkim-signature:dkim-signature; bh=Oog4wjXbmSq5ZuRftG6A4/VaXCPVpgtjLAMFOOxaCFs=; b=TrHTavqEGeGLUrLCwBTBNK9ybFhTtHIg6KC9dQsyxf9cGy5I49KSvI52FxEDdhhmZ6 svDCAVMWLtB4L+wrtwm3dXg81XJ6VxqCzrizr7qC/6fSE2Z5y1J0s6ByRoD4aKnbHd+n IQ6ceXaf61jI/6KfPLhjJihBYDopK+PwXIicGcC2MdfdTErxrrbenM43K1AwLcRq8bkb q0S71TGrLIZN6X10ANYJGMYwS3lTvk0RXxKn+3veJr18UE0NQlvvU/uTk7sNWhgtbKky q7yWKIk1tQR3VrpOe9Lt/Hin3eXtqkH5n1Y5QiUHNvTPEvQxtY/3jCDeabf9Brxkizl9 iyhw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@cadence.com header.s=proofpoint header.b=EUhbRXJT; dkim=pass header.i=@cadence.com header.s=selector1 header.b=qG+bmImT; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=cadence.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e2si11672416pgs.94.2018.12.03.02.20.36; Mon, 03 Dec 2018 02:20:51 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@cadence.com header.s=proofpoint header.b=EUhbRXJT; dkim=pass header.i=@cadence.com header.s=selector1 header.b=qG+bmImT; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=cadence.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726132AbeLCKUZ (ORCPT + 99 others); Mon, 3 Dec 2018 05:20:25 -0500 Received: from mx0b-0014ca01.pphosted.com ([208.86.201.193]:54274 "EHLO mx0a-0014ca01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725947AbeLCKUZ (ORCPT ); Mon, 3 Dec 2018 05:20:25 -0500 Received: from pps.filterd (m0042333.ppops.net [127.0.0.1]) by mx0b-0014ca01.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id wB3AIhpa018108; Mon, 3 Dec 2018 02:19:51 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cadence.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-transfer-encoding : mime-version; s=proofpoint; bh=Oog4wjXbmSq5ZuRftG6A4/VaXCPVpgtjLAMFOOxaCFs=; b=EUhbRXJT1p2VP1u9mbPLPlPBWkmCaY25Phonw7Rq1Iqys95xGlhp/LZqCn+NEd5t+JER 38XSxeb/5L5sKiTL4j7tGf5qyQj3oXNG/BnwTgQRsEL/U69GurmBwYOl+5imDYhRfvRW eoKI7CzoDVLBguTu9bcFYzcdrntnUbOFQJ/i0jiCFkf1FkE+maaXdPB292JxMnzy/Qpx N5sZfqsZk5VTBhJ2pgxoAEB41ByQMuezB19gdbsaY/sPzOTEWKLXAYnQdLAP9dzhXNCq qHKiBnSOjjwNdD8eaqAtmGk6xpC/95eDwM5CkX0IJbMzQyGfUt3REhqedcJAb1KtcgGw /A== Authentication-Results: cadence.com; spf=pass smtp.mailfrom=pawell@cadence.com Received: from nam02-sn1-obe.outbound.protection.outlook.com (mail-sn1nam02lp2059.outbound.protection.outlook.com [104.47.36.59]) by mx0b-0014ca01.pphosted.com with ESMTP id 2p3ps2j61k-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Mon, 03 Dec 2018 02:19:51 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cadence.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Oog4wjXbmSq5ZuRftG6A4/VaXCPVpgtjLAMFOOxaCFs=; b=qG+bmImT5uCRV7+fSwL+xPfJn7uJNSIuPzn/sXJKoHESZnfHuj6a1ZxOmP613Otu2XkkSW/MHA1HBHHXlGp3D03sAi7zKu4ZWoyrkMNsU+Jo/3bmbYHwjBdnduE0Zt5XZ8HxOPPS+rsyAUQKIGIt3EUVoCexQWIRCNs56niyY0A= Received: from BYAPR07MB4709.namprd07.prod.outlook.com (52.135.204.159) by BYAPR07MB4854.namprd07.prod.outlook.com (52.135.205.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1382.21; Mon, 3 Dec 2018 10:19:49 +0000 Received: from BYAPR07MB4709.namprd07.prod.outlook.com ([fe80::e0dc:ebd5:e248:d644]) by BYAPR07MB4709.namprd07.prod.outlook.com ([fe80::e0dc:ebd5:e248:d644%6]) with mapi id 15.20.1382.020; Mon, 3 Dec 2018 10:19:47 +0000 From: Pawel Laszczak To: Roger Quadros , "devicetree@vger.kernel.org" CC: "gregkh@linuxfoundation.org" , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Alan Douglas , "jbergsagel@ti.com" , "nsekhar@ti.com" , "nm@ti.com" , Suresh Punnoose , "peter.chen@nxp.com" , Pawel Jez , Rahul Kumar , Felipe Balbi Subject: RE: [RFC PATCH v2 08/15] usb:cdns3: Implements device operations part of the API Thread-Topic: [RFC PATCH v2 08/15] usb:cdns3: Implements device operations part of the API Thread-Index: AQHUfycXLoLM1JyA10aQRtXGNL+S26VlK76AgAQ822CAA3ct4A== Date: Mon, 3 Dec 2018 10:19:47 +0000 Message-ID: References: <1542535751-16079-1-git-send-email-pawell@cadence.com> <1542535751-16079-9-git-send-email-pawell@cadence.com> <5BFE8883.7090802@ti.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-dg-ref: PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNccGF3ZWxsXGFwcGRhdGFccm9hbWluZ1wwOWQ4NDliNi0zMmQzLTRhNDAtODVlZS02Yjg0YmEyOWUzNWJcbXNnc1xtc2ctZjU3ZGI5YWUtZjZlNC0xMWU4LTg3MjYtMWM0ZDcwMWRmYmE0XGFtZS10ZXN0XGY1N2RiOWFmLWY2ZTQtMTFlOC04NzI2LTFjNGQ3MDFkZmJhNGJvZHkudHh0IiBzej0iMjE3NCIgdD0iMTMxODgzMDU5ODc4MDAwNDQ0IiBoPSJOYVFJOFJwWWdQVmlldk1KbFM2aENhNmZwcW89IiBpZD0iIiBibD0iMCIgYm89IjEiLz48L21ldGE+ x-dg-rorf: x-originating-ip: [185.217.253.59] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;BYAPR07MB4854;20:UC1yqtsuYw9oJYXtvrmy30+1vdBDbrg7eRXKfn1v0jXgrpkmi3+pJqsa6Fi7JplWAwmphmdWzt79XgOiswxRRwv5CJtkX3+O/AgFwY3q8meckFxAkFLmT7l0QrVFxnhO7Ox3F5Q3QFpiVH/TRzYCR2mwuqld2w62YVktaIkjlx/a7gulZguQTOwCZU6VrbFJ1FkW1xlhb5qQScap9C73tVSRgpYpehHqerXwYD2p1Wxt4EJ5YcE/69fBl0jlMbwh x-ms-exchange-antispam-srfa-diagnostics: SOS;SOR; x-forefront-antispam-report: SFV:SKI;SCL:-1;SFV:NSPM;SFS:(10009020)(346002)(39860400002)(366004)(396003)(136003)(376002)(36092001)(199004)(189003)(51444003)(71190400001)(93886005)(229853002)(14454004)(6246003)(4326008)(53936002)(2906002)(2501003)(217873002)(55016002)(11346002)(3846002)(9686003)(68736007)(26005)(54906003)(110136005)(5660300001)(186003)(6436002)(478600001)(66066001)(6506007)(97736004)(446003)(81166006)(99286004)(8936002)(7416002)(316002)(476003)(86362001)(76176011)(8676002)(71200400001)(486006)(7696005)(81156014)(25786009)(105586002)(74316002)(106356001)(305945005)(33656002)(256004)(6116002)(14444005)(7736002)(102836004);DIR:OUT;SFP:1101;SCL:1;SRVR:BYAPR07MB4854;H:BYAPR07MB4709.namprd07.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;A:1;MX:1; x-ms-office365-filtering-correlation-id: ff91d4de-2e1d-4c11-7952-08d65908da58 x-microsoft-antispam: BCL:0;PCL:0;RULEID:(2390098)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(2017052603328)(7153060)(7193020);SRVR:BYAPR07MB4854; x-ms-traffictypediagnostic: BYAPR07MB4854: x-microsoft-antispam-prvs: x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(10201501046)(93006095)(93001095)(3231455)(999002)(944501493)(52105112)(3002001)(148016)(149066)(150057)(6041310)(20161123564045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123560045)(201708071742011)(7699051)(76991095);SRVR:BYAPR07MB4854;BCL:0;PCL:0;RULEID:;SRVR:BYAPR07MB4854; x-forefront-prvs: 08756AC3C8 received-spf: None (protection.outlook.com: cadence.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: PRO8fZE+IQzX1omAQRTCvZdQzDYX6CDA4a5Jtkhq5FKmhV+3tsKSfxefGElVtrN1bj58OAqIso04nG0VVSRRClaZmoE9WRGB5X6pNE+46VtwKT8xgKC22QXYRNjBBDd0a1p7qUXsRV3406o4uHB9OScsB1YRr45zT9v4afRWP750KQrWy+MBeqPOK3pxmG0yWbfVKUB+sS5877189lRyH7UpbNSJzALVKU7sjlsPsDtsOP47XHCRawSQvptXv9tG1dNoy2jLPF+aW2X/Q2xA8yqns01dtPEWyJRUkyo0ZjIL5kWVRqrs8gVPUe3UsMScc+RMHHBKf/ZQQZeGZmBEmVsTNjjntqu7NzPXqH/aOYw= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: cadence.com X-MS-Exchange-CrossTenant-Network-Message-Id: ff91d4de-2e1d-4c11-7952-08d65908da58 X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Dec 2018 10:19:47.6515 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: d36035c5-6ce6-4662-a3dc-e762e61ae4c9 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR07MB4854 X-Proofpoint-SPF-Result: pass X-Proofpoint-SPF-Record: v=spf1 include:_spf.salesforce.com include:mktomail.com include:spf-0014ca01.pphosted.com include:spf.protection.outlook.com include:auth.msgapp.com include:spf.mandrillapp.com ~all X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:,, definitions=2018-12-03_05:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_check_notspam policy=outbound_check score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1810050000 definitions=main-1812030100 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi >>> + >>> +static struct usb_ep *cdns3_gadget_match_ep(struct usb_gadget *gadget, >>> + struct usb_endpoint_descriptor *desc, >>> + struct usb_ss_ep_comp_descriptor *comp_desc) >>> +{ >>> + struct cdns3_device *priv_dev =3D gadget_to_cdns3_device(gadget); >>> + struct cdns3_endpoint *priv_ep; >>> + unsigned long flags; >>> + >>> + priv_ep =3D cdns3_find_available_ss_ep(priv_dev, desc); >>> + if (IS_ERR(priv_ep)) { >>> + dev_err(&priv_dev->dev, "no available ep\n"); >>> + return NULL; >>> + } >>> + >>> + dev_dbg(&priv_dev->dev, "match endpoint: %s\n", priv_ep->name); >>> + >>> + spin_lock_irqsave(&priv_dev->lock, flags); >>> + priv_ep->endpoint.desc =3D desc; >>> + priv_ep->dir =3D usb_endpoint_dir_in(desc) ? USB_DIR_IN : USB_DIR_OU= T; >>> + priv_ep->type =3D usb_endpoint_type(desc); >>> + >>> + list_add_tail(&priv_ep->ep_match_pending_list, >>> + &priv_dev->ep_match_list); >>> + spin_unlock_irqrestore(&priv_dev->lock, flags); >>> + return &priv_ep->endpoint; >>> +} >> >>Why do you need a custom match_ep? >>doesn't usb_ep_autoconfig suffice? > >I need to test it but at first glance it looks like usb_ep_autoconfig suf= fice. > >> >>You can check if EP is claimed or not by checking the ep->claimed flag. >> I checked it and did not work correct. The flag claimed is set in usb_ep_au= toconf, but=20 a little bit later during USB RESET this flag is cleared.=20 So, I can't base on this flag. I think that it's incorrect behavior in gad= get core driver or in function drivers. Maybe we should have additional flag in usb_ep object used during resetting= , or usb_ep_autoconf function should be called after each USB RESET.=20 I notice that I can based on ep->address it is set also in usb_ep_autoconf = and probably is not cleared anywhere. It's little tricky, but It looks like it works correct. I add some comment = for this checking.=20 Maybe in feature I will be able to replace it with claimed flag. Felipe what's your opinion about claimed flag. It's should be fixed or not= ? Cheers Pawel