Re: [PATCH 1/5] drivercore: add new error value for deferred probe

2011-10-07 Thread Greg KH
On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: Add new error value so that drivers can request deferred probe from drivercore. Signed-off-by: G, Manjunath Kondaiah manj...@ti.com Reported-by: Grant Likely grant.lik...@secretlab.ca --- Cc:

Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism

2011-10-07 Thread Greg KH
On Fri, Oct 07, 2011 at 10:33:07AM +0500, G, Manjunath Kondaiah wrote: From: Grant Likely grant.lik...@secretlab.ca Allow drivers to report at probe time that they cannot get all the resources required by the device, and should be retried at a later time. This should completely solve

Re: [PATCH 0/5] Driver Probe Deferral Mechanism

2011-10-07 Thread Greg KH
On Fri, Oct 07, 2011 at 10:33:05AM +0500, G, Manjunath Kondaiah wrote: Why did you send this series of patches out twice? Were they different? confused, greg k-h -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 0/5] Driver Probe Deferral Mechanism

2011-10-07 Thread G, Manjunath Kondaiah
Hi Greg, On Thu, Oct 06, 2011 at 11:50:42PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 10:33:05AM +0500, G, Manjunath Kondaiah wrote: Why did you send this series of patches out twice? Were they different? confused, Looks like this patch series has reached only individual recepients

Re: [PATCH 1/5] drivercore: add new error value for deferred probe

2011-10-07 Thread Mark Brown
On Thu, Oct 06, 2011 at 11:43:49PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: +#define EPROBE_DEFER 517 /* restart probe again after some time */ Can we really do this? Isn't this some user/kernel api here? What's wrong with

Re: [PATCH 4/5] gpiolib: handle deferral probe error

2011-10-07 Thread Alan Cox
On Fri, 07 Oct 2011 10:33:09 +0500 G, Manjunath Kondaiah manj...@ti.com wrote: The gpio library should return -EPROBE_DEFER in gpio_request if gpio driver is not ready. Why not use the perfectly good existing error codes we have for this ? We have EAGAIN and EUNATCH both of which look

Re: [PATCH 1/4] mmc: mmci: Bugfix in pio read for small packets

2011-10-07 Thread Ulf Hansson
Stefan NILSSON9 wrote: Hi Russel, On 10/01/2011 06:09 PM, Russell King - ARM Linux wrote: Does this even work? From the MMCI spec, I see no way for the MMCI peripheral to know the size of the read/write on the APB bus. The APB bus signals the MMCI uses are: PCLK - APB bus clock PRESETn -

Re: [PATCH 1/4] mmc: mmci: Bugfix in pio read for small packets

2011-10-07 Thread Ulf Hansson
Russell King - ARM Linux wrote: On Tue, Sep 27, 2011 at 09:46:08AM +0200, Ulf Hansson wrote: From: Stefan Nilsson XK stefan.xk.nils...@stericsson.com Corrects a bug in pio read when reading packets 4 bytes. These small packets are only relevant for SDIO transfers. Does this even work? From

Re: [PATCH] PM: HIBERNATION: add resume_wait param to support MMC-like devices as resume file

2011-10-07 Thread Barry Song
2011/10/7 Pavel Machek pa...@ucw.cz: 2011/9/29 Luca Tettamanti kronos...@gmail.com: On Thu, Sep 29, 2011 at 11:29 AM, Barry Song barry.s...@csr.com wrote: From: Barry Song baohua.s...@csr.com Some devices like mmc are async detected very slow. For example, drivers/mmc/host/sdhci.c

Re: [PATCH] PM: HIBERNATION: add resume_wait param to support MMC-like devices as resume file

2011-10-07 Thread Barry Song
2011/10/7 Barry Song 21cn...@gmail.com: 2011/10/7 Pavel Machek pa...@ucw.cz: 2011/9/29 Luca Tettamanti kronos...@gmail.com: On Thu, Sep 29, 2011 at 11:29 AM, Barry Song barry.s...@csr.com wrote: From: Barry Song baohua.s...@csr.com Some devices like mmc are async detected very slow. For

Re: [PATCH 1/4] mmc: mmci: Bugfix in pio read for small packets

2011-10-07 Thread Russell King - ARM Linux
On Fri, Oct 07, 2011 at 03:38:44PM +0200, Ulf Hansson wrote: In this hot path we already do a read of the FIFOCNT register for every loop in pio_read, won't this sometimes cause caches to flush and similar, thus cost quite a lot - at least a lot more than executing a switch/if sentence

Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism

2011-10-07 Thread Josh Triplett
On Thu, Oct 06, 2011 at 11:49:28PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 10:33:07AM +0500, G, Manjunath Kondaiah wrote: +config PROBE_DEFER + bool Deferred Driver Probe + default y + help + This option provides deferring driver probe if it has dependency on +

Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism

2011-10-07 Thread Greg KH
On Fri, Oct 07, 2011 at 01:57:15PM -0700, Josh Triplett wrote: On Thu, Oct 06, 2011 at 11:49:28PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 10:33:07AM +0500, G, Manjunath Kondaiah wrote: +config PROBE_DEFER + bool Deferred Driver Probe + default y + help + This option

Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism

2011-10-07 Thread Grant Likely
On Fri, Oct 7, 2011 at 12:49 AM, Greg KH g...@kroah.com wrote: On Fri, Oct 07, 2011 at 10:33:07AM +0500, G, Manjunath Kondaiah wrote: +config PROBE_DEFER +     bool Deferred Driver Probe +     default y +     help +       This option provides deferring driver probe if it has dependency on +

Re: [PATCH 4/5] gpiolib: handle deferral probe error

2011-10-07 Thread Grant Likely
On Fri, Oct 7, 2011 at 4:06 AM, Alan Cox a...@lxorguk.ukuu.org.uk wrote: On Fri, 07 Oct 2011 10:33:09 +0500 G, Manjunath Kondaiah manj...@ti.com wrote: The gpio library should return -EPROBE_DEFER in gpio_request if gpio driver is not ready. Why not use the perfectly good existing error

Re: [PATCH 1/5] drivercore: add new error value for deferred probe

2011-10-07 Thread Grant Likely
On Fri, Oct 7, 2011 at 12:43 AM, Greg KH g...@kroah.com wrote: On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: Add new error value so that drivers can request deferred probe from drivercore. Signed-off-by: G, Manjunath Kondaiah manj...@ti.com Reported-by: Grant Likely

Re: [PATCH 5/5] omap: hsmmc: use platform_driver_register

2011-10-07 Thread Grant Likely
On Thu, Oct 6, 2011 at 11:05 PM, G, Manjunath Kondaiah manj...@ti.com wrote: Existing omap hsmmc driver uses platform_driver_probe in init function. Change it to use platform_driver_register in order to use deferral probe mechanism. Signed-off-by: G, Manjunath Kondaiah manj...@ti.com

Re: [PATCH 0/2] MMC: disable multiblock reads on controllers that don't support them

2011-10-07 Thread Steve Sakoman
On Thu, Oct 6, 2011 at 1:50 PM, Paul Walmsley p...@pwsan.com wrote: Hello, Some MMC controller instances integrated on older OMAP34xx/35xx chips don't work correctly with multiple-block reads, due to a hardware bug. This series fixes the problem by adding an MMC capability flag for this case

Re: [PATCH 1/5] drivercore: add new error value for deferred probe

2011-10-07 Thread Valdis . Kletnieks
On Fri, 07 Oct 2011 16:12:45 MDT, Grant Likely said: On Fri, Oct 7, 2011 at 12:43 AM, Greg KH g...@kroah.com wrote: On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: +#define EPROBE_DEFER 517     /* restart probe again after some time */ Can we really do this?

Re: [PATCH 1/5] drivercore: add new error value for deferred probe

2011-10-07 Thread Greg KH
On Fri, Oct 07, 2011 at 07:28:33PM -0400, valdis.kletni...@vt.edu wrote: On Fri, 07 Oct 2011 16:12:45 MDT, Grant Likely said: On Fri, Oct 7, 2011 at 12:43 AM, Greg KH g...@kroah.com wrote: On Fri, Oct 07, 2011 at 10:33:06AM +0500, G, Manjunath Kondaiah wrote: +#define EPROBE_DEFER 517  

Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism

2011-10-07 Thread Josh Triplett
On Fri, Oct 07, 2011 at 02:23:26PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 01:57:15PM -0700, Josh Triplett wrote: On Thu, Oct 06, 2011 at 11:49:28PM -0700, Greg KH wrote: On Fri, Oct 07, 2011 at 10:33:07AM +0500, G, Manjunath Kondaiah wrote: +config PROBE_DEFER + bool