Re: [PATCH V2 2/2] ARM: dts: DRA7: Add node for RTC

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

 




On Wednesday 09 July 2014 03:39 PM, Tony Lindgren wrote:
* Keerthy <a0393675@xxxxxx> [140709 02:36]:
On Wednesday 09 July 2014 02:42 PM, Tony Lindgren wrote:
* Lokesh Vutla <lokeshvutla@xxxxxx> [140709 01:37]:
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -249,6 +249,7 @@
  					regulator-min-microvolt = <1050000>;
  					regulator-max-microvolt = <1050000>;
  					regulator-boot-on;
+					regulator-always-on;
  				};
Is this regulator really always on?
This feeds on to RTC which is a free running clock. So i guess always on is
justified no?
Well the dts entries should describe the hardware. If the
regulator can be enabled and disabled, we should not claim it's
always on.

From the PMIC perspective every regulator can be enabled and
disabled. From a Board perspective there are some which need
to be always on. For Ex: SMPS123 which feeds on to the MPU.

I guess RTC also needs the supply to be on as long as we want
the clock to be ticking.

Also adding temporary dts entries just causes more churn on the
dts files which is a PITA for everybody.

Regards,

Tony

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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux