linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: am335x-boneblack: move fixed regulator to board level
@ 2013-09-19 17:12 Nishanth Menon
  2013-09-20 16:21 ` Felipe Balbi
  0 siblings, 1 reply; 3+ messages in thread
From: Nishanth Menon @ 2013-09-19 17:12 UTC (permalink / raw)
  To: linux-arm-kernel

3.3V fixed regulator does not belong to TPS node - as a result
the fixed regulator is never probed and MMC is continually deferred due
to lack of regulator.

Move the fixed regulator to be at root of platform.

Signed-off-by: Nishanth Menon <nm@ti.com>
---

Based on: (benoit's for_3.13/dts branch)
https://git.kernel.org/cgit/linux/kernel/git/bcousson/linux-omap-dt.git/log/?h=for_3.13/dts
 +
https://patchwork.kernel.org/patch/2902711/

Test Log:
before: http://pastebin.com/UuCsBg2d
After: http://pastebin.com/149FWujf
notice how "vmmcsd_fixed: 3300 mV " does not appear in the log before the patch.

 arch/arm/boot/dts/am335x-bone-common.dtsi |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index 56361ce..fe2742e 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -215,6 +215,13 @@
 			default-state = "off";
 		};
 	};
+
+	vmmcsd_fixed: fixedregulator at 0 {
+		compatible = "regulator-fixed";
+		regulator-name = "vmmcsd_fixed";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+	};
 };
 
 /include/ "tps65217.dtsi"
@@ -259,13 +266,6 @@
 			regulator-always-on;
 		};
 	};
-
-	vmmcsd_fixed: fixedregulator at 0 {
-		compatible = "regulator-fixed";
-		regulator-name = "vmmcsd_fixed";
-		regulator-min-microvolt = <3300000>;
-		regulator-max-microvolt = <3300000>;
-	};
 };
 
 &cpsw_emac0 {
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH] ARM: dts: am335x-boneblack: move fixed regulator to board level
  2013-09-19 17:12 [PATCH] ARM: dts: am335x-boneblack: move fixed regulator to board level Nishanth Menon
@ 2013-09-20 16:21 ` Felipe Balbi
  2013-09-27 14:37   ` Balaji T K
  0 siblings, 1 reply; 3+ messages in thread
From: Felipe Balbi @ 2013-09-20 16:21 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Sep 19, 2013 at 12:12:44PM -0500, Nishanth Menon wrote:
> 3.3V fixed regulator does not belong to TPS node - as a result
> the fixed regulator is never probed and MMC is continually deferred due
> to lack of regulator.
> 
> Move the fixed regulator to be at root of platform.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Tested-by: Felipe Balbi <balbi@ti.com>

I boot tested this on my BBB.

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130920/d67cbab2/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] ARM: dts: am335x-boneblack: move fixed regulator to board level
  2013-09-20 16:21 ` Felipe Balbi
@ 2013-09-27 14:37   ` Balaji T K
  0 siblings, 0 replies; 3+ messages in thread
From: Balaji T K @ 2013-09-27 14:37 UTC (permalink / raw)
  To: linux-arm-kernel

On Friday 20 September 2013 09:51 PM, Felipe Balbi wrote:
> On Thu, Sep 19, 2013 at 12:12:44PM -0500, Nishanth Menon wrote:
>> 3.3V fixed regulator does not belong to TPS node - as a result
>> the fixed regulator is never probed and MMC is continually deferred due
>> to lack of regulator.
>>
>> Move the fixed regulator to be at root of platform.
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>
> Tested-by: Felipe Balbi <balbi@ti.com>
>

Tested-by: Balaji T K <balajitk@ti.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-09-27 14:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-19 17:12 [PATCH] ARM: dts: am335x-boneblack: move fixed regulator to board level Nishanth Menon
2013-09-20 16:21 ` Felipe Balbi
2013-09-27 14:37   ` Balaji T K

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).