Re: [PATCH 8/9] DT: stm32f469-disco: add node for SDIO controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andrea

On 11/14/2016 11:36 AM, Andrea Merello wrote:
On Thu, Nov 10, 2016 at 11:54 AM, Alexandre Torgue
<alexandre.torgue@xxxxxx> wrote:
Hi Andrea


On 11/08/2016 02:43 PM, Andrea Merello wrote:

Signed-off-by: Andrea Merello <andrea.merello@xxxxxxxxx>
---
 arch/arm/boot/dts/stm32f429.dtsi      |  2 +-
 arch/arm/boot/dts/stm32f469-disco.dts | 30 ++++++++++++++++++++++++++++++
 2 files changed, 31 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32f429.dtsi
b/arch/arm/boot/dts/stm32f429.dtsi
index d6d0548..23e6a5e 100644
--- a/arch/arm/boot/dts/stm32f429.dtsi
+++ b/arch/arm/boot/dts/stm32f429.dtsi
@@ -185,7 +185,7 @@
                        interrupts = <1>, <2>, <3>, <6>, <7>, <8>, <9>,
<10>, <23>, <40>, <41>, <42>, <62>, <76>;
                };

-               pin-controller {
+               pinctrl:pin-controller {
                        #address-cells = <1>;
                        #size-cells = <1>;
                        compatible = "st,stm32f429-pinctrl";
diff --git a/arch/arm/boot/dts/stm32f469-disco.dts
b/arch/arm/boot/dts/stm32f469-disco.dts
index e911af8..e16dfc3 100644
--- a/arch/arm/boot/dts/stm32f469-disco.dts
+++ b/arch/arm/boot/dts/stm32f469-disco.dts
@@ -64,12 +64,42 @@
        aliases {
                serial0 = &usart3;
        };
+
+       mmc_vcard: mmc_vcard {
+               compatible = "regulator-fixed";
+               regulator-name = "mmc_vcard";
+               regulator-min-microvolt = <3300000>;
+               regulator-max-microvolt = <3300000>;

As I assume you will send a v2 (due to kbuild issue) remove empty line
please.

Sure. I will do. (But unfortunately it will not happen very soon :( )
No pb.

In the meantime comments and testing are appreciated. Also, I forgot
to say that, if one wants to try this, he/she will need the bootloader
to set properly the 48MHz clk for SD (for afboot use my tree -
pull-req pending on Maxime Coquelin tree). For clk driver probably my
patch for making it aware of the 48MHz setting is still needed (I've
not looked into other recent clk patches yet)..

did you see patches sent by Gabriel Fernandez: "[PATCH 0/6] Add STM32F4 missing clocks". This series should add support for 48 MHz. So it is needed to check first if Gabriel's series is enough.

Thanks
Alex



+
+       };
 };

 &clk_hse {
        clock-frequency = <8000000>;
 };

+&pinctrl {
+       sdio-cd {
+               sdio_cd: sdio-cd {
+                       pins {
+                               pinmux = <STM32F429_PG2_FUNC_GPIO>;
+                               bias-pull-up;
+                       };
+               };
+       };
+};
+
+&sdio {
+       status = "okay";
+       vmmc-supply = <&mmc_vcard>;
+       cd-gpios = <&gpiog 2 0>;
+       cd-inverted;
+       pinctrl-names = "default", "opendrain", "cd";
+       pinctrl-0 = <&sdio_pins>, <&sdio_cd>;
+       pinctrl-1 = <&sdio_pins_od>, <&sdio_cd>;
+       bus-width = <4>;
+};
+
 &usart3 {
        status = "okay";
 };


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



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux