From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Wolfram Sang <wsa@the-dreams.de>
Cc: linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-i2c@vger.kernel.org,
Krzysztof Kozlowski <k.kozlowski@samsung.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Robert Baldyga <r.baldyga@samsung.com>,
"balbi@ti.com" <balbi@ti.com>
Subject: Re: [PATCH] i2c: exynos5: register driver early in subsys initcall to avoid probe defer
Date: Thu, 19 Nov 2015 14:20:41 +0100 [thread overview]
Message-ID: <564DCCA9.3050604@samsung.com> (raw)
In-Reply-To: <20151101160203.GB2295@katana>
Hello,
On 2015-11-01 17:02, Wolfram Sang wrote:
> On Fri, Oct 30, 2015 at 03:23:28PM +0100, Marek Szyprowski wrote:
>> i2c bus drivers should be registered earlier than typical drivers to let
>> important i2c devices (like PMICs/regulators) get registered early
>> enough to avoid probe defer of all devices in typical embedded system.
>>
>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> I don't accept such patches anymore. Some more background:
>
> http://www.spinics.net/lists/linux-i2c/msg13102.html
> http://www.spinics.net/lists/linux-i2c/msg16644.html
Well, then how can we get Exynos5xxx-based boards working with USB gadget
driver compiled-in? USB Device controller driver (UDC) needs regulator to
probe successfully. On the other hand USB gadget subsystem doesn't wait
until all UDC drivers gets probed and fails to bind compiled-in gadget
drivers if no UDC is available on probe time.
This problem is known for ages, there were number of attempts to add
support for deferred probe or solve the problem of deferred UDC driver
probe, but none got accepted so far:
http://www.spinics.net/lists/linux-usb/msg102795.html (20 Feb 2014)
http://www.spinics.net/lists/linux-usb/msg105279.html (2 Apr 2014)
http://thread.gmane.org/gmane.linux.usb.general/109745 (10 Jun 2014)
https://lkml.org/lkml/2015/6/22/554 (23 Jun 2015)
Till now it worked somehow for boards with older i2c bus drivers, which
were registered in subsys initcall. Boards, which use newer i2c bus drivers
cannot be used with usb gadgets compiled-in.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
next prev parent reply other threads:[~2015-11-19 13:20 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-30 14:23 [PATCH] i2c: exynos5: register driver early in subsys initcall to avoid probe defer Marek Szyprowski
2015-11-01 16:02 ` Wolfram Sang
2015-11-19 13:20 ` Marek Szyprowski [this message]
2015-11-19 13:46 ` Wolfram Sang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=564DCCA9.3050604@samsung.com \
--to=m.szyprowski@samsung.com \
--cc=b.zolnierkie@samsung.com \
--cc=balbi@ti.com \
--cc=k.kozlowski@samsung.com \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=r.baldyga@samsung.com \
--cc=wsa@the-dreams.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox