Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From	Felipe Balbi <balbi@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers
Date	Thu, 10 Dec 2015 18:20:03 +0100
Message-ID	<qEd2P-GC-21@gated-at.bofh.it> (permalink)
References	<qxV8B-7TP-13@gated-at.bofh.it> <qEcT8-Dg-19@gated-at.bofh.it> <qEcT8-Dg-21@gated-at.bofh.it>
X-Original-To	Marek Szyprowski <m.szyprowski@samsung.com>, <linux-usb@vger.kernel.org>, <linux-kernel@vger.kernel.org>
User-Agent	Notmuch/0.21 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-pc-linux-gnu)
MIME-Version	1.0
Content-Type	multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature"
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	77
Organization	linux.* mail to news gateway
X-Original-Cc	Marek Szyprowski <m.szyprowski@samsung.com>, Ruslan Bilovol <ruslan.bilovol@gmail.com>, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
X-Original-Date	Thu, 10 Dec 2015 11:13:24 -0600
X-Original-Message-ID	<878u52jkgb.fsf@saruman.tx.rr.com>
X-Original-References	<1448268998-4682-1-git-send-email-m.szyprowski@samsung.com> <87egeujl0r.fsf@saruman.tx.rr.com> <87bn9yjkn3.fsf@saruman.tx.rr.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1288698

Show key headers only | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

Felipe Balbi <balbi@ti.com> writes:
> Felipe Balbi <balbi@ti.com> writes:
>> Marek Szyprowski <m.szyprowski@samsung.com> 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

-- 
balbi

Thread

Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers Felipe Balbi <balbi@ti.com> - 2015-12-10 18:10 +0100
  Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers Felipe Balbi <balbi@ti.com> - 2015-12-10 18:10 +0100
    Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers Felipe Balbi <balbi@ti.com> - 2015-12-10 18:20 +0100
      Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and  gadget drivers Marek Szyprowski <m.szyprowski@samsung.com> - 2015-12-11 10:40 +0100

csiph-web