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,
ulf.hansson@linaro.org, peppe.cavallaro@st.com
Cc: lee.jones@linaro.org, devicetree@vger.kernel.org,
linux-mmc@vger.kernel.org
Subject: Re: [PATCH v5 0/9] Add sd/emmc support for stih407 family silicon
Date: Thu, 30 Apr 2015 10:56:21 +0200 [thread overview]
Message-ID: <5541EE35.2070002@st.com> (raw)
In-Reply-To: <1428658831-14308-1-git-send-email-peter.griffin@linaro.org>
On 04/10/2015 11:40 AM, Peter Griffin wrote:
> Hi,
>
> This series adds sd/emmc support to the sdhci-st.c driver for stih407
> family silicon. The changes mainly involve configuring some extra glue
> registers in the flashSS which configure the Arasan controller.
>
> This series also adds support for UHS modes for eMMC. To allow
> UHS HS200/SD104 modes to function correctly, due to the
> tight timing constriants, support for delay management is also added.
> Two types of delay management are supported, static delay management and
> dynamic delay management, this delay management is only available
> on eMMC pads on stih410 and later silicon.
>
> This series has been tested with stih410-b2120 revd on eMMC and sd, at
> various clock speeds. As part of this testing a bug was also found in the
> upstream flexgen clock set_rate implementation (now fixed upstream).
>
> 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
>
> It has also been tested on stih416-b2020 to ensure we have caused no
> regressions. Finally the dt documentation has been updated to reflect
> the changes in the driver code. Intrestingly it seems we are the first
> upstream platform to be using some of the uhs bindings such as
> sd-uhs-sdr104.
>
> Changes since v4:
> - Fixup typo (Pete)
>
> Changes since v3:
> - Rebased on Ulf's mmc next branch (rc5 based) (Ulf)
>
> Changes since v2:
> - Some whitespace fixups (Max)
> - if (!ioaddr) suggestion (Max)
> - Add stih418-b2199 suport (Max)
> - Stih410 to STiH410 fixes (Max)
> - rebased on v4.0-rc6 (Pete)
>
> Changes since v1:
> - Partition the changes into smaller patches to aid review process (Ulf)
>
> Peter Griffin (9):
> mmc: sdhci-st: Add macros for register offsets and bitfields for mmcss
> glue regs
> mmc: sdhci-st: Add support for de-asserting reset signal and top regs
> resource
> mmc: sdhci-st: Add delay management functions for top registers
> (eMMC).
> mmc: sdhci-st: Add st_mmcss_cconfig function to configure mmcss glue
> registers.
> mmc: sdhci-st: Add sdhci_st_set_uhs_signaling function.
> mmc: sdhci-st: Update the quirks for this controller.
> mmc: sdhci-st: Update ST SDHCI binding documentation.
> ARM: STi: DT: STiH407: Add dt nodes for sdhci and emmc.
> ARM: STi: DT: STiH418: Add dt nodes for sdhci and emmc.
>
> Documentation/devicetree/bindings/mmc/sdhci-st.txt | 100 +++++-
> arch/arm/boot/dts/stih407-family.dtsi | 30 ++
> arch/arm/boot/dts/stih410-b2120.dts | 10 +
> arch/arm/boot/dts/stih418-b2199.dts | 12 +
> arch/arm/boot/dts/stihxxx-b2120.dtsi | 8 +
> drivers/mmc/host/sdhci-st.c | 354 ++++++++++++++++++++-
> 6 files changed, 500 insertions(+), 14 deletions(-)
>
Patches 8 & 9 applied to STi DT branch.
Thanks,
Maxime
prev parent reply other threads:[~2015-04-30 8:56 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-10 9:40 [PATCH v5 0/9] Add sd/emmc support for stih407 family silicon Peter Griffin
2015-04-10 9:40 ` [PATCH v5 1/9] mmc: sdhci-st: Add macros for register offsets and bitfields for mmcss glue regs Peter Griffin
2015-04-10 9:40 ` [PATCH v5 2/9] mmc: sdhci-st: Add support for de-asserting reset signal and top regs resource Peter Griffin
2015-04-10 9:40 ` [PATCH v5 3/9] mmc: sdhci-st: Add delay management functions for top registers (eMMC) Peter Griffin
2015-04-10 9:40 ` [PATCH v5 4/9] mmc: sdhci-st: Add st_mmcss_cconfig function to configure mmcss glue registers Peter Griffin
2015-04-10 9:40 ` [PATCH v5 5/9] mmc: sdhci-st: Add sdhci_st_set_uhs_signaling function Peter Griffin
2015-04-10 9:40 ` [PATCH v5 6/9] mmc: sdhci-st: Update the quirks for this controller Peter Griffin
2015-04-10 9:40 ` [PATCH v5 7/9] mmc: sdhci-st: Update ST SDHCI binding documentation Peter Griffin
[not found] ` <1428658831-14308-1-git-send-email-peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2015-04-10 9:40 ` [PATCH v5 8/9] ARM: STi: DT: STiH407: Add dt nodes for sdhci and emmc Peter Griffin
2015-04-10 11:06 ` [PATCH v5 0/9] Add sd/emmc support for stih407 family silicon Ulf Hansson
2015-04-30 8:28 ` Maxime Coquelin
[not found] ` <5541E7C5.1000903-qxv4g6HH51o@public.gmane.org>
2015-04-30 8:51 ` Ulf Hansson
2015-04-30 8:55 ` Maxime Coquelin
2015-04-10 9:40 ` [PATCH v5 9/9] ARM: STi: DT: STiH418: Add dt nodes for sdhci and emmc Peter Griffin
2015-04-30 8:56 ` Maxime Coquelin [this message]
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=5541EE35.2070002@st.com \
--to=maxime.coquelin@st.com \
--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=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).