arm64: dts: hikey960: Clean up MMC properties and move to proper file
authoroscardagrach <ryan@edited.us>
Mon, 18 Jun 2018 18:10:08 +0000 (13:10 -0500)
committerWei Xu <xuwei5@hisilicon.com>
Tue, 17 Jul 2018 11:02:30 +0000 (12:02 +0100)
Certain properties should be moved to the board file to reflect
the specific properties of the board, and not the SoC. Move these
properties to proper location and organize properties in both files.

Signed-off-by: Ryan Grachek <ryan@edited.us>
Signed-off-by: Wei Xu <xuwei5@hisilicon.com>
arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
arch/arm64/boot/dts/hisilicon/hi3660.dtsi

index 1b2cabe..7b8cbaf 100644 (file)
 };
 
 &dwmmc1 {
+       bus-width = <0x4>;
+       cap-sd-highspeed;
+       sd-uhs-sdr12;
+       sd-uhs-sdr25;
+       sd-uhs-sdr50;
+       sd-uhs-sdr104;
+       disable-wp;
+       cd-inverted;
+       cd-gpios = <&gpio25 3 0>;
+       pinctrl-names = "default";
+       pinctrl-0 = <&sd_pmx_func
+                    &sd_clk_cfg_func
+                    &sd_cfg_func>;
        vmmc-supply = <&ldo16>;
        vqmmc-supply = <&ldo9>;
        status = "okay";
 };
 
 &dwmmc2 { /* WIFI */
-       broken-cd;
-       /* WL_EN */
-       vmmc-supply = <&wlan_en>;
+       bus-width = <0x4>;
        non-removable;
+       broken-cd;
        cap-power-off-card;
        keep-power-in-suspend;
-       #address-cells = <0x1>;
-       #size-cells = <0x0>;
+       pinctrl-names = "default";
+       pinctrl-0 = <&sdio_pmx_func
+                    &sdio_clk_cfg_func
+                    &sdio_cfg_func>;
+       /* WL_EN */
+       vmmc-supply = <&wlan_en>;
        status = "ok";
 
        wlcore: wlcore@2 {
index 1cd9263..5c47599 100644 (file)
 
                /* SD */
                dwmmc1: dwmmc1@ff37f000 {
-                       #address-cells = <1>;
-                       #size-cells = <0>;
-                       cd-inverted;
                        compatible = "hisilicon,hi3660-dw-mshc";
-                       bus-width = <0x4>;
-                       disable-wp;
-                       cap-sd-highspeed;
-                       card-detect-delay = <200>;
                        reg = <0x0 0xff37f000 0x0 0x1000>;
+                       #address-cells = <1>;
+                       #size-cells = <0>;
                        interrupts = <GIC_SPI 139 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&crg_ctrl HI3660_CLK_GATE_SD>,
                                <&crg_ctrl HI3660_HCLK_GATE_SD>;
                        clock-frequency = <3200000>;
                        resets = <&crg_rst 0x94 18>;
                        reset-names = "reset";
-                       cd-gpios = <&gpio25 3 0>;
                        hisilicon,peripheral-syscon = <&sctrl>;
-                       pinctrl-names = "default";
-                       pinctrl-0 = <&sd_pmx_func
-                                    &sd_clk_cfg_func
-                                    &sd_cfg_func>;
-                       sd-uhs-sdr12;
-                       sd-uhs-sdr25;
-                       sd-uhs-sdr50;
-                       sd-uhs-sdr104;
+                       card-detect-delay = <200>;
                        status = "disabled";
                };
 
                dwmmc2: dwmmc2@ff3ff000 {
                        compatible = "hisilicon,hi3660-dw-mshc";
                        reg = <0x0 0xff3ff000 0x0 0x1000>;
+                       #address-cells = <0x1>;
+                       #size-cells = <0x0>;
                        interrupts = <GIC_SPI 140 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&crg_ctrl HI3660_CLK_GATE_SDIO0>,
                                 <&crg_ctrl HI3660_HCLK_GATE_SDIO0>;
                        resets = <&crg_rst 0x94 20>;
                        reset-names = "reset";
                        card-detect-delay = <200>;
-                       keep-power-in-suspend;
-                       pinctrl-names = "default";
-                       pinctrl-0 = <&sdio_pmx_func
-                                    &sdio_clk_cfg_func
-                                    &sdio_cfg_func>;
                        status = "disabled";
                };