Re: [PATCH v1 01/11] ARM: amba: Add Qualcomm vendor ID.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch adds Qualcomm amba vendor Id to the list. This ID is used in mmci driver. Signed-off-by: Srinivas Kandagatla srinivas.kandaga...@linaro.org (...) +

Re: [PATCHv2] mmc: dw_mmc: change to use recommended reset procedure

2014-05-13 Thread Sonny Rao
On Mon, May 12, 2014 at 10:02 PM, Seungwon Jeon tgih@samsung.com wrote: As I mentioned in previous version, you put all reset stuff in existing fifo_reset function. Although databook mentions ciu_reset case for SBE error, it's not obvious when ciu_reset is needed in other error cases.

Re: [PATCH v1 02/11] mmc: mmci: Add Qualcomm Id to amba id table

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch adds a fake Qualcomm ID 0x00051180 to the amba_ids, as Qualcomm SDCC controller is pl180, but amba id registers read 0x0's. The plan is to remove SDCC

Re: [PATCH v1 03/11] mmc: mmci: Add Qcom datactrl register variant

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Instance of this IP on Qualcomm's SOCs has bit different layout for datactrl register. Bit postion datactrl[16:4] hold the true block size instead of power of 2.

Re: [PATCH v1 05/11] mmc: mmci: use NSEC_PER_SEC macro

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch replaces a constant used in calculating timeout with a proper macro. This is make code more readable. Signed-off-by: Srinivas Kandagatla

Re: [PATCH v1 06/11] mmc: mmci: Qcomm: Add 3 clock cycle delay after register write

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Most of the Qcomm SD card controller registers must be updated to the MCLK domain so subsequent writes to registers will be ignored until 3 clock cycles have

Re: [PATCH v1 07/11] mmc: mmci: move ST specific register extensions access under condition.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch moves some of the ST specific register extensions access under condition, so that other SOCs like Qualcomm or ARM would not a side effect of writing to

Re: Re: mmc: core: retries counter for data command initialization.

2014-05-13 Thread Ulf Hansson
On 12 May 2014 16:33, Konstantin Dorfman kdorf...@codeaurora.org wrote: Hello Chris, Ulf, Sorry for nagging. No worries. Let me have some more thinking before I come back to you though. Kind regards Ulf Hansson Do you have opinion on issue with retries counter for data command? Since

Re: [PATCH v1 08/11] mmc: mmci: Qcom fix MCICLK register settings.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org MCICLK register layout is bit different to the standard pl180 register layout. Qcom SDCC controller some setup in MCICLK register to get it going. So this patch

Re: [PATCH v1 09/11] mmc: mmci: Add clock support for Qualcomm.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org MCICLK going to card bus is directly driven by the clock controller, so the driver has to set the required rates depending on the state of the card. This bit of

Re: [PATCH v1 10/11] mmc: mmci: Add Qcom variations to MCICommand register.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:21 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Some bits which control Command Path State Machine (CPSM) are new in Qcom integration, so this patch adds support to those bits. Signed-off-by: Srinivas

Re: [PATCH v1 11/11] mmc: mmci: Add Qcom specific pio_read function.

2014-05-13 Thread Linus Walleij
On Tue, Apr 29, 2014 at 10:21 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org MCIFIFOCNT register behaviour on Qcom chips is very different than the other pl180 integrations. MCIFIFOCNT register contains the number of words that are still

Re: [PATCHv2] mmc: dw_mmc: change to use recommended reset procedure

2014-05-13 Thread Arnd Bergmann
On Monday 12 May 2014 18:38:41 Sonny Rao wrote: +#ifndef CONFIG_MMC_DW_IDMAC + if (host-use_dma) + flags |= SDMMC_CTRL_DMA_RESET; +#endif Can you change the above like this? if (IS_ENABLED(CONFIG_MMC_DW_IDMAC) host-use_dma) flags |=

Re: [GIT PULL] MMC updates for mmc-next for 3.16, take 2

2014-05-13 Thread Ulf Hansson
On 13 May 2014 00:23, Chris Ball ch...@printf.net wrote: Hi Ulf, On Thu, May 08 2014, Ulf Hansson wrote: Note 2) I have not included the patch below; since I think it would be better if you just drop the patch which this is reverting from your mmc next branch. [PATCH 1/2] mmc: rtsx: Revert

Re: [PATCH v1 02/11] mmc: mmci: Add Qualcomm Id to amba id table

2014-05-13 Thread Srinivas Kandagatla
Thanks Linus W. On 13/05/14 08:17, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch adds a fake Qualcomm ID 0x00051180 to the amba_ids, as Qualcomm SDCC controller is pl180, but

Re: [PATCH v1 06/11] mmc: mmci: Qcomm: Add 3 clock cycle delay after register write

2014-05-13 Thread Srinivas Kandagatla
Thanks Linus W for reviewing the patches. On 13/05/14 08:29, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Most of the Qcomm SD card controller registers must be updated to the MCLK

Re: [PATCH v1 05/11] mmc: mmci: use NSEC_PER_SEC macro

2014-05-13 Thread Srinivas Kandagatla
On 13/05/14 08:20, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:20 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch replaces a constant used in calculating timeout with a proper macro. This is make code more readable.

Re: [PATCH v1 07/11] mmc: mmci: move ST specific register extensions access under condition.

2014-05-13 Thread Srinivas Kandagatla
On 13/05/14 09:08, Linus Walleij wrote: /* Keep ST Micro busy mode if enabled */ - datactrl |= host-datactrl_reg MCI_ST_DPSM_BUSYMODE; + if (host-hw_designer == AMBA_VENDOR_ST) + datactrl |= host-datactrl_reg MCI_ST_DPSM_BUSYMODE; Do not hard-check the

Re: [PATCH v1 08/11] mmc: mmci: Qcom fix MCICLK register settings.

2014-05-13 Thread Srinivas Kandagatla
Thanks Linus W. On 13/05/14 09:19, Linus Walleij wrote: Again follow the pattern of storing register templates in the vendor_data struct. I think you will quickly realize how this can be cut down with new fields like .clk_4bitmode etc. /* Modified PL180 on Versatile Express platform */

Re: [PATCH v1 09/11] mmc: mmci: Add clock support for Qualcomm.

2014-05-13 Thread Srinivas Kandagatla
Thanks Linus W, On 13/05/14 09:28, Linus Walleij wrote: code is conditioned on hw designer. Signed-off-by: Srinivas Kandagatla srinivas.kandaga...@linaro.org (...) + if (host-hw_designer == AMBA_VENDOR_QCOM) { + host-cclk = host-mclk; + }

Re: [PATCH v1 01/11] ARM: amba: Add Qualcomm vendor ID.

2014-05-13 Thread Srinivas Kandagatla
Thanks Linus W, On 13/05/14 08:16, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch adds Qualcomm amba vendor Id to the list. This ID is used in mmci driver. Signed-off-by:

Re: [PATCH v1 11/11] mmc: mmci: Add Qcom specific pio_read function.

2014-05-13 Thread Srinivas Kandagatla
On 13/05/14 09:34, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:21 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org MCIFIFOCNT register behaviour on Qcom chips is very different than the other pl180 integrations. MCIFIFOCNT register

Re: [PATCH v1 00/11] Add Qualcomm SD Card Controller support.

2014-05-13 Thread Ulf Hansson
On 29 April 2014 10:18, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Hi Russell, This patch series adds Qualcomm SD Card Controller support in pl180 mmci driver. QCom SDCC is basically a pl180, but bit more customized, some of the register

Re: [PATCH v1 00/11] Add Qualcomm SD Card Controller support.

2014-05-13 Thread Srinivas Kandagatla
On 13/05/14 11:04, Ulf Hansson wrote: On 29 April 2014 10:18, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org Hi Russell, This patch series adds Qualcomm SD Card Controller support in pl180 mmci driver. QCom SDCC is basically a pl180, but bit

Re: [PATCH] mmc: dw_mmc: change to use recommended reset procedure

2014-05-13 Thread James Hogan
Hi, On 12/05/14 22:44, Sonny Rao wrote: Doug mentioned that James Hogan might have an answer. James, are there Imgtec SoCs which use dw_mmc and use DMA but don't use the IDMAC? If so, we can add that support into this reset procedure patch. Yes, the Toumaz TZ1090 SoC has the dw_mmc

RE: [PATCHv2] mmc: dw_mmc: change to use recommended reset procedure

2014-05-13 Thread Seungwon Jeon
On Tuesday, May 13, Sonny Rao wrote: On Mon, May 12, 2014 at 10:02 PM, Seungwon Jeon tgih@samsung.com wrote: As I mentioned in previous version, you put all reset stuff in existing fifo_reset function. Although databook mentions ciu_reset case for SBE error, it's not obvious when

RE: [PATCH 3/5] mmc: dw_mmc-exynos: add the MMC_CAP_ERASE by default.

2014-05-13 Thread Seungwon Jeon
Hi Jaehoon, I think we can remove MMC_CAP_ERASE from host's cap. Can you check below patch? [PATCH v4] mmc: remove MMC_CAP_ERASE to enable trim in eMMC/SD Device Thanks, Seungwon Jeon On Mon, May 12, 2014, Jaehoon Chung wrote: To use the erase command, add the MMC_CAP_ERASE capability by

Re: [PATCH] tmio_mmc_pio: prevent endless loop in tmio_mmc_set_clock()

2014-05-13 Thread Sergei Shtylyov
Hello. On 05/13/2014 05:38 AM, Simon Horman wrote: I've spent a couple of days with the driver just hanging due to me forgetting to specify the external crystal frequency, so that clk_get_rate() returned 0 and thus the loop in tmio_mmc_set_clock() never ended. I don't think that's an

Re: [PATCH 3/5] mmc: dw_mmc-exynos: add the MMC_CAP_ERASE by default.

2014-05-13 Thread Jaehoon Chung
Hi, Seungwon. On 05/13/2014 08:15 PM, Seungwon Jeon wrote: Hi Jaehoon, I think we can remove MMC_CAP_ERASE from host's cap. Thanks for reminding. :) It's my mistake..Confused with linux-3.10. Best Regards, Jaehoon Chung Can you check below patch? [PATCH v4] mmc: remove MMC_CAP_ERASE

Re: [PATCH] mmc: dw_mmc: Make sure we don't get stuck when we get an error

2014-05-13 Thread Doug Anderson
Seungwon, On Mon, May 12, 2014 at 9:52 PM, Seungwon Jeon tgih@samsung.com wrote: Hi Doug, On Tue, May 13, 2014, Doug Anderson wrote: Seungwon, On Sat, May 10, 2014 at 7:11 AM, Seungwon Jeon tgih@samsung.com wrote: On Fri, May 09, 2014, Sonny Rao wrote: On Thu, May 8, 2014 at

Re: [PATCH] tmio_mmc_pio: prevent endless loop in tmio_mmc_set_clock()

2014-05-13 Thread Simon Horman
On Tue, May 13, 2014 at 04:09:57PM +0400, Sergei Shtylyov wrote: Hello. On 05/13/2014 05:38 AM, Simon Horman wrote: I've spent a couple of days with the driver just hanging due to me forgetting to specify the external crystal frequency, so that clk_get_rate() returned 0 and thus the

Re: [PATCH v1 01/11] ARM: amba: Add Qualcomm vendor ID.

2014-05-13 Thread Stephen Boyd
On 05/13, Srinivas Kandagatla wrote: Thanks Linus W, On 13/05/14 08:16, Linus Walleij wrote: On Tue, Apr 29, 2014 at 10:19 AM, srinivas.kandaga...@linaro.org wrote: From: Srinivas Kandagatla srinivas.kandaga...@linaro.org This patch adds Qualcomm amba vendor Id to the list. This ID is

Re: [PATCH] mmc: sdhci: remove mdelay in eMMC tuning

2014-05-13 Thread Grant Grundler
On Wed, May 7, 2014 at 1:35 AM, Ulf Hansson ulf.hans...@linaro.org wrote: On 7 May 2014 03:52, Nick Sanders nsand...@chromium.org wrote: This patch removes an unneccesary 1ms mdelay in the HS200 tuning loop, called 40 times per retuning. Currently this causes a latency of 40ms on any emmc

Re: [PATCH] mmc: sdhci: remove mdelay in eMMC tuning

2014-05-13 Thread Chris Ball
Hi, On Wed, May 14 2014, Grant Grundler wrote: The delay is left in place for SD Cards, which use MMC_SEND_TUNING_BLOCK rather than MMC_SEND_TUNING_BLOCK_HS200. I'm not able to find evidence that this is required for SD in the specs I have access to, however this delay has been present from

Re: [PATCH 2/5] mmc: dw_mmc: remove the supports-highspeed property.

2014-05-13 Thread Jaehoon Chung
Hi, Dinh. On 05/13/2014 12:48 AM, Dinh Nguyen wrote: On Mon, 2014-05-12 at 20:14 +0900, Jaehoon Chung wrote: Remove the support-highspeed property in dw-mmc. support-highspeed property can be replaced to cap-sd/mmc-highspeed. Signed-off-by: Jaehoon Chung jh80.ch...@samsung.com ---

Re: [PATCH 2/5] mmc: dw_mmc: remove the supports-highspeed property.

2014-05-13 Thread Jaehoon Chung
On 05/13/2014 02:43 AM, Dinh Nguyen wrote: On Mon, 2014-05-12 at 20:14 +0900, Jaehoon Chung wrote: Remove the support-highspeed property in dw-mmc. support-highspeed property can be replaced to cap-sd/mmc-highspeed. s/support-highspeed/supports-highspeed Will fix. Dinh -- To