Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Mon, Apr 14, 2014 at 11:06 PM, Alexandre Courbot wrote: > On Tue, Apr 15, 2014 at 10:42 AM, Andrew Bresticker > wrote: >> If regulator_get_optional() returns EPROBE_DEFER, it indicates >> that the regulator may show up later (e.g. the DT property is >> present but the corresponding regulator

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Russell King - ARM Linux
On Tue, Apr 15, 2014 at 12:59:37PM -0700, Andrew Bresticker wrote: > I believe the crash Alex is seeing is due to a race between tearing > down the sdhci driver on probe deferral and the card-detect IRQ. > Looking at it now. You probably want to check out my massive sdhci patch set posted a while

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Tue, Apr 15, 2014 at 11:28 AM, Stephen Warren wrote: > On 04/14/2014 07:42 PM, Andrew Bresticker wrote: >> If regulator_get_optional() returns EPROBE_DEFER, it indicates >> that the regulator may show up later (e.g. the DT property is >> present but the corresponding regulator may not have

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Tue, Apr 15, 2014 at 12:03 PM, Tim Kryger wrote: > On Mon, Apr 14, 2014 at 6:42 PM, Andrew Bresticker > wrote: >> If regulator_get_optional() returns EPROBE_DEFER, it indicates >> that the regulator may show up later (e.g. the DT property is >> present but the corresponding regulator may not

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Tim Kryger
On Mon, Apr 14, 2014 at 6:42 PM, Andrew Bresticker wrote: > If regulator_get_optional() returns EPROBE_DEFER, it indicates > that the regulator may show up later (e.g. the DT property is > present but the corresponding regulator may not have probed). > Instead of continuing without the regulator,

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Stephen Warren
On 04/14/2014 07:42 PM, Andrew Bresticker wrote: > If regulator_get_optional() returns EPROBE_DEFER, it indicates > that the regulator may show up later (e.g. the DT property is > present but the corresponding regulator may not have probed). > Instead of continuing without the regulator, return

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Alexandre Courbot
On Tue, Apr 15, 2014 at 10:42 AM, Andrew Bresticker wrote: > If regulator_get_optional() returns EPROBE_DEFER, it indicates > that the regulator may show up later (e.g. the DT property is > present but the corresponding regulator may not have probed). > Instead of continuing without the

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Alexandre Courbot
On Tue, Apr 15, 2014 at 10:42 AM, Andrew Bresticker abres...@chromium.org wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator may not have probed). Instead of continuing

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Stephen Warren
On 04/14/2014 07:42 PM, Andrew Bresticker wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator may not have probed). Instead of continuing without the regulator, return

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Tim Kryger
On Mon, Apr 14, 2014 at 6:42 PM, Andrew Bresticker abres...@chromium.org wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator may not have probed). Instead of continuing

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Tue, Apr 15, 2014 at 12:03 PM, Tim Kryger tim.kry...@linaro.org wrote: On Mon, Apr 14, 2014 at 6:42 PM, Andrew Bresticker abres...@chromium.org wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Tue, Apr 15, 2014 at 11:28 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 04/14/2014 07:42 PM, Andrew Bresticker wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Russell King - ARM Linux
On Tue, Apr 15, 2014 at 12:59:37PM -0700, Andrew Bresticker wrote: I believe the crash Alex is seeing is due to a race between tearing down the sdhci driver on probe deferral and the card-detect IRQ. Looking at it now. You probably want to check out my massive sdhci patch set posted a while

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Andrew Bresticker
On Mon, Apr 14, 2014 at 11:06 PM, Alexandre Courbot gnu...@gmail.com wrote: On Tue, Apr 15, 2014 at 10:42 AM, Andrew Bresticker abres...@chromium.org wrote: If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but

[PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-14 Thread Andrew Bresticker
If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator may not have probed). Instead of continuing without the regulator, return EPROBE_DEFER from sdhci_add_host(). Also, fix regulator

[PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-14 Thread Andrew Bresticker
If regulator_get_optional() returns EPROBE_DEFER, it indicates that the regulator may show up later (e.g. the DT property is present but the corresponding regulator may not have probed). Instead of continuing without the regulator, return EPROBE_DEFER from sdhci_add_host(). Also, fix regulator