All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, patrice.chotard@st.com,
	srinivas.kandagatla@gmail.com, chris@printf.net,
	ulf.hansson@linaro.org
Cc: lee.jones@linaro.org, devicetree@vger.kernel.org,
	linux-mmc@vger.kernel.org, peppe.cavallaro@st.com
Subject: Re: [PATCH v2 6/8] mmc: sdhci-st: Update the quirks for this controller.
Date: Tue, 3 Mar 2015 11:57:47 +0100	[thread overview]
Message-ID: <54F593AB.5080400@st.com> (raw)
In-Reply-To: <1424956227-18258-7-git-send-email-peter.griffin@linaro.org>


On 02/26/2015 02:10 PM, Peter Griffin wrote:
> Some additional quirks need to be enabled now we support UHS
> modes. This avoids some spurious warnings like
>
> "Got data interrupt 0x00000002 even though no data operation was in progress"
>
> Testing on stih410-b2120 board achieves the following speeds
> with HS200 eMMC card.
>
> max-frequency = 200Mhz
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 270 MB in  3.02 seconds =  89.54 MB/sec
>
> max-frequency = 100Mhz
> root@debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 210 MB in  3.00 seconds =  70.00 MB/sec
>
> max-frequency = 50Mhz
> root@debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 118 MB in  3.00 seconds =  39.28 MB/sec
>
> This is better than the 3.10 kernel which achieves 77.59 MB/sec
> at 200Mhz clock (same board/soc/eMMC).
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> Signed-off-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> ---
>   drivers/mmc/host/sdhci-st.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>

Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

WARNING: multiple messages have this Message-ID (diff)
From: maxime.coquelin@st.com (Maxime Coquelin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 6/8] mmc: sdhci-st: Update the quirks for this controller.
Date: Tue, 3 Mar 2015 11:57:47 +0100	[thread overview]
Message-ID: <54F593AB.5080400@st.com> (raw)
In-Reply-To: <1424956227-18258-7-git-send-email-peter.griffin@linaro.org>


On 02/26/2015 02:10 PM, Peter Griffin wrote:
> Some additional quirks need to be enabled now we support UHS
> modes. This avoids some spurious warnings like
>
> "Got data interrupt 0x00000002 even though no data operation was in progress"
>
> Testing on stih410-b2120 board achieves the following speeds
> with HS200 eMMC card.
>
> max-frequency = 200Mhz
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 270 MB in  3.02 seconds =  89.54 MB/sec
>
> max-frequency = 100Mhz
> root at debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 210 MB in  3.00 seconds =  70.00 MB/sec
>
> max-frequency = 50Mhz
> root at debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 118 MB in  3.00 seconds =  39.28 MB/sec
>
> This is better than the 3.10 kernel which achieves 77.59 MB/sec
> at 200Mhz clock (same board/soc/eMMC).
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> Signed-off-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> ---
>   drivers/mmc/host/sdhci-st.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>

Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <patrice.chotard@st.com>,
	<srinivas.kandagatla@gmail.com>, <chris@printf.net>,
	<ulf.hansson@linaro.org>
Cc: <lee.jones@linaro.org>, <devicetree@vger.kernel.org>,
	<linux-mmc@vger.kernel.org>, <peppe.cavallaro@st.com>
Subject: Re: [PATCH v2 6/8] mmc: sdhci-st: Update the quirks for this controller.
Date: Tue, 3 Mar 2015 11:57:47 +0100	[thread overview]
Message-ID: <54F593AB.5080400@st.com> (raw)
In-Reply-To: <1424956227-18258-7-git-send-email-peter.griffin@linaro.org>


On 02/26/2015 02:10 PM, Peter Griffin wrote:
> Some additional quirks need to be enabled now we support UHS
> modes. This avoids some spurious warnings like
>
> "Got data interrupt 0x00000002 even though no data operation was in progress"
>
> Testing on stih410-b2120 board achieves the following speeds
> with HS200 eMMC card.
>
> max-frequency = 200Mhz
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 270 MB in  3.02 seconds =  89.54 MB/sec
>
> max-frequency = 100Mhz
> root@debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 210 MB in  3.00 seconds =  70.00 MB/sec
>
> max-frequency = 50Mhz
> root@debian-armhf:~# hdparm -t /dev/mmcblk0p1
> /dev/mmcblk0p1:
>   Timing buffered disk reads: 118 MB in  3.00 seconds =  39.28 MB/sec
>
> This is better than the 3.10 kernel which achieves 77.59 MB/sec
> at 200Mhz clock (same board/soc/eMMC).
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> Signed-off-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> ---
>   drivers/mmc/host/sdhci-st.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>

Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

  reply	other threads:[~2015-03-03 10:57 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-26 13:10 [PATCH v2 0/8] Add sd/emmc support for stih407 family silicon Peter Griffin
2015-02-26 13:10 ` Peter Griffin
2015-02-26 13:10 ` [PATCH v2 1/8] mmc: sdhci-st: Add macros for register offsets and bitfields for mmcss glue regs Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:24   ` Maxime Coquelin
2015-03-03 10:24     ` Maxime Coquelin
2015-03-03 10:24     ` Maxime Coquelin
     [not found]     ` <54F58BED.4020900-qxv4g6HH51o@public.gmane.org>
2015-03-30 10:29       ` Peter Griffin
2015-03-30 10:29         ` Peter Griffin
2015-03-30 10:29         ` Peter Griffin
2015-02-26 13:10 ` [PATCH v2 2/8] mmc: sdhci-st: Add support for de-asserting reset signal and top regs resource Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:34   ` Maxime Coquelin
2015-03-03 10:34     ` Maxime Coquelin
2015-03-03 10:34     ` Maxime Coquelin
2015-02-26 13:10 ` [PATCH v2 3/8] mmc: sdhci-st: Add delay management functions for top registers (eMMC) Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:47   ` Maxime Coquelin
2015-03-03 10:47     ` Maxime Coquelin
2015-03-03 10:47     ` Maxime Coquelin
2015-03-30 10:33     ` Peter Griffin
2015-03-30 10:33       ` Peter Griffin
2015-02-26 13:10 ` [PATCH v2 4/8] mmc: sdhci-st: Add st_mmcss_cconfig function to configure mmcss glue registers Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:53   ` Maxime Coquelin
2015-03-03 10:53     ` Maxime Coquelin
2015-03-03 10:53     ` Maxime Coquelin
2015-02-26 13:10 ` [PATCH v2 5/8] mmc: sdhci-st: Add sdhci_st_set_uhs_signaling function Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:56   ` Maxime Coquelin
2015-03-03 10:56     ` Maxime Coquelin
2015-03-03 10:56     ` Maxime Coquelin
2015-03-30 10:46     ` Peter Griffin
2015-03-30 10:46       ` Peter Griffin
2015-02-26 13:10 ` [PATCH v2 6/8] mmc: sdhci-st: Update the quirks for this controller Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 10:57   ` Maxime Coquelin [this message]
2015-03-03 10:57     ` Maxime Coquelin
2015-03-03 10:57     ` Maxime Coquelin
2015-02-26 13:10 ` [PATCH v2 7/8] mmc: sdhci-st: Update ST SDHCI binding documentation Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 11:01   ` Maxime Coquelin
2015-03-03 11:01     ` Maxime Coquelin
2015-03-03 11:01     ` Maxime Coquelin
2015-02-26 13:10 ` [PATCH v2 8/8] ARM: STi: DT: STiH407: Add dt nodes for sdhci and emmc Peter Griffin
2015-02-26 13:10   ` Peter Griffin
2015-03-03 11:03   ` Maxime Coquelin
2015-03-03 11:03     ` Maxime Coquelin
2015-03-03 11:03     ` Maxime Coquelin
2015-03-30 11:23     ` Peter Griffin
2015-03-30 11:23       ` Peter Griffin
2015-03-30 11:37       ` Lee Jones
2015-03-30 11:37         ` Lee Jones
2015-03-30 11:43         ` Maxime Coquelin
2015-03-30 11:43           ` Maxime Coquelin
2015-03-30 11:43           ` Maxime Coquelin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54F593AB.5080400@st.com \
    --to=maxime.coquelin@st.com \
    --cc=chris@printf.net \
    --cc=devicetree@vger.kernel.org \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=patrice.chotard@st.com \
    --cc=peppe.cavallaro@st.com \
    --cc=peter.griffin@linaro.org \
    --cc=srinivas.kandagatla@gmail.com \
    --cc=ulf.hansson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.