2014-07-11 17:48:23

by Georgi Djakov

[permalink] [raw]
Subject: [PATCH 0/3] mmc: sdhci-msm: Minor fixes and cleanups

This patchset contains fixes in the documentation, cleanups
and enables COMPILE_TEST for better testing coverage.

Georgi Djakov (3):
mmc: sdhci-msm: Fix the binding example
mmc: sdhci-msm: Remove unnecessary header file inclusion
mmc: sdhci-msm: Get COMPILE_TEST support

.../devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++----
drivers/mmc/host/Kconfig | 2 +-
drivers/mmc/host/sdhci-msm.c | 1 -
3 files changed, 5 insertions(+), 6 deletions(-)

--
1.7.9.5


2014-07-11 17:48:36

by Georgi Djakov

[permalink] [raw]
Subject: [PATCH 3/3] mmc: sdhci-msm: Get COMPILE_TEST support

Increase the build testing coverage.

Signed-off-by: Georgi Djakov <[email protected]>
---
drivers/mmc/host/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index a565254..1325cae 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -343,7 +343,7 @@ config MMC_ATMELMCI

config MMC_SDHCI_MSM
tristate "Qualcomm SDHCI Controller Support"
- depends on ARCH_QCOM
+ depends on ARCH_QCOM || (ARM && COMPILE_TEST)
depends on MMC_SDHCI_PLTFM
help
This selects the Secure Digital Host Controller Interface (SDHCI)
--
1.7.9.5

2014-07-11 17:48:29

by Georgi Djakov

[permalink] [raw]
Subject: [PATCH 2/3] mmc: sdhci-msm: Remove unnecessary header file inclusion

The header <linux/regulator/consumer.h> is unused. Remove it.

Signed-off-by: Georgi Djakov <[email protected]>
---
drivers/mmc/host/sdhci-msm.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/mmc/host/sdhci-msm.c b/drivers/mmc/host/sdhci-msm.c
index 40573a5..1a6661e 100644
--- a/drivers/mmc/host/sdhci-msm.c
+++ b/drivers/mmc/host/sdhci-msm.c
@@ -16,7 +16,6 @@

#include <linux/module.h>
#include <linux/of_device.h>
-#include <linux/regulator/consumer.h>
#include <linux/delay.h>
#include <linux/mmc/mmc.h>
#include <linux/slab.h>
--
1.7.9.5

2014-07-11 17:49:08

by Georgi Djakov

[permalink] [raw]
Subject: [PATCH 1/3] mmc: sdhci-msm: Fix the binding example

The DT binding example in the documentation is missing the -supply
suffix for the vmmc and vqmmc regulators. Fix it.

Signed-off-by: Georgi Djakov <[email protected]>
---
.../devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt
index 81b33b5..485483a 100644
--- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt
+++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt
@@ -27,8 +27,8 @@ Example:
bus-width = <8>;
non-removable;

- vmmc = <&pm8941_l20>;
- vqmmc = <&pm8941_s3>;
+ vmmc-supply = <&pm8941_l20>;
+ vqmmc-supply = <&pm8941_s3>;

pinctrl-names = "default";
pinctrl-0 = <&sdc1_clk &sdc1_cmd &sdc1_data>;
@@ -44,8 +44,8 @@ Example:
bus-width = <4>;
cd-gpios = <&msmgpio 62 0x1>;

- vmmc = <&pm8941_l21>;
- vqmmc = <&pm8941_l13>;
+ vmmc-supply = <&pm8941_l21>;
+ vqmmc-supply = <&pm8941_l13>;

pinctrl-names = "default";
pinctrl-0 = <&sdc2_clk &sdc2_cmd &sdc2_data>;
--
1.7.9.5

2014-07-26 09:18:13

by Ulf Hansson

[permalink] [raw]
Subject: Re: [PATCH 0/3] mmc: sdhci-msm: Minor fixes and cleanups

On 11 July 2014 19:48, Georgi Djakov <[email protected]> wrote:
> This patchset contains fixes in the documentation, cleanups
> and enables COMPILE_TEST for better testing coverage.
>
> Georgi Djakov (3):
> mmc: sdhci-msm: Fix the binding example
> mmc: sdhci-msm: Remove unnecessary header file inclusion
> mmc: sdhci-msm: Get COMPILE_TEST support
>
> .../devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++----
> drivers/mmc/host/Kconfig | 2 +-
> drivers/mmc/host/sdhci-msm.c | 1 -
> 3 files changed, 5 insertions(+), 6 deletions(-)
>

Thanks! Applied for next!

Kind regards
Uffe

> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/