ARM: tegra: fix Tegra186 SDHCI clock/reset names
authorStephen Warren <swarren@nvidia.com>
Thu, 18 Aug 2016 17:08:44 +0000 (11:08 -0600)
committerTom Warren <twarren@nvidia.com>
Thu, 25 Aug 2016 20:47:49 +0000 (13:47 -0700)
The Tegra SDHCI binding dictates that the reseet name for the Tegra SDHCI
clock be "sdhci" not "sdmmc", and that the clock is accessed by index
rather than by name. Fix the Tegra186 DT and MMC driver to honor this.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
arch/arm/dts/tegra186.dtsi
drivers/mmc/tegra_mmc.c

index 54ef7e3e4adce3e415f4729983f46b0fc1e22ebc..65ff4246c01e511618c48ea53afa06178109de44 100644 (file)
                compatible = "nvidia,tegra186-sdhci";
                reg = <0x0 0x03400000 0x0 0x200>;
                resets = <&bpmp TEGRA186_RESET_SDMMC1>;
-               reset-names = "sdmmc";
+               reset-names = "sdhci";
                clocks = <&bpmp TEGRA186_CLK_SDMMC1>;
-               clock-names = "sdmmc";
                interrupts = <GIC_SPI 62 0x04>;
                status = "disabled";
        };
                compatible = "nvidia,tegra186-sdhci";
                reg = <0x0 0x03460000 0x0 0x200>;
                resets = <&bpmp TEGRA186_RESET_SDMMC4>;
-               reset-names = "sdmmc";
+               reset-names = "sdhci";
                clocks = <&bpmp TEGRA186_CLK_SDMMC4>;
-               clock-names = "sdmmc";
                interrupts = <GIC_SPI 31 0x04>;
                status = "disabled";
        };
index f0a39a63c641524a8b8798499e70fbe79794290b..3d0845e31735acc0bb83fa973908ca7d876c529a 100644 (file)
@@ -648,12 +648,12 @@ static int mmc_get_config(const void *blob, int node, struct mmc_host *host,
                struct udevice dev;
                int ret;
                dev.of_offset = node;
-               ret = reset_get_by_name(&dev, "sdmmc", &host->reset_ctl);
+               ret = reset_get_by_name(&dev, "sdhci", &host->reset_ctl);
                if (ret) {
-                       debug("reset_get_by_index() failed: %d\n", ret);
+                       debug("reset_get_by_name() failed: %d\n", ret);
                        return ret;
                }
-               ret = clk_get_by_name(&dev, "sdmmc", &host->clk);
+               ret = clk_get_by_index(&dev, 0, &host->clk);
                if (ret) {
                        debug("clk_get_by_index() failed: %d\n", ret);
                        return ret;