Re: [PATCH v5 0/9] Add sd/emmc support for stih407 family silicon

From: Maxime Coquelin
Date: Thu Apr 30 2015 - 04:29:34 EST


Hi Ulf,

On 04/10/2015 01:06 PM, Ulf Hansson wrote:
On 10 April 2015 at 11:40, Peter Griffin <peter.griffin@xxxxxxxxxx> 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(-)

--
1.9.1

Thanks! Applied.
Do you confirm you didn't applied the DT patches?

Thanks,
Maxime


Kind regards
Uffe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/