Groups > linux.kernel > #1659598 > unrolled thread

[PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

Started by	Kishon Vijay Abraham I <kishon@ti.com>
First post	2017-06-07 11:40 +0200
Last post	2017-06-09 12:50 +0200
Articles	12 — 3 participants

Back to article view | Back to linux.kernel

  [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:40 +0200
    [PATCH v2 8/8] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc" Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:40 +0200
      [PATCH v3] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc" Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-09 14:10 +0200
    [PATCH v2 1/8] ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2 Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:50 +0200
    [PATCH v2 5/8] ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2 Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:50 +0200
    [PATCH v2 4/8] ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO lines Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:50 +0200
    [PATCH v2 3/8] ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1 Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-07 11:50 +0200
    Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Ulf Hansson <ulf.hansson@linaro.org> - 2017-06-08 09:30 +0200
      Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Tony Lindgren <tony@atomide.com> - 2017-06-08 10:50 +0200
        Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-09 14:30 +0200
      Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Kishon Vijay Abraham I <kishon@ti.com> - 2017-06-09 10:10 +0200
        Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Ulf Hansson <ulf.hansson@linaro.org> - 2017-06-09 12:50 +0200

#1659598 — [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:40 +0200
Subject	[PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node
Message-ID	<tPFBw-4Pg-5@gated-at.bofh.it>

There are the set of fixes that were sent initially as part
of [1].

These are mostly fixes w.r.t populating regulators in
mmc dt node. It was working before because the regulators
connecting to the IO lines are always on regulators. This will
break once we add UHS support where voltage has to be changed
dynamically.

It also includes a patch to replace all vmmc_aux-supply
with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
This will break old dt compatibility but not sure how better
to handle this.

Performed enumeration testing on dra72-evm-revc, dra72-evm,
dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,

This is a preparation series for adding UHS mode support in
the above mentioned platforms.

Changes from v1:
*) Included a patch to replace deprecated "vmmc_aux" with
   "vqmmc"

[1] -> https://lkml.org/lkml/2017/5/19/196

Kishon Vijay Abraham I (6):
  ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
  ARM: dts: dra72-evm: Add vqmmc supply to mmc1
  ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1
  ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
    lines
  ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
  ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"

Ravikumar Kattekola (1):
  ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2

Sekhar Nori (1):
  ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM

 arch/arm/boot/dts/am572x-idk.dts               |  2 +-
 arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts  |  2 +-
 arch/arm/boot/dts/dra7-evm.dts                 | 12 ++++++++--
 arch/arm/boot/dts/dra7.dtsi                    |  5 +++++
 arch/arm/boot/dts/dra71-evm.dts                | 16 ++++++++++++-
 arch/arm/boot/dts/dra72-evm-common.dtsi        |  2 --
 arch/arm/boot/dts/dra72-evm-revc.dts           | 18 +++++++++++++++
 arch/arm/boot/dts/dra72-evm-tps65917.dtsi      |  2 +-
 arch/arm/boot/dts/dra72-evm.dts                | 18 +++++++++++++++
 arch/arm/boot/dts/omap3-beagle-xm.dts          |  2 +-
 arch/arm/boot/dts/omap3-beagle.dts             |  2 +-
 arch/arm/boot/dts/omap3-cm-t3517.dts           |  2 +-
 arch/arm/boot/dts/omap3-cm-t3730.dts           |  2 +-
 arch/arm/boot/dts/omap3-devkit8000-common.dtsi |  2 +-
 arch/arm/boot/dts/omap3-evm-common.dtsi        |  2 +-
 arch/arm/boot/dts/omap3-n900.dts               |  2 +-
 arch/arm/boot/dts/omap3-overo-base.dtsi        |  3 +--
 arch/arm/boot/dts/omap3-tao3530.dtsi           |  2 +-
 arch/arm/boot/dts/omap3-zoom3.dts              |  2 +-
 arch/arm/boot/dts/omap3430-sdp.dts             |  2 +-
 arch/arm/mach-omap2/pdata-quirks.c             | 31 ++++++++++++++++++++++++++
 include/linux/platform_data/hsmmc-omap.h       |  3 +++
 22 files changed, 114 insertions(+), 20 deletions(-)

-- 
2.11.0

[toc] | [next] | [standalone]

#1659600 — [PATCH v2 8/8] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:40 +0200
Subject	[PATCH v2 8/8] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
Message-ID	<tPFBx-4Pg-33@gated-at.bofh.it>
In reply to	#1659598

Replace deprecated "vmmc_aux" with the generic "vqmmc" binding for
MMC IO supply.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/am572x-idk.dts               | 2 +-
 arch/arm/boot/dts/dra7-evm.dts                 | 2 +-
 arch/arm/boot/dts/dra71-evm.dts                | 2 +-
 arch/arm/boot/dts/dra72-evm-tps65917.dtsi      | 2 +-
 arch/arm/boot/dts/omap3-beagle-xm.dts          | 2 +-
 arch/arm/boot/dts/omap3-beagle.dts             | 2 +-
 arch/arm/boot/dts/omap3-cm-t3517.dts           | 2 +-
 arch/arm/boot/dts/omap3-cm-t3730.dts           | 2 +-
 arch/arm/boot/dts/omap3-devkit8000-common.dtsi | 2 +-
 arch/arm/boot/dts/omap3-evm-common.dtsi        | 2 +-
 arch/arm/boot/dts/omap3-n900.dts               | 2 +-
 arch/arm/boot/dts/omap3-overo-base.dtsi        | 3 +--
 arch/arm/boot/dts/omap3-tao3530.dtsi           | 2 +-
 arch/arm/boot/dts/omap3-zoom3.dts              | 2 +-
 arch/arm/boot/dts/omap3430-sdp.dts             | 2 +-
 15 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/arch/arm/boot/dts/am572x-idk.dts b/arch/arm/boot/dts/am572x-idk.dts
index 8350b4b34b08..8d538f5d9a5f 100644
--- a/arch/arm/boot/dts/am572x-idk.dts
+++ b/arch/arm/boot/dts/am572x-idk.dts
@@ -79,7 +79,7 @@
 &mmc1 {
 	status = "okay";
 	vmmc-supply = <&v3_3d>;
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 	bus-width = <4>;
 	cd-gpios = <&gpio6 27 0>; /* gpio 219 */
 };
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index e5d7e7f5f645..e4c4037e6314 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -446,7 +446,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins_default>;
 	vmmc-supply = <&evm_3v3_sd>;
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 	bus-width = <4>;
 	/*
 	 * SDCD signal is not being used here - using the fact that GPIO mode
diff --git a/arch/arm/boot/dts/dra71-evm.dts b/arch/arm/boot/dts/dra71-evm.dts
index c7a56b6263a6..02c11e658810 100644
--- a/arch/arm/boot/dts/dra71-evm.dts
+++ b/arch/arm/boot/dts/dra71-evm.dts
@@ -172,7 +172,7 @@
 };
 
 &mmc1 {
-	vmmc_aux-supply = <&vpo_sd_1v8_3v3>;
+	vqmmc-supply = <&vpo_sd_1v8_3v3>;
 };
 
 &mmc2 {
diff --git a/arch/arm/boot/dts/dra72-evm-tps65917.dtsi b/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
index e6df676886c0..57bfe5caf5e4 100644
--- a/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
+++ b/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
@@ -146,5 +146,5 @@
 };
 
 &mmc1 {
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 };
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index 673cee2234b2..683b96a8f73e 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -299,7 +299,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index 4be85ce59dd1..4d2eaf843fa9 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -283,7 +283,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-cm-t3517.dts b/arch/arm/boot/dts/omap3-cm-t3517.dts
index 53ae04f9104d..3d293b345e99 100644
--- a/arch/arm/boot/dts/omap3-cm-t3517.dts
+++ b/arch/arm/boot/dts/omap3-cm-t3517.dts
@@ -129,7 +129,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&wl12xx_vmmc2>;
-	vmmc_aux-supply = <&wl12xx_vaux2>;
+	vqmmc-supply = <&wl12xx_vaux2>;
 	non-removable;
 	bus-width = <4>;
 	cap-power-off-card;
diff --git a/arch/arm/boot/dts/omap3-cm-t3730.dts b/arch/arm/boot/dts/omap3-cm-t3730.dts
index 2294f5b0aa10..bdf4b7fdda39 100644
--- a/arch/arm/boot/dts/omap3-cm-t3730.dts
+++ b/arch/arm/boot/dts/omap3-cm-t3730.dts
@@ -69,7 +69,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&wl12xx_vmmc2>;
-	vmmc_aux-supply = <&wl12xx_vaux2>;
+	vqmmc-supply = <&wl12xx_vaux2>;
 	non-removable;
 	bus-width = <4>;
 	cap-power-off-card;
diff --git a/arch/arm/boot/dts/omap3-devkit8000-common.dtsi b/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
index f330c69cc683..4036cef4f9fc 100644
--- a/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
+++ b/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
@@ -149,7 +149,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-evm-common.dtsi b/arch/arm/boot/dts/omap3-evm-common.dtsi
index 090475083c2f..b831f2ed1abc 100644
--- a/arch/arm/boot/dts/omap3-evm-common.dtsi
+++ b/arch/arm/boot/dts/omap3-evm-common.dtsi
@@ -98,7 +98,7 @@
 &mmc1 {
 	interrupts-extended = <&intc 83 &omap3_pmx_core 0x11a>;
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index 49f37084e435..92c1f4ac4040 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -741,7 +741,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&vaux3>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 	non-removable;
 	no-sdio;
diff --git a/arch/arm/boot/dts/omap3-overo-base.dtsi b/arch/arm/boot/dts/omap3-overo-base.dtsi
index 401fae838fe9..c11c86f9e09d 100644
--- a/arch/arm/boot/dts/omap3-overo-base.dtsi
+++ b/arch/arm/boot/dts/omap3-overo-base.dtsi
@@ -191,8 +191,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&w3cbw003c_npoweron>;
-	vqmmc-supply = <&w3cbw003c_bt_nreset>;
-	vmmc_aux-supply = <&w3cbw003c_wifi_nreset>;
+	vqmmc-supply = <&w3cbw003c_wifi_nreset>;
 	bus-width = <4>;
 	cap-sdio-irq;
 	non-removable;
diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi
index dc80886b5329..ac6ff9b4db94 100644
--- a/arch/arm/boot/dts/omap3-tao3530.dtsi
+++ b/arch/arm/boot/dts/omap3-tao3530.dtsi
@@ -224,7 +224,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	cd-gpios = <&twl_gpio 0 GPIO_ACTIVE_HIGH>;
 	bus-width = <8>;
 };
diff --git a/arch/arm/boot/dts/omap3-zoom3.dts b/arch/arm/boot/dts/omap3-zoom3.dts
index 45e2ce0803de..96d0301a336a 100644
--- a/arch/arm/boot/dts/omap3-zoom3.dts
+++ b/arch/arm/boot/dts/omap3-zoom3.dts
@@ -174,7 +174,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <4>;
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index abd6921143be..908951eb5943 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -33,7 +33,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	/*
 	 * S6-3 must be in ON position for 8 bit mode to function
 	 * Else, use 4 bit mode
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1662360 — [PATCH v3] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-09 14:10 +0200
Subject	[PATCH v3] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
Message-ID	<tQqTL-1lV-1@gated-at.bofh.it>
In reply to	#1659600

Replace deprecated "vmmc_aux" with the generic "vqmmc" binding for
MMC IO supply.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
Changes from v2:
Rebased on [1].
[1] -> https://www.spinics.net/lists/arm-kernel/msg586977.html
 arch/arm/boot/dts/am572x-idk.dts               | 2 +-
 arch/arm/boot/dts/dra7-evm.dts                 | 2 +-
 arch/arm/boot/dts/dra71-evm.dts                | 2 +-
 arch/arm/boot/dts/dra72-evm-tps65917.dtsi      | 2 +-
 arch/arm/boot/dts/omap3-beagle-xm.dts          | 2 +-
 arch/arm/boot/dts/omap3-beagle.dts             | 2 +-
 arch/arm/boot/dts/omap3-cm-t3517.dts           | 2 +-
 arch/arm/boot/dts/omap3-cm-t3730.dts           | 2 +-
 arch/arm/boot/dts/omap3-devkit8000-common.dtsi | 2 +-
 arch/arm/boot/dts/omap3-evm-common.dtsi        | 2 +-
 arch/arm/boot/dts/omap3-n900.dts               | 2 +-
 arch/arm/boot/dts/omap3-overo-base.dtsi        | 2 +-
 arch/arm/boot/dts/omap3-tao3530.dtsi           | 2 +-
 arch/arm/boot/dts/omap3-zoom3.dts              | 2 +-
 arch/arm/boot/dts/omap3430-sdp.dts             | 2 +-
 15 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/arch/arm/boot/dts/am572x-idk.dts b/arch/arm/boot/dts/am572x-idk.dts
index 8350b4b34b08..8d538f5d9a5f 100644
--- a/arch/arm/boot/dts/am572x-idk.dts
+++ b/arch/arm/boot/dts/am572x-idk.dts
@@ -79,7 +79,7 @@
 &mmc1 {
 	status = "okay";
 	vmmc-supply = <&v3_3d>;
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 	bus-width = <4>;
 	cd-gpios = <&gpio6 27 0>; /* gpio 219 */
 };
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index e5d7e7f5f645..e4c4037e6314 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -446,7 +446,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins_default>;
 	vmmc-supply = <&evm_3v3_sd>;
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 	bus-width = <4>;
 	/*
 	 * SDCD signal is not being used here - using the fact that GPIO mode
diff --git a/arch/arm/boot/dts/dra71-evm.dts b/arch/arm/boot/dts/dra71-evm.dts
index c7a56b6263a6..02c11e658810 100644
--- a/arch/arm/boot/dts/dra71-evm.dts
+++ b/arch/arm/boot/dts/dra71-evm.dts
@@ -172,7 +172,7 @@
 };
 
 &mmc1 {
-	vmmc_aux-supply = <&vpo_sd_1v8_3v3>;
+	vqmmc-supply = <&vpo_sd_1v8_3v3>;
 };
 
 &mmc2 {
diff --git a/arch/arm/boot/dts/dra72-evm-tps65917.dtsi b/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
index e6df676886c0..57bfe5caf5e4 100644
--- a/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
+++ b/arch/arm/boot/dts/dra72-evm-tps65917.dtsi
@@ -146,5 +146,5 @@
 };
 
 &mmc1 {
-	vmmc_aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 };
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index 673cee2234b2..683b96a8f73e 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -299,7 +299,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index 4be85ce59dd1..4d2eaf843fa9 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -283,7 +283,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-cm-t3517.dts b/arch/arm/boot/dts/omap3-cm-t3517.dts
index 53ae04f9104d..3d293b345e99 100644
--- a/arch/arm/boot/dts/omap3-cm-t3517.dts
+++ b/arch/arm/boot/dts/omap3-cm-t3517.dts
@@ -129,7 +129,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&wl12xx_vmmc2>;
-	vmmc_aux-supply = <&wl12xx_vaux2>;
+	vqmmc-supply = <&wl12xx_vaux2>;
 	non-removable;
 	bus-width = <4>;
 	cap-power-off-card;
diff --git a/arch/arm/boot/dts/omap3-cm-t3730.dts b/arch/arm/boot/dts/omap3-cm-t3730.dts
index 2294f5b0aa10..bdf4b7fdda39 100644
--- a/arch/arm/boot/dts/omap3-cm-t3730.dts
+++ b/arch/arm/boot/dts/omap3-cm-t3730.dts
@@ -69,7 +69,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&wl12xx_vmmc2>;
-	vmmc_aux-supply = <&wl12xx_vaux2>;
+	vqmmc-supply = <&wl12xx_vaux2>;
 	non-removable;
 	bus-width = <4>;
 	cap-power-off-card;
diff --git a/arch/arm/boot/dts/omap3-devkit8000-common.dtsi b/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
index f330c69cc683..4036cef4f9fc 100644
--- a/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
+++ b/arch/arm/boot/dts/omap3-devkit8000-common.dtsi
@@ -149,7 +149,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-evm-common.dtsi b/arch/arm/boot/dts/omap3-evm-common.dtsi
index 090475083c2f..b831f2ed1abc 100644
--- a/arch/arm/boot/dts/omap3-evm-common.dtsi
+++ b/arch/arm/boot/dts/omap3-evm-common.dtsi
@@ -98,7 +98,7 @@
 &mmc1 {
 	interrupts-extended = <&intc 83 &omap3_pmx_core 0x11a>;
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 };
 
diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index 49f37084e435..92c1f4ac4040 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -741,7 +741,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&vaux3>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <8>;
 	non-removable;
 	no-sdio;
diff --git a/arch/arm/boot/dts/omap3-overo-base.dtsi b/arch/arm/boot/dts/omap3-overo-base.dtsi
index cd220342a805..f25e158e7163 100644
--- a/arch/arm/boot/dts/omap3-overo-base.dtsi
+++ b/arch/arm/boot/dts/omap3-overo-base.dtsi
@@ -181,7 +181,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&w3cbw003c_npoweron>;
-	vmmc_aux-supply = <&w3cbw003c_wifi_nreset>;
+	vqmmc-supply = <&w3cbw003c_wifi_nreset>;
 	bus-width = <4>;
 	cap-sdio-irq;
 	non-removable;
diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi
index dc80886b5329..ac6ff9b4db94 100644
--- a/arch/arm/boot/dts/omap3-tao3530.dtsi
+++ b/arch/arm/boot/dts/omap3-tao3530.dtsi
@@ -224,7 +224,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	cd-gpios = <&twl_gpio 0 GPIO_ACTIVE_HIGH>;
 	bus-width = <8>;
 };
diff --git a/arch/arm/boot/dts/omap3-zoom3.dts b/arch/arm/boot/dts/omap3-zoom3.dts
index 45e2ce0803de..96d0301a336a 100644
--- a/arch/arm/boot/dts/omap3-zoom3.dts
+++ b/arch/arm/boot/dts/omap3-zoom3.dts
@@ -174,7 +174,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	bus-width = <4>;
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index abd6921143be..908951eb5943 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -33,7 +33,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vmmc1>;
-	vmmc_aux-supply = <&vsim>;
+	vqmmc-supply = <&vsim>;
 	/*
 	 * S6-3 must be in ON position for 8 bit mode to function
 	 * Else, use 4 bit mode
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1659609 — [PATCH v2 1/8] ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:50 +0200
Subject	[PATCH v2 1/8] ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
Message-ID	<tPFLb-4SX-5@gated-at.bofh.it>
In reply to	#1659598

On dra72/dra71 evms, mmc2 vdd/ios are connected to a common 1.8V supply
not 3.3V. Also the regulator that supplies 1.8V is different on dra71-evm
so move the supply property from common dtsi to evm specific dts files.

Fixes: a4240d3af677 ("ARM: dts: Add support for dra72-evm rev C (SR2.0)")
Signed-off-by: Ravikumar Kattekola <rk@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/dra71-evm.dts         | 14 ++++++++++++++
 arch/arm/boot/dts/dra72-evm-common.dtsi |  2 --
 arch/arm/boot/dts/dra72-evm-revc.dts    | 14 ++++++++++++++
 arch/arm/boot/dts/dra72-evm.dts         | 14 ++++++++++++++
 4 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/dra71-evm.dts b/arch/arm/boot/dts/dra71-evm.dts
index 4d57a55473af..c7a56b6263a6 100644
--- a/arch/arm/boot/dts/dra71-evm.dts
+++ b/arch/arm/boot/dts/dra71-evm.dts
@@ -32,6 +32,16 @@
 			  3000000 0x1>;
 	};
 
+	evm_1v8_sw: fixedregulator-evm_1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "evm_1v8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+		vin-supply = <&lp8732_buck0_reg>;
+		regulator-always-on;
+		regulator-boot-on;
+	};
+
 	poweroff: gpio-poweroff {
 		compatible = "gpio-poweroff";
 		gpios = <&gpio7 30 GPIO_ACTIVE_HIGH>;
@@ -165,6 +175,10 @@
 	vmmc_aux-supply = <&vpo_sd_1v8_3v3>;
 };
 
+&mmc2 {
+	vmmc-supply = <&evm_1v8_sw>;
+};
+
 &mac {
 	mode-gpios = <&pcf_gpio_21 4 GPIO_ACTIVE_LOW>,
 		     <&pcf_hdmi 9 GPIO_ACTIVE_LOW>,	/* P11 */
diff --git a/arch/arm/boot/dts/dra72-evm-common.dtsi b/arch/arm/boot/dts/dra72-evm-common.dtsi
index ad24544adf0f..d97370217eff 100644
--- a/arch/arm/boot/dts/dra72-evm-common.dtsi
+++ b/arch/arm/boot/dts/dra72-evm-common.dtsi
@@ -418,8 +418,6 @@
 	status = "okay";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins_default>;
-
-	vmmc-supply = <&evm_3v3_sw>;
 	bus-width = <8>;
 	ti,non-removable;
 	max-frequency = <192000000>;
diff --git a/arch/arm/boot/dts/dra72-evm-revc.dts b/arch/arm/boot/dts/dra72-evm-revc.dts
index 3ecac56bf504..47dcd527ec58 100644
--- a/arch/arm/boot/dts/dra72-evm-revc.dts
+++ b/arch/arm/boot/dts/dra72-evm-revc.dts
@@ -15,6 +15,16 @@
 		device_type = "memory";
 		reg = <0x0 0x80000000 0x0 0x80000000>; /* 2GB */
 	};
+
+	evm_1v8_sw: fixedregulator-evm_1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "evm_1v8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+		vin-supply = <&smps4_reg>;
+		regulator-always-on;
+		regulator-boot-on;
+	};
 };
 
 &i2c1 {
@@ -82,3 +92,7 @@
 		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
 	};
 };
+
+&mmc2 {
+	vmmc-supply = <&evm_1v8_sw>;
+};
diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
index cd9c4ff12654..adbcc570eba9 100644
--- a/arch/arm/boot/dts/dra72-evm.dts
+++ b/arch/arm/boot/dts/dra72-evm.dts
@@ -13,6 +13,16 @@
 		device_type = "memory";
 		reg = <0x0 0x80000000 0x0 0x40000000>; /* 1024 MB */
 	};
+
+	evm_1v8_sw: fixedregulator-evm_1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "evm_1v8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+		vin-supply = <&smps4_reg>;
+		regulator-always-on;
+		regulator-boot-on;
+	};
 };
 
 &i2c1 {
@@ -43,3 +53,7 @@
 	phy_id = <&davinci_mdio>, <3>;
 	phy-mode = "rgmii";
 };
+
+&mmc2 {
+	vmmc-supply = <&evm_1v8_sw>;
+};
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1659613 — [PATCH v2 5/8] ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:50 +0200
Subject	[PATCH v2 5/8] ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
Message-ID	<tPFLb-4SX-15@gated-at.bofh.it>
In reply to	#1659598

From: Ravikumar Kattekola <rk@ti.com>

On DRA75x EVM, MMC2 vdd/ios are connected to a common supply fixed at 1.8V
not 3.3V

Fixes: 6cf02dbb4b71 ("ARM: dts: dra7-evm: Add mmc2 node for eMMC support")
Signed-off-by: Ravikumar Kattekola <rk@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/dra7-evm.dts | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index 31a9e061ddd0..e5d7e7f5f645 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -25,6 +25,14 @@
 		stdout-path = &uart1;
 	};
 
+	evm_1v8_sw: fixedregulator-evm_1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "evm_1v8";
+		vin-supply = <&smps9_reg>;
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
+
 	evm_3v3_sd: fixedregulator-sd {
 		compatible = "regulator-fixed";
 		regulator-name = "evm_3v3_sd";
@@ -451,7 +459,7 @@
 	status = "okay";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins_default>;
-	vmmc-supply = <&evm_3v3_sw>;
+	vmmc-supply = <&evm_1v8_sw>;
 	bus-width = <8>;
 };
 
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1659621 — [PATCH v2 4/8] ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO lines

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:50 +0200
Subject	[PATCH v2 4/8] ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO lines
Message-ID	<tPFLc-4SX-37@gated-at.bofh.it>
In reply to	#1659598

The dt binding documentation of omap-hsmmc recommends using
"vmmc_aux" for IO supply lines. However
commit 0af28cc92690d8c ("ARM: dts: am57xx-beagle-x15: Add support
for rev B1") added it as "vmmc-aux". Now that anyways it
needs to be fixed, use the generic "vqmmc" binding instead of
the deprecated "vmmc_aux".

Fixes: commit 0af28cc92690d8c ("ARM: dts: am57xx-beagle-x15: Add
support for rev B1")

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts b/arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts
index 39a92aff0a0d..578398c781b1 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts
+++ b/arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts
@@ -20,7 +20,7 @@
 
 &mmc1 {
 	vmmc-supply = <&vdd_3v3>;
-	vmmc-aux-supply = <&ldo1_reg>;
+	vqmmc-supply = <&ldo1_reg>;
 };
 
 /* errata i880 "Ethernet RGMII2 Limited to 10/100 Mbps" */
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1659628 — [PATCH v2 3/8] ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-07 11:50 +0200
Subject	[PATCH v2 3/8] ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1
Message-ID	<tPFLd-4SX-53@gated-at.bofh.it>
In reply to	#1659598

Add vqmmc-supply property to mmc1 dt node and populate
it with ldo1_reg to reflect ldo1_out is connected to mmc1 IO lines.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/dra72-evm-revc.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/dra72-evm-revc.dts b/arch/arm/boot/dts/dra72-evm-revc.dts
index 47dcd527ec58..5532309971ae 100644
--- a/arch/arm/boot/dts/dra72-evm-revc.dts
+++ b/arch/arm/boot/dts/dra72-evm-revc.dts
@@ -93,6 +93,10 @@
 	};
 };
 
+&mmc1 {
+	vqmmc-supply = <&ldo1_reg>;
+};
+
 &mmc2 {
 	vmmc-supply = <&evm_1v8_sw>;
 };
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1660831

From	Ulf Hansson <ulf.hansson@linaro.org>
Date	2017-06-08 09:30 +0200
Message-ID	<tQ03g-1dY-13@gated-at.bofh.it>
In reply to	#1659598

On 7 June 2017 at 11:37, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> There are the set of fixes that were sent initially as part
> of [1].
>
> These are mostly fixes w.r.t populating regulators in
> mmc dt node. It was working before because the regulators
> connecting to the IO lines are always on regulators. This will
> break once we add UHS support where voltage has to be changed
> dynamically.
>
> It also includes a patch to replace all vmmc_aux-supply
> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
> This will break old dt compatibility but not sure how better
> to handle this.
>
> Performed enumeration testing on dra72-evm-revc, dra72-evm,
> dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
> pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,
>
> This is a preparation series for adding UHS mode support in
> the above mentioned platforms.
>
> Changes from v1:
> *) Included a patch to replace deprecated "vmmc_aux" with
>    "vqmmc"
>
> [1] -> https://lkml.org/lkml/2017/5/19/196
>
> Kishon Vijay Abraham I (6):
>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>   ARM: dts: dra72-evm: Add vqmmc supply to mmc1
>   ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1
>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>     lines
>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>   ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
>
> Ravikumar Kattekola (1):
>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>
> Sekhar Nori (1):
>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>
>  arch/arm/boot/dts/am572x-idk.dts               |  2 +-
>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts  |  2 +-
>  arch/arm/boot/dts/dra7-evm.dts                 | 12 ++++++++--
>  arch/arm/boot/dts/dra7.dtsi                    |  5 +++++
>  arch/arm/boot/dts/dra71-evm.dts                | 16 ++++++++++++-
>  arch/arm/boot/dts/dra72-evm-common.dtsi        |  2 --
>  arch/arm/boot/dts/dra72-evm-revc.dts           | 18 +++++++++++++++
>  arch/arm/boot/dts/dra72-evm-tps65917.dtsi      |  2 +-
>  arch/arm/boot/dts/dra72-evm.dts                | 18 +++++++++++++++
>  arch/arm/boot/dts/omap3-beagle-xm.dts          |  2 +-
>  arch/arm/boot/dts/omap3-beagle.dts             |  2 +-
>  arch/arm/boot/dts/omap3-cm-t3517.dts           |  2 +-
>  arch/arm/boot/dts/omap3-cm-t3730.dts           |  2 +-
>  arch/arm/boot/dts/omap3-devkit8000-common.dtsi |  2 +-
>  arch/arm/boot/dts/omap3-evm-common.dtsi        |  2 +-
>  arch/arm/boot/dts/omap3-n900.dts               |  2 +-
>  arch/arm/boot/dts/omap3-overo-base.dtsi        |  3 +--
>  arch/arm/boot/dts/omap3-tao3530.dtsi           |  2 +-
>  arch/arm/boot/dts/omap3-zoom3.dts              |  2 +-
>  arch/arm/boot/dts/omap3430-sdp.dts             |  2 +-
>  arch/arm/mach-omap2/pdata-quirks.c             | 31 ++++++++++++++++++++++++++
>  include/linux/platform_data/hsmmc-omap.h       |  3 +++
>  22 files changed, 114 insertions(+), 20 deletions(-)
>
> --
> 2.11.0
>

If it helps, I can host a branch with the updates on the omap_hsmmc
driver, such it can be pulled in from arm soc?

I guess you need that as, the DT changes relies on the new vqmmc
binding. Or you thing it doesn't matter, because for the current
changes that regulator is always an always on regulator?

Br
Uffe

[toc] | [prev] | [next] | [standalone]

#1660949

From	Tony Lindgren <tony@atomide.com>
Date	2017-06-08 10:50 +0200
Message-ID	<tQ1iG-1UD-7@gated-at.bofh.it>
In reply to	#1660831

* Ulf Hansson <ulf.hansson@linaro.org> [170608 00:24]:
> If it helps, I can host a branch with the updates on the omap_hsmmc
> driver, such it can be pulled in from arm soc?

Great that works for me. Let's just make sure git bisect
keeps working.

> I guess you need that as, the DT changes relies on the new vqmmc
> binding. Or you thing it doesn't matter, because for the current
> changes that regulator is always an always on regulator?

Or maybe we can just add the new regulator, then do a
follow-up series to remove the old ones?

Regards,

Tony

[toc] | [prev] | [next] | [standalone]

#1662375

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-09 14:30 +0200
Message-ID	<tQrd7-1sb-11@gated-at.bofh.it>
In reply to	#1660949

Hi Tony,

On Thursday 08 June 2017 02:16 PM, Tony Lindgren wrote:
> * Ulf Hansson <ulf.hansson@linaro.org> [170608 00:24]:
>> If it helps, I can host a branch with the updates on the omap_hsmmc
>> driver, such it can be pulled in from arm soc?
> 
> Great that works for me. Let's just make sure git bisect
> keeps working.
> 
>> I guess you need that as, the DT changes relies on the new vqmmc
>> binding. Or you thing it doesn't matter, because for the current
>> changes that regulator is always an always on regulator?
> 
> Or maybe we can just add the new regulator, then do a
> follow-up series to remove the old ones?

Since the driver has a fallback mechanism wherein if it's not able to find
vqmmc, it'll fallback to vmmc_aux, I don't think that's required.

Thanks
Kishon

[toc] | [prev] | [next] | [standalone]

#1662075

From	Kishon Vijay Abraham I <kishon@ti.com>
Date	2017-06-09 10:10 +0200
Message-ID	<tQn9w-7uN-25@gated-at.bofh.it>
In reply to	#1660831

Hi,

On Thursday 08 June 2017 12:50 PM, Ulf Hansson wrote:
> On 7 June 2017 at 11:37, Kishon Vijay Abraham I <kishon@ti.com> wrote:
>> There are the set of fixes that were sent initially as part
>> of [1].
>>
>> These are mostly fixes w.r.t populating regulators in
>> mmc dt node. It was working before because the regulators
>> connecting to the IO lines are always on regulators. This will
>> break once we add UHS support where voltage has to be changed
>> dynamically.
>>
>> It also includes a patch to replace all vmmc_aux-supply
>> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
>> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
>> This will break old dt compatibility but not sure how better
>> to handle this.
>>
>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>> dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
>> pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,
>>
>> This is a preparation series for adding UHS mode support in
>> the above mentioned platforms.
>>
>> Changes from v1:
>> *) Included a patch to replace deprecated "vmmc_aux" with
>>    "vqmmc"
>>
>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>
>> Kishon Vijay Abraham I (6):
>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>   ARM: dts: dra72-evm: Add vqmmc supply to mmc1
>>   ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1
>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>>     lines
>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>   ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
>>
>> Ravikumar Kattekola (1):
>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>
>> Sekhar Nori (1):
>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>
>>  arch/arm/boot/dts/am572x-idk.dts               |  2 +-
>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts  |  2 +-
>>  arch/arm/boot/dts/dra7-evm.dts                 | 12 ++++++++--
>>  arch/arm/boot/dts/dra7.dtsi                    |  5 +++++
>>  arch/arm/boot/dts/dra71-evm.dts                | 16 ++++++++++++-
>>  arch/arm/boot/dts/dra72-evm-common.dtsi        |  2 --
>>  arch/arm/boot/dts/dra72-evm-revc.dts           | 18 +++++++++++++++
>>  arch/arm/boot/dts/dra72-evm-tps65917.dtsi      |  2 +-
>>  arch/arm/boot/dts/dra72-evm.dts                | 18 +++++++++++++++
>>  arch/arm/boot/dts/omap3-beagle-xm.dts          |  2 +-
>>  arch/arm/boot/dts/omap3-beagle.dts             |  2 +-
>>  arch/arm/boot/dts/omap3-cm-t3517.dts           |  2 +-
>>  arch/arm/boot/dts/omap3-cm-t3730.dts           |  2 +-
>>  arch/arm/boot/dts/omap3-devkit8000-common.dtsi |  2 +-
>>  arch/arm/boot/dts/omap3-evm-common.dtsi        |  2 +-
>>  arch/arm/boot/dts/omap3-n900.dts               |  2 +-
>>  arch/arm/boot/dts/omap3-overo-base.dtsi        |  3 +--
>>  arch/arm/boot/dts/omap3-tao3530.dtsi           |  2 +-
>>  arch/arm/boot/dts/omap3-zoom3.dts              |  2 +-
>>  arch/arm/boot/dts/omap3430-sdp.dts             |  2 +-
>>  arch/arm/mach-omap2/pdata-quirks.c             | 31 ++++++++++++++++++++++++++
>>  include/linux/platform_data/hsmmc-omap.h       |  3 +++
>>  22 files changed, 114 insertions(+), 20 deletions(-)
>>
>> --
>> 2.11.0
>>
> 
> If it helps, I can host a branch with the updates on the omap_hsmmc
> driver, such it can be pulled in from arm soc?
> 
> I guess you need that as, the DT changes relies on the new vqmmc
> binding. Or you thing it doesn't matter, because for the current
> changes that regulator is always an always on regulator?

Since there is fallback mechanism in omap_hsmmc driver (falls back to
vmmc_aux), dt changes shouldn't rely on the omap_hsmmc driver.

However I'll send a fix for omap3-overo-base which used vqmmc incorrectly (that
should be merged before omap_hsmmc driver changes) so that git bisect does not
break.

Thanks
Kishon

[toc] | [prev] | [next] | [standalone]

#1662281

From	Ulf Hansson <ulf.hansson@linaro.org>
Date	2017-06-09 12:50 +0200
Message-ID	<tQpEm-qv-25@gated-at.bofh.it>
In reply to	#1662075

On 9 June 2017 at 10:05, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> Hi,
>
> On Thursday 08 June 2017 12:50 PM, Ulf Hansson wrote:
>> On 7 June 2017 at 11:37, Kishon Vijay Abraham I <kishon@ti.com> wrote:
>>> There are the set of fixes that were sent initially as part
>>> of [1].
>>>
>>> These are mostly fixes w.r.t populating regulators in
>>> mmc dt node. It was working before because the regulators
>>> connecting to the IO lines are always on regulators. This will
>>> break once we add UHS support where voltage has to be changed
>>> dynamically.
>>>
>>> It also includes a patch to replace all vmmc_aux-supply
>>> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
>>> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
>>> This will break old dt compatibility but not sure how better
>>> to handle this.
>>>
>>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>>> dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
>>> pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,
>>>
>>> This is a preparation series for adding UHS mode support in
>>> the above mentioned platforms.
>>>
>>> Changes from v1:
>>> *) Included a patch to replace deprecated "vmmc_aux" with
>>>    "vqmmc"
>>>
>>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>>
>>> Kishon Vijay Abraham I (6):
>>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>>   ARM: dts: dra72-evm: Add vqmmc supply to mmc1
>>>   ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1
>>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>>>     lines
>>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>>   ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
>>>
>>> Ravikumar Kattekola (1):
>>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>>
>>> Sekhar Nori (1):
>>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>>
>>>  arch/arm/boot/dts/am572x-idk.dts               |  2 +-
>>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts  |  2 +-
>>>  arch/arm/boot/dts/dra7-evm.dts                 | 12 ++++++++--
>>>  arch/arm/boot/dts/dra7.dtsi                    |  5 +++++
>>>  arch/arm/boot/dts/dra71-evm.dts                | 16 ++++++++++++-
>>>  arch/arm/boot/dts/dra72-evm-common.dtsi        |  2 --
>>>  arch/arm/boot/dts/dra72-evm-revc.dts           | 18 +++++++++++++++
>>>  arch/arm/boot/dts/dra72-evm-tps65917.dtsi      |  2 +-
>>>  arch/arm/boot/dts/dra72-evm.dts                | 18 +++++++++++++++
>>>  arch/arm/boot/dts/omap3-beagle-xm.dts          |  2 +-
>>>  arch/arm/boot/dts/omap3-beagle.dts             |  2 +-
>>>  arch/arm/boot/dts/omap3-cm-t3517.dts           |  2 +-
>>>  arch/arm/boot/dts/omap3-cm-t3730.dts           |  2 +-
>>>  arch/arm/boot/dts/omap3-devkit8000-common.dtsi |  2 +-
>>>  arch/arm/boot/dts/omap3-evm-common.dtsi        |  2 +-
>>>  arch/arm/boot/dts/omap3-n900.dts               |  2 +-
>>>  arch/arm/boot/dts/omap3-overo-base.dtsi        |  3 +--
>>>  arch/arm/boot/dts/omap3-tao3530.dtsi           |  2 +-
>>>  arch/arm/boot/dts/omap3-zoom3.dts              |  2 +-
>>>  arch/arm/boot/dts/omap3430-sdp.dts             |  2 +-
>>>  arch/arm/mach-omap2/pdata-quirks.c             | 31 ++++++++++++++++++++++++++
>>>  include/linux/platform_data/hsmmc-omap.h       |  3 +++
>>>  22 files changed, 114 insertions(+), 20 deletions(-)
>>>
>>> --
>>> 2.11.0
>>>
>>
>> If it helps, I can host a branch with the updates on the omap_hsmmc
>> driver, such it can be pulled in from arm soc?
>>
>> I guess you need that as, the DT changes relies on the new vqmmc
>> binding. Or you thing it doesn't matter, because for the current
>> changes that regulator is always an always on regulator?
>
> Since there is fallback mechanism in omap_hsmmc driver (falls back to
> vmmc_aux), dt changes shouldn't rely on the omap_hsmmc driver.

Ahh, I get it.

So no immutable branch needed, although only the DTS changes is need
for me below.

>
> However I'll send a fix for omap3-overo-base which used vqmmc incorrectly (that
> should be merged before omap_hsmmc driver changes) so that git bisect does not
> break.

Great! Thanks for sorting this out!

>
> Thanks
> Kishon

Kind regards
Uffe

[toc] | [prev] | [standalone]

csiph-web

[PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

Contents

#1659598 — [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

#1659600 — [PATCH v2 8/8] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"

#1662360 — [PATCH v3] ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"

#1659609 — [PATCH v2 1/8] ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2

#1659613 — [PATCH v2 5/8] ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2

#1659621 — [PATCH v2 4/8] ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO lines

#1659628 — [PATCH v2 3/8] ARM: dts: dra72-evm-revc: Add vqmmc supply to mmc1

#1660831

#1660949

#1662375

#1662075

#1662281