Re: [PATCH v4 5/7] ARM: dts: add pbias dt node

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

 



On Wednesday 11 December 2013 04:12 AM, Tony Lindgren wrote:
* Balaji T K <balajitk@xxxxxx> [131210 02:17]:
Add pbias regulator node as a child of system control
module - syscon.

Signed-off-by: Balaji T K <balajitk@xxxxxx>
---
  arch/arm/boot/dts/dra7.dtsi     |   14 ++++++++++++++
  arch/arm/boot/dts/omap2430.dtsi |   14 ++++++++++++++
  arch/arm/boot/dts/omap3.dtsi    |   14 ++++++++++++++
  arch/arm/boot/dts/omap4.dtsi    |   14 ++++++++++++++
  arch/arm/boot/dts/omap5.dtsi    |   14 ++++++++++++++
  5 files changed, 70 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index d0df4c4..1a52676 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -119,6 +119,19 @@
  			pinctrl-single,function-mask = <0x3fffffff>;
  		};

+		tisyscon: tisyscon@4A100600 {
+			compatible = "ti,control-syscon", "syscon", "simple-bus";
+			reg = <0x4A002E00 0x4>;

This should map the whole GENERAL area of the SCM register space instead
in one or multiple sections or driver instances. This way the same driver
can be used for the misc SCM registers from other drivers.

Hi,

OK, make sense, others driver wont have to patch the base address later on.

If you don't define it that way now, you'll end up with broken child driver
offsets in the .dts files later on.

yes, Unless someone changes the base and not the offset, given that all are in one dtsi


+				pbias_regulator: pbias_regulator {
+					compatible = "regulator-pbias-omap5";
+					pbias-reg-offset = <0>;
+					regulator-name = "pbias_regulator";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <3000000>;
+					regulator-enable-ramp-delay = <10>;

Then the pbias_regulator needs to have the standard reg property as an
offset from the SCM GENERAL base address.

OK

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




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux