Hi Ulf
On 03/01/2018 10:06 AM, Ulf Hansson wrote:
Hi Ludovic,
On 28 February 2018 at 16:47, Ludovic Barre <ludovic.Barre@xxxxxx> wrote:
From: Ludovic Barre <ludovic.barre@xxxxxx>
This patch serie adds support of stm32 SDMMC controller.
stm32h7 is the first SoC to use stm32 SDMMC controller
(previous SoC had pl180 controller).
I am a not convinced this isn't a new improved variant of the pl180.
According to register layout and the code you submitted in patch2,
there are great similarities to pl180 and the mmci driver.
In fact, ST designers which created stm32-sdmmc hardware block from
scratch are the same which have done the modifications on pl180 variant
(stm32 legacy f4, f7).
So some registers or bits name seem identical (or strongly inspirited)
but the engine and features are different.
You could find the datasheet of STM32H7x3 on:
http://www.st.com/content/ccc/resource/technical/document/reference_manual/group0/c9/a3/76/fa/55/46/45/fa/DM00314099/files/DM00314099.pdf/jcr:content/translations/en.DM00314099.pdf
Chapters: 55 Secure digital input/output MultiMediaCard interface
(SDMMC)
This hardware block has own roadmap and some features are already in the
pipe for next SoC.
For code design: like I also worked on pl180 in the past :-)
my code is inspirited of this driver.
BR
Ludo
In other words, it more sounds like an improved variant of pl180,
hence I am thinking that you should try to extend the mmci driver
instead. Could you please have a look at that, unless you can specify
more exactly why that isn't suitable?
The SDMMC features include the following:
-Full compliance with MultiMediaCard System Specification Version 4.51.
Card support for three different databus modes:
1-bit (default), 4-bit and 8-bit.
-Full compliance with SD memory card specifications version 4.1.
SDR104 speed limited to maximum allowed I/O speed, SPI mode and
UHS-II mode not supported.
-Full compliance with SDIO card specification version 4.0.
change V2:
-dt-bindings: properties name
st,dir-output-high; st,neg-edge; st,use-ckin
-dt-bindings: node example s/sdmmc@/mmc@/g
-removes return of enable/disable_imask functions
-adds dev_warn if clk divisor is too short.
-replaces overprotection of n_elem by fail of dma_map_sg
-removes shortcut host->base
-Use device_property_present
Ludovic Barre (5):
dt-bindings: mmc: document the stm32 sdmmc bindings
mmc: add stm32 sdmmc controller driver
ARM: dts: stm32: add sdmmc support for stm32h743
ARM: dts: stm32: add sdmmc1 support for stm32h743i-eval
ARM: configs: stm32: add mmc and ext2/3/4 support
.../devicetree/bindings/mmc/st,stm32-sdmmc.txt | 35 +
arch/arm/boot/dts/stm32h743-pinctrl.dtsi | 26 +
arch/arm/boot/dts/stm32h743.dtsi | 26 +
arch/arm/boot/dts/stm32h743i-eval.dts | 11 +
arch/arm/configs/stm32_defconfig | 4 +-
drivers/mmc/host/Kconfig | 8 +
drivers/mmc/host/Makefile | 1 +
drivers/mmc/host/stm32-sdmmc.c | 708 +++++++++++++++++++++
drivers/mmc/host/stm32-sdmmc.h | 220 +++++++
9 files changed, 1038 insertions(+), 1 deletion(-)
create mode 100644 Documentation/devicetree/bindings/mmc/st,stm32-sdmmc.txt
create mode 100644 drivers/mmc/host/stm32-sdmmc.c
create mode 100644 drivers/mmc/host/stm32-sdmmc.h
--
2.7.4
Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
