From: Felipe Balbi <balbi@ti.com>
To: Marek Szyprowski <m.szyprowski@samsung.com>, <linux-usb@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
CC: Marek Szyprowski <m.szyprowski@samsung.com>,
        Ruslan Bilovol <ruslan.bilovol@gmail.com>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [PATCH v7 0/4] usb/gadget: independent registration of gadgets and gadget drivers
In-Reply-To: <87bn9yjkn3.fsf@saruman.tx.rr.com>
References: <1448268998-4682-1-git-send-email-m.szyprowski@samsung.com> <87egeujl0r.fsf@saruman.tx.rr.com> <87bn9yjkn3.fsf@saruman.tx.rr.com>
User-Agent: Notmuch/0.21 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-pc-linux-gnu)
Date: Thu, 10 Dec 2015 11:13:24 -0600
Message-ID: <878u52jkgb.fsf@saruman.tx.rr.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1;
	protocol="application/pgp-signature"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 3065
Lines: 77

--=-=-=
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable


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

=2D-=20
balbi

--=-=-=
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBAgAGBQJWabK1AAoJEIaOsuA1yqREdh4P/jtIWCrzBp2EWKrUeL0VHeKB
kSTJfwAEabxcm9azLwMf++En/56gAOSztwtcdr238CrONvcM3BvRixGDCZos8buG
leyEVHrSSa1S5Gx3NeFnqyBC20jgQdl4Wu+oq+RYqLBVSWsZh8pYQ/JwJwSxHk4P
OtAJiTN+iXIqO0tRnSP489bf+/pX6uybafVrQLavNLWWWREpJHjKkKl+0950GCja
7AS0Fp3ZLf6J3yMfWeAhsBkzez4h8pmzAptHMYKjey8juEYVYhRlVymWZEpdAnwb
hr+Rl4oO2PBEOPMWemnbQvRZiUWMMvXjS/8+ndQXQb+K8JEWLxpoNot1HtzKwwBg
DCPmrccw1WrtEkkkljI2td41EVnLxjs53dLt085Hp6sCNvqWJADvLAyZxSVbATof
TqRD6LEc/iCaDEhMQ90IN2J47uOkgLELACwgFyoqWTfSQtsFUdFEZ1CYQUpTkv7i
8HfL4NHjr7alrq7KtuJLP9tBP1y2ywt87RfKh9nJaL9CyRVSOnL1Nkh3lwOTictM
ja2kr2A/KdZoyjbm8f11hMLGaZ9o3nZdvSG/UCmer4/rdfLNpurS8GmLSHVFQjat
ShAsnsJKO46cWGm0LjyYHhQfaFlX9fI+Rqvtcs4px8eKdAEP0k15/hUB16QUByb6
jKx49YjKctpcpuCGCTGJ
=H3uu
-----END PGP SIGNATURE-----
--=-=-=--
--
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/