[PATCH] mmc: debugfs: expose the SDCLK frq in sys ios (V4)

2011-11-03 Thread Giuseppe CAVALLARO
This patch is to expose the actual SDCLK frequency in /sys/kernel/debug/mmcX/ios entry. For example, if the max clk for a normal speed card is 20MHz this is reported in /sys/kernel/debug/mmcX/ios. Unfortunately the actual SDCLK frequency (i.e. Baseclock / divisor) is not reported at all: for

Re: [PATCH] mmc: debugfs: expose the SDCLK frq in sys ios (v3)

2011-11-03 Thread Giuseppe CAVALLARO
On 11/2/2011 11:38 AM, Adrian Hunter wrote: On 27/10/11 09:16, Giuseppe CAVALLARO wrote: This patch is to expose the actual SDCLK frequency in /sys/kernel/debug/mmcX/ios entry. For example, if the max clk for a normal speed card is 20MHz this is reported in /sys/kernel/debug/mmcX/ios.

Re: [PATCH] mmc: debugfs: expose the SDCLK frq in sys ios (v3)

2011-11-03 Thread Giuseppe CAVALLARO
On 10/28/2011 12:02 AM, Sebastian Rasmussen wrote: Hi! +++ b/drivers/mmc/host/sdhci.c @@ -1107,9 +1113,13 @@ static void sdhci_set_clock(struct sdhci_host *host, unsigned int clock) if ((host-max_clk / div) = clock) break;

Re: [PATCH] mmc: debugfs: expose the SDCLK frq in sys ios (V4)

2011-11-03 Thread Adrian Hunter
On 03/11/11 08:45, Giuseppe CAVALLARO wrote: This patch is to expose the actual SDCLK frequency in /sys/kernel/debug/mmcX/ios entry. For example, if the max clk for a normal speed card is 20MHz this is reported in /sys/kernel/debug/mmcX/ios. Unfortunately the actual SDCLK frequency (i.e.

Re: [PATCH v3] mmc: sdhci: remove state argument from sdhci_suspend_host

2011-11-03 Thread Adrian Hunter
On 02/11/11 16:29, Manuel Lauss wrote: Drop the state argument from sdhci_suspend_host. Its only user is the PCI glue; this allows to move all SDHCI glues to use dev_pm_ops instead. Signed-off-by: Manuel Lauss manuel.la...@googlemail.com --- There is a checkpatch warning: WARNING:

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

2011-11-03 Thread S, Venkatraman
On Thu, Nov 3, 2011 at 5:29 AM, Kevin Hilman khil...@ti.com wrote: Hi Chris, Chris Ball c...@laptop.org writes: [...] Oops.  Thank you; I've pushed this to mmc-next now, and will send the fix to Linus.  I've checked for other errors in the original patch. Where did you push this fix? I

[PATCH] mmc: core: Quirk for long data read time

2011-11-03 Thread Stefan Nilsson XK
Adds a quirk that sets the data read timeout to a fixed value instead of relying on the information in the CSD. The timeout value choosen is 300ms since that has proven enough for the problematic cards found, but could be increased if other cards require this. This patch also enables this quirk

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

2011-11-03 Thread Chris Ball
Hi, On Thu, Nov 03 2011, S, Venkatraman wrote: Oops.  Thank you; I've pushed this to mmc-next now, and will send the fix to Linus.  I've checked for other errors in the original patch. Where did you push this fix? I don't see this in the mmc-next branch of git://dev.laptop.org/users/cjb/mmc

Re: [PATCH] mmc: core: Quirk for long data read time

2011-11-03 Thread Linus Walleij
2011/11/3 Stefan Nilsson XK stefan.xk.nils...@stericsson.com: Adds a quirk that sets the data read timeout to a fixed value instead of relying on the information in the CSD. The timeout value choosen is 300ms since that has proven enough for the problematic cards found, but could be increased

[PATCH] mmc: omap_hsmmc: fix typo around pr_info() call

2011-11-03 Thread Daniel Mack
This was introduced by a3c76eb9 (mmc: replace printk with appropriate display macro). Signed-off-by: Daniel Mack zon...@gmail.com Cc: Girish K S girish.shivananja...@linaro.org Cc: Chris Ball c...@laptop.org --- drivers/mmc/host/omap_hsmmc.c |2 +- 1 files changed, 1 insertions(+), 1

Re: [PATCH 1/1] [PATCH v4 1/1] mmc: Support of DUAL BUFFER DESC[ring] mode for dw_mmc

2011-11-03 Thread Shashidhar Hiremath
Hi Chris, Can this patch be accepted by criteria that its an additional feature supported by the hardware and hence good to have the support in the driver.Also note the patch has been tested. On Wed, Oct 5, 2011 at 7:44 AM, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Shashidhar. I tested

Re: [PATCH] mmc: omap_hsmmc: fix typo around pr_info() call

2011-11-03 Thread S, Venkatraman
On Thu, Nov 3, 2011 at 4:54 PM, Daniel Mack zon...@gmail.com wrote: This was introduced by a3c76eb9 (mmc: replace printk with appropriate display macro). Signed-off-by: Daniel Mack zon...@gmail.com Cc: Girish K S girish.shivananja...@linaro.org Cc: Chris Ball c...@laptop.org ---  

Re: [PATCH 1/1] [PATCH v4 1/1] mmc: Support of DUAL BUFFER DESC[ring] mode for dw_mmc

2011-11-03 Thread Chris Ball
Hi, On Thu, Nov 03 2011, Shashidhar Hiremath wrote: Hi Chris, Can this patch be accepted by criteria that its an additional feature supported by the hardware and hence good to have the support in the driver.Also note the patch has been tested. I think Will and James should make the call on

Re: [PATCH 1/1] [PATCH v4 1/1] mmc: Support of DUAL BUFFER DESC[ring] mode for dw_mmc

2011-11-03 Thread Will Newton
On Thu, Nov 3, 2011 at 12:35 PM, Chris Ball c...@laptop.org wrote: Hi, On Thu, Nov 03 2011, Shashidhar Hiremath wrote: Hi Chris,   Can this patch be accepted by criteria that its an additional feature supported by the hardware and hence good to have the support in the driver.Also note the

omap_hsmmc noisy/broken for suspend resume

2011-11-03 Thread Kevin Hilman
Hello, Using Linus' master branch, I was testing PM functionality on OMAP3/4 and noticed that the MMC driver is now rather noisy during suspend on OMAP3[1], but seems to work. On OMAP4, MMC seems to prevent suspend/resume all together. I haven't found the root cause, but know that by not

Re: [PATCH 1/1] [PATCH v4 1/1] mmc: Support of DUAL BUFFER DESC[ring] mode for dw_mmc

2011-11-03 Thread Jaehoon Chung
On 11/04/2011 12:18 AM, Will Newton wrote: On Thu, Nov 3, 2011 at 12:35 PM, Chris Ball c...@laptop.org wrote: Hi, On Thu, Nov 03 2011, Shashidhar Hiremath wrote: Hi Chris, Can this patch be accepted by criteria that its an additional feature supported by the hardware and hence good to

Re: omap_hsmmc noisy/broken for suspend resume

2011-11-03 Thread Girish K S
On 3 November 2011 17:24, Kevin Hilman khil...@ti.com wrote: Hello, Using Linus' master branch, I was testing PM functionality on OMAP3/4 and noticed that the MMC driver is now rather noisy during suspend on OMAP3[1], but seems to work. On OMAP4, MMC seems to prevent suspend/resume all