Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752747AbbLNIMY (ORCPT ); Mon, 14 Dec 2015 03:12:24 -0500 Received: from mail-bn1bon0113.outbound.protection.outlook.com ([157.56.111.113]:23998 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751783AbbLNIMV (ORCPT ); Mon, 14 Dec 2015 03:12:21 -0500 Authentication-Results: spf=permerror (sender IP is 192.88.168.50) smtp.mailfrom=freescale.com; samsung.com; dkim=none (message not signed) header.d=none;samsung.com; dmarc=none action=none header.from=freescale.com; Date: Mon, 14 Dec 2015 16:08:10 +0800 From: Peter Chen To: Felipe Balbi CC: Marek Szyprowski , , , Ruslan Bilovol , Bartlomiej Zolnierkiewicz Subject: Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers Message-ID: <20151214080809.GA6211@shlinux2> References: <1448268998-4682-1-git-send-email-m.szyprowski@samsung.com> <87egeujl0r.fsf@saruman.tx.rr.com> <87bn9yjkn3.fsf@saruman.tx.rr.com> <878u52jkgb.fsf@saruman.tx.rr.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <878u52jkgb.fsf@saruman.tx.rr.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BN1AFFO11FD040;1:Oej38irdYvytU1yvxR56W8PJY0PgaVblJGjj6I/hbM0nPezTwN37vfEnunHCZqZOSuEQELpsPUcRL87c1EoCJiJHah/4O6gZVf+XoA/lbZtq3jq0MIDnT7TP6OtrCcKVaNXi/s3Ox9rQ+h3QSMno9iQLsK+BesLvu8dRaQN/b2qsSZBnsqZpCQLwutwecMQu7F+feNtQQEsbQRKwpbgiskvlmGGwcliASRmRuoV+uiC4+oSabtN9gxLzVaY6/V8XrH9kmaVhO1rStpuTbHcVUSLS7FqqFsiXDDx6CLKfKrrIYKdE7mCjb4lMKrQIqUkUIAiMkMZO+poN/TcRruvDt8fRv1F2swKwi5d0KyXeyBeTkndg3X36OcyDol5K2Wuc6xPmloQFjHrLG76LbsP7OkrpbYAvbZBJR2MjRa6NjVo= X-Forefront-Antispam-Report: CIP:192.88.168.50;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10019020)(6009001)(2980300002)(1060300003)(448002)(24454002)(199003)(189002)(6806005)(104016004)(5008740100001)(47776003)(77096005)(15975445007)(50466002)(97756001)(33716001)(87936001)(93886004)(19580405001)(85326001)(106466001)(46406003)(1076002)(5001960100002)(110436001)(33656002)(86362001)(2950100001)(606003)(19580395003)(4290100001)(97736004)(81156007)(4001350100001)(76176999)(92566002)(54356999)(83506001)(23726003)(1096002)(1220700001)(50986999)(11100500001)(110136002);DIR:OUT;SFP:1102;SCL:1;SRVR:CH1PR03MB611;H:tx30smr01.am.freescale.net;FPR:;SPF:PermError;PTR:InfoDomainNonexistent;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;CH1PR03MB611;2:ZN3W7NB1Yb9fMVk0KuqQEGItmhpKw5ftoeVK+LctZn9NmnEJYuXP+ekrAu5nYFKRTHI8OQql2pfQ6NRljeTgzzmfh4kpOOJP6AjbB3gFaERni3UVGRD4SPP+e5b4iDNWY7NYCuJXbosHarkt5yv75g==;3:BtEaURbpfItwk9AcfsHCq/G5EZ+mHipNCs4RBNVd6J3UbROnrxoiNGAbUJPYqg/5RhwVXCk2V2dT6cc7RPYf2DpB2wIqJ48Tti4F4bbnGi5MFaszR45WIIcSJpd91v44DM6UCc3/q1bje/p1AQbKdBNvZlH3rV4CMJufhwU5mGCMhs5RY6KXwYvHSJpJZ0ZfIlwh7Koj6JkQG4GYxD9kUniHvv0YA5QEx+X8Ukx2oaQ=;25:SyEz51TTjR4c2xs6HK4+UiD5vAbyb+9xpE+VIA1FS0p5f0FI+nJ2nLMXL5p3VIZJY0UlADh4FcTtlF4SwQ4MVP2tYSRaW9bQPSMYgumLsfWFOm1ADkKtknaGd2UmtJNhprdZ3MgxCrFLBjwp4oO/0U1sAg81+acACoqRVZfCfGWQN90zbPzWZVNm0cVEQ5jofgrSMTvHILfhfC1jYestl/xyRiUqcg9TY1emnvE/jh7f5W4jZuGcd/JPzHkhrZk2kVbOf3PtBDOhcn+i1w8sZg== X-Microsoft-Antispam: UriScan:;BCL:1;PCL:0;RULEID:;SRVR:CH1PR03MB611; X-Microsoft-Exchange-Diagnostics: 1;CH1PR03MB611;20:JLwtr+eOG/mSmV64xG/bkzRoyw36PDnpfm8isMSr7f5weCcR6UINttb3Sg1Wdfo/cc3o4vQdRFy4UGJ1F23XqHGQWEt4/GR8G/R+I8xZ/ORR49eYvDdNx22L99DnJy9HNdtacElaFMqaM0quSrne6YILwtGdECOx8RyG9zuOUCFP7wKc+YbNvEEGtMQ9ZZu5ihZXggcDnMyrmvnvDmO78EH9EKqn26ZiBhQr7FP54KR8VhAsWGz/aT5DMUjuVtQuWtpHnGmPiQ9o9fq1zamrtnamFIh351SS6nXxRmTICTYGEmeE29FkKum/xMaWXGPcB1xpbz7IYLUHj8IfPUhyjX70PwMQY5Z7o3UThoM1SeE=;4:iEgdGg836NSMkgnp/cXeCZCryBk+4SfpXnwfcHuS6uc+2dwp7K4+b98l3tJQOqwuK7n6gXb2wMOsg/nQJyckTzAKIb1yoUW9k/ZcJiZH0CcNtXoWeR3wWRkRA5+I8MKVWBmRakuZMPtrHIWR9gUALSrkWToJWU4a2jPcDFgCteU+F94cY9mKaslDOUVAJ38trLgr+GaQX8RQjD6es9BwRxMT67IaCLdj1LOqor6fzve/HYAttLe+yHbKn3Su0O+ihQ9bq61NybH0IFLG/aX4ZzR2x7CKOpVCnylvWtFq8n03yL5iITrkTxmcvRN1B0IK3XwcxPHnDfbZYx6fVsgnYhCjUm/U0rMSmzuZOUHFsfJ8OBLD/QRqfC1vhUJFEY3zMbTzLrOwt7KBL4Y8ATj5/A== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:1;PCL:0;RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(1201001)(3002001)(10201501046);SRVR:CH1PR03MB611;BCL:1;PCL:0;RULEID:;SRVR:CH1PR03MB611; X-Forefront-PRVS: 0790FB1F33 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CH1PR03MB611;23:rxe/M0J6gawnVimX/0nBkRbbPvYNDFoMIUrPGzvv2O?= =?us-ascii?Q?SIltpZPWFQWthUFz/FmBshKzuq+hhs9wc+uVbGKKfo5LUS8ET2wY9S/XnAgG?= =?us-ascii?Q?LIMtk0hTzzq3fxfVCyrtgHqdrDoAmBnLrfDyOjAm62IWiTq1Cf3cAat+OPpl?= =?us-ascii?Q?txTrqIIxsZdQ581LzS1BhGbDu+jFoYVK+aTen3yivPglRwx3gQ79aSwOS6Cp?= =?us-ascii?Q?ZnlBVl2ZNXmpOJ0R2xMguzxvndqmRzYrFm6+CqD2RSLAzZOGfWT8HxEWSczh?= =?us-ascii?Q?nceBWcDhuIzYde1CUKEIzMuaCICZdg/sSaxkqSKxndhZw5C8jXxclxP6yPLJ?= =?us-ascii?Q?SoUm195Db2JKu2fDIL3SdmkBUUV5Z1Au6PIvybTkXdKIamOukO0HYdy3PF60?= =?us-ascii?Q?zWtZ0r0WiDQsFpgAiNQZ29GC/OxbjS7ZxcSCN2C27jEFQVYUj6LV5nhxsm3x?= =?us-ascii?Q?pUqpml1bRr8gsdZRFsS9F0hnEX2ZiDoeL3iHJWDb1MPTnmFmc/cQV/nAHHSu?= =?us-ascii?Q?fz3+KqtsW9BSc7D04zlGcmWWavuCw9BUOt6eZLN8WNoYDICLZm71H1ps4NBf?= =?us-ascii?Q?7n4Flp8eW20tACt1s/ZSu2CjSO34tsvqxHvNbPJKtXcYuu8H23kLzjsNZwU+?= =?us-ascii?Q?DRYwQ0e0mHG+8rhItm6MG8ZlyRn0Wn0KKVMyuLWXejLDWygHpTqzA7Ak/S+6?= =?us-ascii?Q?DuHgU2BUoPhe1aCfvv/8ffI+tT0FZyQXWxrfJ54UD9TKDkL1rlqJTlTDQF/w?= =?us-ascii?Q?mtKhAfn1tt2NNrHusE7CE39fFOb5tEG3BhYcWpFoX155GMeH2xCogF03pMxa?= =?us-ascii?Q?3jSkmn5VuTMKC5h3kVm+oes4KJdoZCTgJnJP82IBxpPjYlBMiksaZBSosENX?= =?us-ascii?Q?yCjLRcr7J76ePX+aYRIkuXyTq7st86azIKiDp5iu2nIJFsi/p/LCDb/p3JUt?= =?us-ascii?Q?o4171kXwSdjby0BEcANWQTv3whlpgAG7YqBtljlEX5FUJdES57ypyYXI5ckq?= =?us-ascii?Q?2PZwuH+rDisneNjnJDqKKWxW4xvkmylCBYJHRpMYn5AywwNS777EPjGZEshF?= =?us-ascii?Q?u3B04NIAUIqiqTwnNjaHbKyt8rKklb4TLtdBnC5RQpJvGGghCth1osIAnL7r?= =?us-ascii?Q?NGJmklMP63W+uzK1DerGQx8k2VY37/CIqXmtmqIiyiWLMHsvYi9g=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;CH1PR03MB611;5:wOlGT+gPaWWVqxfc+f+BqXIF88xSQq/gfXVDQ8JtjK7372ihdG/VRIkdLmhJ/dJRssTf5LEjOiARVrZbO7bhikHpSc6g43IfT7WzAQMIhl9W7mWP5pEKKp+b/cAUHeK1Y+QlBTfUOf4R1BQ1BbWYuA==;24:fK2orWwjeNU/asqkvca6FBd9k6sTQmImifV93l/48OY+u+rpxIlLPLHpqTZZqeHg5NVVAvBhBDrNBcXaEs8r//4KQxVsV1RC1AnOJbdKKu8= SpamDiagnosticOutput: 1:5 SpamDiagnosticMetadata: 00000000%2D0000%2D0000%2D0000%2D000000000000 SpamDiagnosticMetadata: 1 X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Dec 2015 08:12:18.5070 (UTC) X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH1PR03MB611 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2614 Lines: 64 On Thu, Dec 10, 2015 at 11:13:24AM -0600, Felipe Balbi wrote: > > Hi, > > Felipe Balbi writes: > > Felipe Balbi writes: > >> Marek Szyprowski writes: > >>> Hello, > >>> > >>> This is a resurrection of the patches initially submitted by Ruslan > >>> Bilovol in the following thread: https://lkml.org/lkml/2015/6/22/554 > >>> > >>> The changes since the original submission (v5) includes rebase onto > >>> latest linux-next branch, simplification of the code requested by Alan > >>> Stern and Felipe Balbi, removal of a patch, which deleted __init/__exit > >>> attributes (this change has been already merged) and fixes of the > >>> checkpatch issues. > >>> > >>> This feature is urgently needed, because it is not longer possible to > >>> use workaround to avoid deferred probe in UDC drivers due to > >>> not-yet-probed i2c regulator drivers (for more information see > >>> https://lkml.org/lkml/2015/10/30/374 ). > >>> > >>> This patchset has been successfully tested on Odroid XU3 boards with > >>> DWC3 UDC driver being deferred by missing regulator drivers. > >> > >> there is one problem with this patchset. If I try to statically link > >> gadget drivers, only one can be chosen, even though I can enable both > >> dwc3 and dummy_hcd just fine. And, actually, this brings another > >> problem. How do we handle systems which have 2 USB peripheral > >> controllers (say, 2 instances of dwc3) and choose which gadget driver > >> will bind to which controller ? > > > > We also seem to have issues with Kconfig. If I try to make gadget driver > > built-in, when compiling I'll get asked again if I want gadget drivers > > built-in. > > > > Another one: I just tried dummy_hcd built-in, g_zero built-in, dwc3 as a > > module. I can never load anything to dwc3 ;-) > > In all fairness, none of these are regressions. Can we agree to look at > these during v4.5-rc so maybe v4.6 has a final solution ? > > cheers > Hi Felipe, To support all of these, it will let legacy gadget driver usage as same as configfs. If the user wants these relative new features, why not let they use configfs. For legacy gadget drivers, it is better to keep the main structure unchanging, then the user will not be surprise. Here, we just want a solution for known issue, but not new features. -- Best Regards, Peter Chen -- 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/