* [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
@ 2012-08-07 19:04 ` Dongjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Dongjin Kim @ 2012-08-07 19:04 UTC (permalink / raw)
Cc: Dongjin Kim, Russell King, linux-arm-kernel, linux-kernel
Add a dedicated DTS file for max77686 PMIC.
This file is supposed to be included in a board DTS in order to control the
power for certain consumer devices or AP with 9 BUCKs and 26 LDOs
Signed-off-by: Dongjin Kim <dongjin.kim@agreeyamobility.net>
---
arch/arm/boot/dts/max77686.dtsi | 303 +++++++++++++++++++++++++++++++++++++++
1 file changed, 303 insertions(+)
create mode 100644 arch/arm/boot/dts/max77686.dtsi
diff --git a/arch/arm/boot/dts/max77686.dtsi b/arch/arm/boot/dts/max77686.dtsi
new file mode 100644
index 0000000..4a46167
--- /dev/null
+++ b/arch/arm/boot/dts/max77686.dtsi
@@ -0,0 +1,303 @@
+/*
+ * arch/arm/boot/dts/max77686.dtsi
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+/ {
+ max77686@09 {
+ compatible = "maxim,max77686";
+
+ voltage-regulators {
+ buck1 {
+ regulator-compatible = "BUCK1";
+ regulator-name = "VDD_MIF_1.0V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1050000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck2 {
+ regulator-compatible = "BUCK2";
+ regulator-name = "VDD_ARM_1.3V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck3 {
+ regulator-compatible = "BUCK3";
+ regulator-name = "VDD_INT_1.0V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1150000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck4 {
+ regulator-compatible = "BUCK4";
+ regulator-name = "VDD_G3D_1.0V";
+ regulator-min-microvolt = <850000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-boot-on;
+ };
+
+ buck5 {
+ regulator-compatible = "BUCK5";
+ regulator-name = "VDDQ_CKEM1M2_1.2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck6 {
+ regulator-compatible = "BUCK6";
+ regulator-name = "VDD_INL_1.35V";
+ regulator-min-microvolt = <1350000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck7 {
+ regulator-compatible = "BUCK7";
+ regulator-name = "VDD_INL_2.0V";
+ regulator-min-microvolt = <2000000>;
+ regulator-max-microvolt = <2000000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck8 {
+ regulator-compatible = "BUCK8";
+ regulator-name = "VDD_BUCK8_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck9 {
+ regulator-compatible = "BUCK9";
+ regulator-name = "VDD_BUCK9_1.2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ ldo1 {
+ regulator-compatible = "LDO1";
+ regulator-name = "VDD_ALIVE_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo2 {
+ regulator-compatible = "LDO2";
+ regulator-name = "VDDQ_M1M2_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo3 {
+ regulator-compatible = "LDO3";
+ regulator-name = "VDDQ_M0_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo4 {
+ regulator-compatible = "LDO4";
+ regulator-name = "VDDQ_MMC2_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo5 {
+ regulator-compatible = "LDO5";
+ regulator-name = "VDDQ_MMC13_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo6 {
+ regulator-compatible = "LDO6";
+ regulator-name = "VDD_MPLL_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo7 {
+ regulator-compatible = "LDO7";
+ regulator-name = "VDD_VPLL_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo8 {
+ regulator-compatible = "LDO8";
+ regulator-name = "VDD10_HDMI_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo9 {
+ regulator-compatible = "LDO9";
+ regulator-name = "VDD_VTCORE_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo10 {
+ regulator-compatible = "LDO10";
+ regulator-name = "VDDQ_MIPIHSI_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo11 {
+ regulator-compatible = "LDO11";
+ regulator-name = "VDD18_ABB1_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo12 {
+ regulator-compatible = "LDO12";
+ regulator-name = "VDD33_UOTG_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo13 {
+ regulator-compatible = "LDO13";
+ regulator-name = "VDDQ_C2C_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo14 {
+ regulator-compatible = "LDO14";
+ regulator-name = "VDD18_ABB2_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo15 {
+ regulator-compatible = "LDO15";
+ regulator-name = "VDD10_HSIC_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo16 {
+ regulator-compatible = "LDO16";
+ regulator-name = "VDD18_HSIC_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo17 {
+ regulator-compatible = "LDO17";
+ regulator-name = "VDDQ_CAM_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo18 {
+ regulator-compatible = "LDO18";
+ regulator-name = "VDD_LDO18_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo19 {
+ regulator-compatible = "LDO19";
+ regulator-name = "VDD_VTCAM_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo20 {
+ regulator-compatible = "LDO20";
+ regulator-name = "VDD_LDO20_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo21 {
+ regulator-compatible = "LDO21";
+ regulator-name = "VDD_SDCARD_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo22 {
+ regulator-compatible = "LDO22";
+ regulator-name = "VDD_LDO22_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo23 {
+ regulator-compatible = "LDO23";
+ regulator-name = "VDD_TOUCH_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo24 {
+ regulator-compatible = "LDO24";
+ regulator-name = "VDD_TOUCHLED_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo25 {
+ regulator-compatible = "LDO25";
+ regulator-name = "VDDQ_LCD_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo26 {
+ regulator-compatible = "LDO26";
+ regulator-name = "VDD_MOTOR_3.0V";
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
--
1.7.9.5
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
@ 2012-08-07 19:04 ` Dongjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Dongjin Kim @ 2012-08-07 19:04 UTC (permalink / raw)
To: linux-arm-kernel
Add a dedicated DTS file for max77686 PMIC.
This file is supposed to be included in a board DTS in order to control the
power for certain consumer devices or AP with 9 BUCKs and 26 LDOs
Signed-off-by: Dongjin Kim <dongjin.kim@agreeyamobility.net>
---
arch/arm/boot/dts/max77686.dtsi | 303 +++++++++++++++++++++++++++++++++++++++
1 file changed, 303 insertions(+)
create mode 100644 arch/arm/boot/dts/max77686.dtsi
diff --git a/arch/arm/boot/dts/max77686.dtsi b/arch/arm/boot/dts/max77686.dtsi
new file mode 100644
index 0000000..4a46167
--- /dev/null
+++ b/arch/arm/boot/dts/max77686.dtsi
@@ -0,0 +1,303 @@
+/*
+ * arch/arm/boot/dts/max77686.dtsi
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+/ {
+ max77686 at 09 {
+ compatible = "maxim,max77686";
+
+ voltage-regulators {
+ buck1 {
+ regulator-compatible = "BUCK1";
+ regulator-name = "VDD_MIF_1.0V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1050000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck2 {
+ regulator-compatible = "BUCK2";
+ regulator-name = "VDD_ARM_1.3V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck3 {
+ regulator-compatible = "BUCK3";
+ regulator-name = "VDD_INT_1.0V";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1150000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck4 {
+ regulator-compatible = "BUCK4";
+ regulator-name = "VDD_G3D_1.0V";
+ regulator-min-microvolt = <850000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-boot-on;
+ };
+
+ buck5 {
+ regulator-compatible = "BUCK5";
+ regulator-name = "VDDQ_CKEM1M2_1.2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck6 {
+ regulator-compatible = "BUCK6";
+ regulator-name = "VDD_INL_1.35V";
+ regulator-min-microvolt = <1350000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck7 {
+ regulator-compatible = "BUCK7";
+ regulator-name = "VDD_INL_2.0V";
+ regulator-min-microvolt = <2000000>;
+ regulator-max-microvolt = <2000000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck8 {
+ regulator-compatible = "BUCK8";
+ regulator-name = "VDD_BUCK8_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ buck9 {
+ regulator-compatible = "BUCK9";
+ regulator-name = "VDD_BUCK9_1.2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ ldo1 {
+ regulator-compatible = "LDO1";
+ regulator-name = "VDD_ALIVE_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo2 {
+ regulator-compatible = "LDO2";
+ regulator-name = "VDDQ_M1M2_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo3 {
+ regulator-compatible = "LDO3";
+ regulator-name = "VDDQ_M0_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo4 {
+ regulator-compatible = "LDO4";
+ regulator-name = "VDDQ_MMC2_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo5 {
+ regulator-compatible = "LDO5";
+ regulator-name = "VDDQ_MMC13_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo6 {
+ regulator-compatible = "LDO6";
+ regulator-name = "VDD_MPLL_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo7 {
+ regulator-compatible = "LDO7";
+ regulator-name = "VDD_VPLL_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo8 {
+ regulator-compatible = "LDO8";
+ regulator-name = "VDD10_HDMI_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo9 {
+ regulator-compatible = "LDO9";
+ regulator-name = "VDD_VTCORE_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo10 {
+ regulator-compatible = "LDO10";
+ regulator-name = "VDDQ_MIPIHSI_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo11 {
+ regulator-compatible = "LDO11";
+ regulator-name = "VDD18_ABB1_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo12 {
+ regulator-compatible = "LDO12";
+ regulator-name = "VDD33_UOTG_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo13 {
+ regulator-compatible = "LDO13";
+ regulator-name = "VDDQ_C2C_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo14 {
+ regulator-compatible = "LDO14";
+ regulator-name = "VDD18_ABB2_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo15 {
+ regulator-compatible = "LDO15";
+ regulator-name = "VDD10_HSIC_1.0V";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <1000000>;
+ regulator-always-on;
+ };
+
+ ldo16 {
+ regulator-compatible = "LDO16";
+ regulator-name = "VDD18_HSIC_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo17 {
+ regulator-compatible = "LDO17";
+ regulator-name = "VDDQ_CAM_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo18 {
+ regulator-compatible = "LDO18";
+ regulator-name = "VDD_LDO18_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo19 {
+ regulator-compatible = "LDO19";
+ regulator-name = "VDD_VTCAM_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo20 {
+ regulator-compatible = "LDO20";
+ regulator-name = "VDD_LDO20_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo21 {
+ regulator-compatible = "LDO21";
+ regulator-name = "VDD_SDCARD_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo22 {
+ regulator-compatible = "LDO22";
+ regulator-name = "VDD_LDO22_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo23 {
+ regulator-compatible = "LDO23";
+ regulator-name = "VDD_TOUCH_2.8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-always-on;
+ };
+
+ ldo24 {
+ regulator-compatible = "LDO24";
+ regulator-name = "VDD_TOUCHLED_3.3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ ldo25 {
+ regulator-compatible = "LDO25";
+ regulator-name = "VDDQ_LCD_1.8V";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ };
+
+ ldo26 {
+ regulator-compatible = "LDO26";
+ regulator-name = "VDD_MOTOR_3.0V";
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
--
1.7.9.5
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
2012-08-07 19:04 ` Dongjin Kim
@ 2012-08-07 19:34 ` Mark Brown
-1 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2012-08-07 19:34 UTC (permalink / raw)
To: Dongjin Kim; +Cc: Russell King, linux-kernel, linux-arm-kernel, Dongjin Kim
On Wed, Aug 08, 2012 at 04:04:11AM +0900, Dongjin Kim wrote:
> + buck1 {
> + regulator-compatible = "BUCK1";
> + regulator-name = "VDD_MIF_1.0V";
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1050000>;
> + regulator-always-on;
> + regulator-boot-on;
> + };
These settings for the regulators are clearly specific to the board
rather than the device so should be in the board file.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
@ 2012-08-07 19:34 ` Mark Brown
0 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2012-08-07 19:34 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 08, 2012 at 04:04:11AM +0900, Dongjin Kim wrote:
> + buck1 {
> + regulator-compatible = "BUCK1";
> + regulator-name = "VDD_MIF_1.0V";
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1050000>;
> + regulator-always-on;
> + regulator-boot-on;
> + };
These settings for the regulators are clearly specific to the board
rather than the device so should be in the board file.
^ permalink raw reply [flat|nested] 8+ messages in thread
* 회신: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
2012-08-07 19:34 ` Mark Brown
@ 2012-08-07 19:50 ` Dongjin Kim
-1 siblings, 0 replies; 8+ messages in thread
From: Dongjin Kim @ 2012-08-07 19:50 UTC (permalink / raw)
To: Mark Brown; +Cc: Russell King, linux-kernel, linux-arm-kernel, Dongjin Kim
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="ks_c_5601-1987", Size: 1701 bytes --]
Hello,
Are you meaning VDD_MIF_1.0V, for example, would be replaced with another common name for other boards and my board file, exynos4412-odroidx.dts, describe the name again if necessary? Like below?
max77686.dtsi:
buck1 {
regulator-compatible = "BUCK1";
regulator-name = "VDD_BUCK1_1.0V";
regulator-min-microvolt = <800000>;
regulator-max-microvolt = <1050000>;
regulator-always-on;
regulator-boot-on;
};
my board's DTS file:
buck1 {
regulator-name = "VDD_MIF_1.0V";
}
Thank you in advance.
Dongjin.
________________________________________
º¸³½ »ç¶÷: Mark Brown [broonie@opensource.wolfsonmicro.com]
º¸³½ ³¯Â¥: 2012³â 8¿ù 8ÀÏ ¼ö¿äÀÏ ¿ÀÀü 4:34
¹Þ´Â »ç¶÷: Dongjin Kim
ÂüÁ¶: Russell King; linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Dongjin Kim
Á¦¸ñ: Re: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
On Wed, Aug 08, 2012 at 04:04:11AM +0900, Dongjin Kim wrote:
> + buck1 {
> + regulator-compatible = "BUCK1";
> + regulator-name = "VDD_MIF_1.0V";
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1050000>;
> + regulator-always-on;
> + regulator-boot-on;
> + };
These settings for the regulators are clearly specific to the board
rather than the device so should be in the board file.
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 8+ messages in thread
* 회신: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
@ 2012-08-07 19:50 ` Dongjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Dongjin Kim @ 2012-08-07 19:50 UTC (permalink / raw)
To: linux-arm-kernel
Hello,
Are you meaning VDD_MIF_1.0V, for example, would be replaced with another common name for other boards and my board file, exynos4412-odroidx.dts, describe the name again if necessary? Like below?
max77686.dtsi:
buck1 {
regulator-compatible = "BUCK1";
regulator-name = "VDD_BUCK1_1.0V";
regulator-min-microvolt = <800000>;
regulator-max-microvolt = <1050000>;
regulator-always-on;
regulator-boot-on;
};
my board's DTS file:
buck1 {
regulator-name = "VDD_MIF_1.0V";
}
Thank you in advance.
Dongjin.
________________________________________
?? ??: Mark Brown [broonie at opensource.wolfsonmicro.com]
?? ??: 2012? 8? 8? ??? ?? 4:34
?? ??: Dongjin Kim
??: Russell King; linux-kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org; Dongjin Kim
??: Re: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
On Wed, Aug 08, 2012 at 04:04:11AM +0900, Dongjin Kim wrote:
> + buck1 {
> + regulator-compatible = "BUCK1";
> + regulator-name = "VDD_MIF_1.0V";
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1050000>;
> + regulator-always-on;
> + regulator-boot-on;
> + };
These settings for the regulators are clearly specific to the board
rather than the device so should be in the board file.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: ȸ��: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
2012-08-07 19:50 ` Dongjin Kim
@ 2012-08-08 13:30 ` Mark Brown
-1 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2012-08-08 13:30 UTC (permalink / raw)
To: Dongjin Kim; +Cc: Russell King, linux-kernel, linux-arm-kernel
On Tue, Aug 07, 2012 at 07:50:11PM +0000, Dongjin Kim wrote:
Please fix your mailer to word wrap within paragraphs.
> Are you meaning VDD_MIF_1.0V, for example, would be replaced with
> another common name for other boards and my board file,
> exynos4412-odroidx.dts, describe the name again if necessary? Like
> below?
No, the entire configuration.
> max77686.dtsi:
> buck1 {
> regulator-compatible = "BUCK1";
> regulator-name = "VDD_BUCK1_1.0V";
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <1050000>;
> regulator-always-on;
> regulator-boot-on;
> };
Especially things like the voltage range might not be appropriate if the
regulator is used for a different purpose on a different board.
^ permalink raw reply [flat|nested] 8+ messages in thread
* ȸ��: [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC
@ 2012-08-08 13:30 ` Mark Brown
0 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2012-08-08 13:30 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 07, 2012 at 07:50:11PM +0000, Dongjin Kim wrote:
Please fix your mailer to word wrap within paragraphs.
> Are you meaning VDD_MIF_1.0V, for example, would be replaced with
> another common name for other boards and my board file,
> exynos4412-odroidx.dts, describe the name again if necessary? Like
> below?
No, the entire configuration.
> max77686.dtsi:
> buck1 {
> regulator-compatible = "BUCK1";
> regulator-name = "VDD_BUCK1_1.0V";
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <1050000>;
> regulator-always-on;
> regulator-boot-on;
> };
Especially things like the voltage range might not be appropriate if the
regulator is used for a different purpose on a different board.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-08-08 13:30 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-07 19:04 [PATCH 1/4] arm/dts: max77686: Add DTS file for max77686 PMIC Dongjin Kim
2012-08-07 19:04 ` Dongjin Kim
2012-08-07 19:34 ` Mark Brown
2012-08-07 19:34 ` Mark Brown
2012-08-07 19:50 ` 회신: " Dongjin Kim
2012-08-07 19:50 ` Dongjin Kim
2012-08-08 13:30 ` ȸ��: " Mark Brown
2012-08-08 13:30 ` Mark Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.