Re: [PATCH v8 0/3] mmc: sdhci-msm: Add support for Qualcomm chipsets

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

 



Hi Christopher,

On 02/04/2014 03:19 PM, Christopher Covington wrote:
Hi Georgi,

On 01/30/2014 01:45 PM, Georgi Djakov wrote:
This patchset adds basic support of the Secure Digital Host Controller
Interface compliant controller found in Qualcomm chipsets.

Tested with eMMC and various micro SD cards on APQ8074 Dragonboard.

[...]

  .../devicetree/bindings/mmc/sdhci-msm.txt          |   83 ++
  drivers/mmc/host/Kconfig                           |   13 +
  drivers/mmc/host/Makefile                          |    1 +
  drivers/mmc/host/sdhci-msm.c                       |  938 ++++++++++++++++++++
  4 files changed, 1035 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/mmc/sdhci-msm.txt
  create mode 100644 drivers/mmc/host/sdhci-msm.c

You must have added entries to the Dragonboard device tree source to test
these changes. Why not include them with this patchset? Is there a stand-alone
patch that I've overlooked?


There have been recently many changes to the devicetree files and in order to avoid possible conflicts i have not added DT nodes yet. But I'll consider adding them, and maybe enable it in the defconfig too. Thanks for your suggestion.

To get it working, take a look at the binding example or just use the following:

sdhci1@f9824900 {
        compatible = "qcom,sdhci-msm";
        reg = <0xf9824900 0x11c>, <0xf9824000 0x800>;
        reg-names = "hc_mem", "core_mem";
        interrupts = <0 123 0>, <0 138 0>;
        interrupt-names = "hc_irq", "pwr_irq";
        bus-width = <8>;
        non-removable;
        clocks = <&gcc GCC_SDCC1_APPS_CLK>, <&gcc GCC_SDCC1_AHB_CLK>;
        clock-names = "core", "iface";
};

sdhci2@f98a4900 {
        compatible = "qcom,sdhci-msm";
        reg = <0xf98a4900 0x11c>, <0xf98a4000 0x800>;
        reg-names = "hc_mem", "core_mem";
        interrupts = <0 125 0>, <0 221 0>;
        interrupt-names = "hc_irq", "pwr_irq";
        bus-width = <4>;
        clocks = <&gcc GCC_SDCC2_APPS_CLK>, <&gcc GCC_SDCC2_AHB_CLK>;
        clock-names = "core", "iface";
};


Thanks,
Georgi

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




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux