Re: MMC quirks relating to performance/lifetime.

2011-02-25 Thread Arnd Bergmann
On Friday 25 February 2011, Andrei Warkentin wrote: Yup. I understand :-). That's the strategy I'm going to follow. For page_size-alignment/splitting I'm looking at the block layer now. Is that the right approach or should I still submit a (cleaned up) patch to mmc/card/block.c for that

Re: [PATCH v4 5/6] OMAP: adapt hsmmc to hwmod framework

2011-02-25 Thread Krishnamoorthy, Balaji T
snip -               if (cpu_is_omap44xx()) -                       mmc-reg_offset = OMAP4_MMC_REG_OFFSET; -               else -                       mmc-reg_offset = 0; +static struct omap_mmc_platform_data *mmc __initdata; can you pass this *mmc as another argument to

Re: [PATCH v4 0/6] OMAP: HSMMC: hwmod adaptation

2011-02-25 Thread Cousson, Benoit
Hi Kishore, On 2/24/2011 8:26 PM, Kadiyala, Kishore wrote: Adding hwmod data for hsmmc device on OMAP2430/OMAP3/OMAP4. Adapting the omap_hsmmc driver to hwmod framework. The patch series is based on omap-for-linus and tested on OMAP2430, OMAP3430SDP OMAP4430SDP. Also boot tested on

Re: [PATCH v4 0/6] OMAP: HSMMC: hwmod adaptation

2011-02-25 Thread Kadiyala, Kishore
Hi Benoit, On Fri, Feb 25, 2011 at 6:24 PM, Cousson, Benoit b-cous...@ti.com wrote: Hi Kishore, On 2/24/2011 8:26 PM, Kadiyala, Kishore wrote: Adding hwmod data for hsmmc device on OMAP2430/OMAP3/OMAP4. Adapting the omap_hsmmc driver to hwmod framework. The patch series is based on

Re: [PATCH v4 0/6] OMAP: HSMMC: hwmod adaptation

2011-02-25 Thread Cousson, Benoit
On 2/25/2011 2:06 PM, Kadiyala, Kishore wrote: Hi Benoit, On Fri, Feb 25, 2011 at 6:24 PM, Cousson, Benoitb-cous...@ti.com wrote: Hi Kishore, On 2/24/2011 8:26 PM, Kadiyala, Kishore wrote: Adding hwmod data for hsmmc device on OMAP2430/OMAP3/OMAP4. Adapting the omap_hsmmc driver to hwmod

Re: [PATCH v4 4/6] OMAP: hwmod data: Add dev_attr and use in the host driver

2011-02-25 Thread Cousson, Benoit
Hi Kishore, On 2/24/2011 8:26 PM, Kadiyala, Kishore wrote: Add a device attribute to hwmod data of omap2430, omap3, omap4. Currently the device attribute holds information regarding dual volt MMC card support by the controller which will be later passed to the host driver via platform data.

Re: [PATCH v4 5/6] OMAP: adapt hsmmc to hwmod framework

2011-02-25 Thread Kadiyala, Kishore
On Fri, Feb 25, 2011 at 6:20 PM, Krishnamoorthy, Balaji T balaj...@ti.com wrote: snip -               if (cpu_is_omap44xx()) -                       mmc-reg_offset = OMAP4_MMC_REG_OFFSET; -               else -                       mmc-reg_offset = 0; +static struct omap_mmc_platform_data

Re: [PATCH v2 2/3] sdhci-pci : Enable runtime PM support

2011-02-25 Thread Matthew Garrett
On Fri, Feb 25, 2011 at 07:33:12AM +, Tardy, Pierre wrote: My understanding is the pci driver is not supposed to do any set_power_state/pci_save_state/wake_enable. Everything is supposed to be generically handle by pci frameworks's runtime_pm impl. Need confirmation from Rafael. The

[PATCH] mmc: sh_mmcif: support aggressive clock gating

2011-02-25 Thread Guennadi Liakhovetski
To support MMC aggressive clock gating the driver has to stop the interface clock, when the .set_ios() method is called with .clock == 0. Signed-off-by: Guennadi Liakhovetski g.liakhovet...@gmx.de --- Tested on ap4evb and ecovec drivers/mmc/host/sh_mmcif.c | 10 +- 1 files changed, 5

[PATCH v5 0/8] OMAP: HSMMC: hwmod adaptation

2011-02-25 Thread Kishore Kadiyala
Adding hwmod data for hsmmc device on OMAP2430/OMAP3/OMAP4. Adapting the omap_hsmmc driver to hwmod framework. Omap2420 platform consists of mmc block as in omap1 and not the hsmmc block as present in omap2430, omap3, omap4 platforms. The series takes care spliting out the mmc device init for

[PATCH v5 1/8] omap: mmc: split out init for 2420

2011-02-25 Thread Kishore Kadiyala
From: Anand Gadiyar gadi...@ti.com The MMC controller on the OMAP2420 is different from those on the OMAP2430, OMAP3 and OMAP4 families - all of the latter are identical. The one on the OMAP2420 is closer to that on OMAP1 chips. Currently, the n8x0 is the only OMAP2420 platform supported in

[PATCH v5 8/8] OMAP: hsmmc: Rename the device and driver

2011-02-25 Thread Kishore Kadiyala
Modifying the device driver name from mmci-omap-hs to omap_hsmmc. Signed-off-by: Kishore Kadiyala kishore.kadiy...@ti.com Acked-by: Benoit Coussonb-cous...@ti.com --- arch/arm/mach-omap2/board-2430sdp.c |2 +- arch/arm/mach-omap2/board-3430sdp.c |6 +++---

[PATCH v5 4/8] OMAP4: hwmod data: enable HSMMC

2011-02-25 Thread Kishore Kadiyala
From: Anand Gadiyar gadi...@ti.com Enabling hsmmc hwmod for OMAP4 Signed-off-by: Anand Gadiyar gadi...@ti.com Signed-off-by: Kishore Kadiyala kishore.kadiy...@ti.com Acked-by: Benoit Coussonb-cous...@ti.com --- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 10 +- 1 files changed, 5

[PATCH v5 6/8] OMAP: hsmmc: Move mux configuration to hsmmc.c

2011-02-25 Thread Kishore Kadiyala
Moving the definition of mux setting API from devices.c to hsmmc.c and renaming it from omap2_mmc_mux to omap_hsmmc_mux. Also calling omap_hsmmc_mux from omap2_hsmmc_init. Signed-off-by: Kishore Kadiyala kishore.kadiy...@ti.com Cc: Chris Ball c...@laptop.org Cc: Tony Lindgren t...@atomide.com ---

[PATCH v5 7/8] OMAP: adapt hsmmc to hwmod framework

2011-02-25 Thread Kishore Kadiyala
Changes involves: 1) Remove controller reset in devices.c which is taken care of by hwmod framework. 2) Omap2420 platform consists of mmc block as in omap1 and not the hsmmc block as present in omap2430, omap3, omap4 platforms. Removing all base address macro defines except keeping one

[PATCH v5 2/8] OMAP2430: hwmod data: Add HSMMC

2011-02-25 Thread Kishore Kadiyala
From: Paul Walmsley p...@pwsan.com Update the omap2430 hwmod data with the HSMMC info. Signed-off-by: Paul Walmsley p...@pwsan.com Signed-off-by: Kevin Hilman khil...@deeprootsystems.com Signed-off-by: Rajendra Nayak rna...@ti.com Signed-off-by: Kishore Kadiyala kishore.kadiy...@ti.com Cc:

[PATCH v5 3/8] OMAP3: hwmod data: Add HSMMC

2011-02-25 Thread Kishore Kadiyala
From: Paul Walmsley p...@pwsan.com Update the omap3 hwmod data with the HSMMC info. Signed-off-by: Paul Walmsley p...@pwsan.com Signed-off-by: Kevin Hilman khil...@deeprootsystems.com Signed-off-by: Rajendra Nayak rna...@ti.com Signed-off-by: Kishore Kadiyala kishore.kadiy...@ti.com Cc: Benoit

Re: [PATCH] mmc: sh_mmcif: support aggressive clock gating

2011-02-25 Thread Chris Ball
Hi Guennadi, On Fri, Feb 25, 2011 at 04:58:38PM +0100, Guennadi Liakhovetski wrote: To support MMC aggressive clock gating the driver has to stop the interface clock, when the .set_ios() method is called with .clock == 0. Signed-off-by: Guennadi Liakhovetski g.liakhovet...@gmx.de ---

Re: [RFC] dw_mmc: didn't support multiple blocks of weird length?

2011-02-25 Thread Will Newton
On Fri, Feb 25, 2011 at 7:35 AM, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi.. I didn't understand this point.. Plz let me explain this code.. why need this function (dw_mci_queue_request(host, slot, mrq)? I don't think there would be any problem merging those two functions. Also I would

Re: [PATCH 2/3] dw_mmc: support mmc power control with regulator

2011-02-25 Thread Will Newton
On Fri, Feb 25, 2011 at 2:08 AM, Jaehoon Chung jh80.ch...@samsung.com wrote: This patch is applied the power control with regulator. Signed-off-by: Jaehoon Chung jh80.ch...@samsung.com Signed-off-by: kyungmin Park kyungmin.p...@samsung.com ---  drivers/mmc/host/dw_mmc.c  |   25

Re: [PATCH] sdhci: always use max timeout for xfers

2011-02-25 Thread Wolfram Sang
Hi Philip, On Fri, Feb 25, 2011 at 09:54:35AM -0800, Philip Rakity wrote: Rather then special case busy etc .. lets just use the max value. OK. Did not remove BROKEN_TIMEOUT QUIRK so existing code will compile we can remove this once existing platform drivers delete usage and get quirk

Re: [PATCH] sdhci: always use max timeout for xfers

2011-02-25 Thread Philip Rakity
On Feb 25, 2011, at 10:02 AM, Wolfram Sang wrote: Hi Philip, On Fri, Feb 25, 2011 at 09:54:35AM -0800, Philip Rakity wrote: Rather then special case busy etc .. lets just use the max value. OK. Did not remove BROKEN_TIMEOUT QUIRK so existing code will compile we can remove this

Re: [PATCH] sdhci: always use max timeout for xfers

2011-02-25 Thread Wolfram Sang
- return count; + return 0xE; Why don't you remove the function entirely? better to rename it --- to set_maximum_timeout since a little clearer. left the old name for historical reasons -- if no need I will change it I'd suggest... } static void

Re: [PATCH v2 2/3] mmc: cb710: Return err value in cb710_wait_while_busy()

2011-02-25 Thread Michał Mirosław
On Wed, Feb 23, 2011 at 11:17:43PM +, Chris Ball wrote: Fixes: drivers/mmc/host/cb710-mmc.c: In function ‘cb710_wait_while_busy’: drivers/mmc/host/cb710-mmc.c:182:6: warning: variable ‘err’ set but not used [-Wunused-but-set-variable] Signed-off-by: Chris Ball c...@laptop.org Cc:

[PATCH V2] sdhci: always use max timeout for xfers

2011-02-25 Thread Philip Rakity
v2 use define for max timeout. remove subroutine call and just set the register directly v1 The card/host controller may sometimes return a value that is too low and cause the h/w to timeout a transfer that would have worked. Using the maximum value avoids this. Signed-off-by: Philip Rakity

Re: [PATCH v2 2/3] sdhci-pci : Enable runtime PM support

2011-02-25 Thread R. J. Wysocki
On Friday, February 25, 2011, Matthew Garrett wrote: On Fri, Feb 25, 2011 at 07:33:12AM +, Tardy, Pierre wrote: My understanding is the pci driver is not supposed to do any set_power_state/pci_save_state/wake_enable. Everything is supposed to be generically handle by pci frameworks's

Re: [PATCH V3 0/5] sdhci-esdhc-imx: use gpio for write protection and card detection

2011-02-25 Thread Rtp
Eric Benard e...@eukrea.com writes: Hi Arnaud, Salut ! On 24/02/2011 12:40, Arnaud Patard (Rtp) wrote: Wolfram Sangw.s...@pengutronix.de writes: Hi, Take #3, changes: * also intercept calls to SDHCI_SIGNAL_ENABLE (needed on mx25) * remove unconditional BROKEN_CARD_DETECTION

Re: [PATCH V2] sdhci: always use max timeout for xfers

2011-02-25 Thread Wolfram Sang
On Fri, Feb 25, 2011 at 10:49:25AM -0800, Philip Rakity wrote: v2 use define for max timeout. remove subroutine call and just set the register directly The generic description goes above the --- line, the incremental history of the patch usually below. v1 The card/host controller may

Re: [PATCH v2 5/5] mmc: sdhci-esdhc: enable esdhc on imx53

2011-02-25 Thread Wolfram Sang
On Tue, Feb 22, 2011 at 06:13:26PM +0800, Richard Zhu wrote: Fix the NO INT in the Multi-BLK IO in SD/MMC, and Multi-BLK read in SDIO This description is too short. Why does it not work before, and why does this patch help? Signed-off-by: Richard Zhu hong-xing@freescale.com ---

Re: [PATCH v5 7/8] OMAP: adapt hsmmc to hwmod framework

2011-02-25 Thread Tony Lindgren
* Kishore Kadiyala kishore.kadiy...@ti.com [110225 09:00]: +static int omap_mmc_init(struct omap_hwmod *oh, void *hsmmcinfo) ... + static int mmc_num; + + c += mmc_num; This does not look right.. Tony -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body

Re: [PATCH v5 7/8] OMAP: adapt hsmmc to hwmod framework

2011-02-25 Thread Tony Lindgren
* Tony Lindgren t...@atomide.com [110225 12:42]: * Kishore Kadiyala kishore.kadiy...@ti.com [110225 09:00]: +static int omap_mmc_init(struct omap_hwmod *oh, void *hsmmcinfo) ... + static int mmc_num; + + c += mmc_num; This does not look right.. So since mmc_num is static, it

Re: [PATCH V3 0/5] sdhci-esdhc-imx: use gpio for write protection and card detection

2011-02-25 Thread Eric Benard
Hi, On 25/02/2011 20:31, Arnaud Patard (Rtp) wrote: Eric Benarde...@eukrea.com writes: may you please test the attached patch. It may give someone with a better knowledge of sdhci than me an idea of what is wrong. I've tested this patch on my efikamx and this patch does solve the issue. I

Re: [PATCH v5 2/8] OMAP2430: hwmod data: Add HSMMC

2011-02-25 Thread Paul Walmsley
Hello Kishore, There are several problems with this data that generate warning messages on boot on 2430SDP. Did you actually test this on 2430SDP? The warning messages would have been obvious in the boot log. Comments below: On Fri, 25 Feb 2011, Kishore Kadiyala wrote: From: Paul Walmsley

Re: [PATCH] MMC: omap_hsmmc: enable interface clock before calling mmc_host_enable()

2011-02-25 Thread Paul Walmsley
Hi Chris, would you like to take this one, or should we merge it through the OMAP tree? - Paul From: Paul Walmsley p...@pwsan.com Date: Fri, 25 Feb 2011 18:22:08 -0700 Subject: [PATCH] MMC: omap_hsmmc: enable interface clock before calling mmc_host_enable() The code path entered via

Re: [PATCH] MMC: omap_hsmmc: enable interface clock before calling mmc_host_enable()

2011-02-25 Thread Chris Ball
Hi Paul, On Fri, Feb 25, 2011 at 06:26:45PM -0700, Paul Walmsley wrote: Hi Chris, would you like to take this one, or should we merge it through the OMAP tree? Thanks for asking -- OMAP folks seem to like merging patches to host/*omap* directly through the OMAP tree, and that's fine with

Re: [PATCH] MMC: omap_hsmmc: enable interface clock before calling mmc_host_enable()

2011-02-25 Thread Paul Walmsley
Hi Chris, On Sat, 26 Feb 2011, Chris Ball wrote: Thanks for asking -- OMAP folks seem to like merging patches to host/*omap* directly through the OMAP tree, and that's fine with me; I don't want to get in the way of a good workflow with responsive maintainers. I do like the patches to be

[PATCH V2] arm: mach-mmp: brownstone.c support multiple sd slots

2011-02-25 Thread Philip Rakity
Support multiple sd/eMMC interfaces. enable mmc1, 2, and 3. mmc2 is used eMMC and slot is marked PERMANENT and 8 bit device. mmc1 is used for Wifi and slot is marked PERMANENT Note: eMMC (mmc2) is set to initialize first to workaround a problem where booting in logical order requires mmc create