public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH v2] ARM: dts: Add regulator entries to Exynos5420 SMDK board
       [not found] <1386236664-27763-1-git-send-email-sachin.kamat@linaro.org>
@ 2013-12-09 21:27 ` Kukjin Kim
  2013-12-18 15:18   ` Tomasz Figa
  0 siblings, 1 reply; 2+ messages in thread
From: Kukjin Kim @ 2013-12-09 21:27 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-samsung-soc, kgene.kim, devicetree@vger.kernel.org

On 12/05/13 18:44, Sachin Kamat wrote:
> Added regulator entries to Exynos5420 SMDK board.
>
> Signed-off-by: Sachin Kamat<sachin.kamat@linaro.org>
> ---
> Changes since v1:
> Changed node name
> ---
>   arch/arm/boot/dts/exynos5420-smdk5420.dts |  216 +++++++++++++++++++++++++++++
>   1 file changed, 216 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> index fb5a1e25c632..62e1ebe4e989 100644
> --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> @@ -120,4 +120,220 @@
>   			reg =<0x50>;
>   		};
>   	};
> +
> +	hsi2c_4: i2c@12CA0000 {
> +		status = "okay";
> +
> +		s2mps11_pmic@66 {
> +			compatible = "samsung,s2mps11-pmic";
> +			reg =<0x66>;
> +			s2mps11,buck2-ramp-delay =<12>;
> +			s2mps11,buck34-ramp-delay =<12>;
> +			s2mps11,buck16-ramp-delay =<12>;
> +			s2mps11,buck6-ramp-enable =<1>;
> +			s2mps11,buck2-ramp-enable =<1>;
> +			s2mps11,buck3-ramp-enable =<1>;
> +			s2mps11,buck4-ramp-enable =<1>;
> +
> +			s2mps11_osc: clocks {
> +				#clock-cells =<1>;
> +				clock-output-names = "s2mps11_ap",
> +						"s2mps11_cp", "s2mps11_bt";
> +			};
> +
> +			regulators {
> +				ldo1_reg: LDO1 {
> +					regulator-name = "vdd_ldo1";
> +					regulator-min-microvolt =<1000000>;
> +					regulator-max-microvolt =<1000000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo3_reg: LDO3 {
> +					regulator-name = "vdd_ldo3";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo5_reg: LDO5 {
> +					regulator-name = "vdd_ldo5";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo6_reg: LDO6 {
> +					regulator-name = "vdd_ldo6";
> +					regulator-min-microvolt =<1000000>;
> +					regulator-max-microvolt =<1000000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo7_reg: LDO7 {
> +					regulator-name = "vdd_ldo7";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo8_reg: LDO8 {
> +					regulator-name = "vdd_ldo8";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo9_reg: LDO9 {
> +					regulator-name = "vdd_ldo9";
> +					regulator-min-microvolt =<3000000>;
> +					regulator-max-microvolt =<3000000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo10_reg: LDO10 {
> +					regulator-name = "vdd_ldo10";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo11_reg: LDO11 {
> +					regulator-name = "vdd_ldo11";
> +					regulator-min-microvolt =<1000000>;
> +					regulator-max-microvolt =<1000000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo12_reg: LDO12 {
> +					regulator-name = "vdd_ldo12";
> +					regulator-min-microvolt =<1800000>;
> +					regulator-max-microvolt =<1800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo13_reg: LDO13 {
> +					regulator-name = "vdd_ldo13";
> +					regulator-min-microvolt =<2800000>;
> +					regulator-max-microvolt =<2800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo15_reg: LDO15 {
> +					regulator-name = "vdd_ldo15";
> +					regulator-min-microvolt =<3100000>;
> +					regulator-max-microvolt =<3100000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo16_reg: LDO16 {
> +					regulator-name = "vdd_ldo16";
> +					regulator-min-microvolt =<2200000>;
> +					regulator-max-microvolt =<2200000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo17_reg: LDO17 {
> +					regulator-name = "tsp_avdd";
> +					regulator-min-microvolt =<3300000>;
> +					regulator-max-microvolt =<3300000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo19_reg: LDO19 {
> +					regulator-name = "vdd_sd";
> +					regulator-min-microvolt =<2800000>;
> +					regulator-max-microvolt =<2800000>;
> +					regulator-always-on;
> +				};
> +
> +				ldo24_reg: LDO24 {
> +					regulator-name = "tsp_io";
> +					regulator-min-microvolt =<2800000>;
> +					regulator-max-microvolt =<2800000>;
> +					regulator-always-on;
> +				};
> +
> +				buck1_reg: BUCK1 {
> +					regulator-name = "vdd_mif";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1300000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck2_reg: BUCK2 {
> +					regulator-name = "vdd_arm";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1500000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck3_reg: BUCK3 {
> +					regulator-name = "vdd_int";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1400000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck4_reg: BUCK4 {
> +					regulator-name = "vdd_g3d";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1400000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck5_reg: BUCK5 {
> +					regulator-name = "vdd_mem";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1400000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck6_reg: BUCK6 {
> +					regulator-name = "vdd_kfc";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1500000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck7_reg: BUCK7 {
> +					regulator-name = "vdd_1.0v_ldo";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1500000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck8_reg: BUCK8 {
> +					regulator-name = "vdd_1.8v_ldo";
> +					regulator-min-microvolt =<800000>;
> +					regulator-max-microvolt =<1500000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck9_reg: BUCK9 {
> +					regulator-name = "vdd_2.8v_ldo";
> +					regulator-min-microvolt =<3000000>;
> +					regulator-max-microvolt =<3750000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				buck10_reg: BUCK10 {
> +					regulator-name = "vdd_vmem";
> +					regulator-min-microvolt =<2850000>;
> +					regulator-max-microvolt =<2850000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +			};
> +		};
> +	};
>   };

just + DT ml.

- Kukjin

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

* Re: [PATCH v2] ARM: dts: Add regulator entries to Exynos5420 SMDK board
  2013-12-09 21:27 ` [PATCH v2] ARM: dts: Add regulator entries to Exynos5420 SMDK board Kukjin Kim
@ 2013-12-18 15:18   ` Tomasz Figa
  0 siblings, 0 replies; 2+ messages in thread
From: Tomasz Figa @ 2013-12-18 15:18 UTC (permalink / raw)
  To: Kukjin Kim; +Cc: Sachin Kamat, linux-samsung-soc, devicetree@vger.kernel.org

Hi Kukjin,

On Tuesday 10 of December 2013 06:27:27 Kukjin Kim wrote:
> On 12/05/13 18:44, Sachin Kamat wrote:
> > Added regulator entries to Exynos5420 SMDK board.
> >
> > Signed-off-by: Sachin Kamat<sachin.kamat@linaro.org>
> > ---
> > Changes since v1:
> > Changed node name
> > ---
> >   arch/arm/boot/dts/exynos5420-smdk5420.dts |  216 +++++++++++++++++++++++++++++
> >   1 file changed, 216 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> > index fb5a1e25c632..62e1ebe4e989 100644
> > --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> > +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> > @@ -120,4 +120,220 @@
> >   			reg =<0x50>;
> >   		};
> >   	};
> > +
> > +	hsi2c_4: i2c@12CA0000 {
> > +		status = "okay";
> > +
> > +		s2mps11_pmic@66 {
> > +			compatible = "samsung,s2mps11-pmic";
> > +			reg =<0x66>;
> > +			s2mps11,buck2-ramp-delay =<12>;
> > +			s2mps11,buck34-ramp-delay =<12>;
> > +			s2mps11,buck16-ramp-delay =<12>;
> > +			s2mps11,buck6-ramp-enable =<1>;
> > +			s2mps11,buck2-ramp-enable =<1>;
> > +			s2mps11,buck3-ramp-enable =<1>;
> > +			s2mps11,buck4-ramp-enable =<1>;
> > +
> > +			s2mps11_osc: clocks {
> > +				#clock-cells =<1>;
> > +				clock-output-names = "s2mps11_ap",
> > +						"s2mps11_cp", "s2mps11_bt";
> > +			};
> > +
> > +			regulators {
> > +				ldo1_reg: LDO1 {
> > +					regulator-name = "vdd_ldo1";
> > +					regulator-min-microvolt =<1000000>;
> > +					regulator-max-microvolt =<1000000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo3_reg: LDO3 {
> > +					regulator-name = "vdd_ldo3";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo5_reg: LDO5 {
> > +					regulator-name = "vdd_ldo5";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo6_reg: LDO6 {
> > +					regulator-name = "vdd_ldo6";
> > +					regulator-min-microvolt =<1000000>;
> > +					regulator-max-microvolt =<1000000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo7_reg: LDO7 {
> > +					regulator-name = "vdd_ldo7";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo8_reg: LDO8 {
> > +					regulator-name = "vdd_ldo8";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo9_reg: LDO9 {
> > +					regulator-name = "vdd_ldo9";
> > +					regulator-min-microvolt =<3000000>;
> > +					regulator-max-microvolt =<3000000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo10_reg: LDO10 {
> > +					regulator-name = "vdd_ldo10";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo11_reg: LDO11 {
> > +					regulator-name = "vdd_ldo11";
> > +					regulator-min-microvolt =<1000000>;
> > +					regulator-max-microvolt =<1000000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo12_reg: LDO12 {
> > +					regulator-name = "vdd_ldo12";
> > +					regulator-min-microvolt =<1800000>;
> > +					regulator-max-microvolt =<1800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo13_reg: LDO13 {
> > +					regulator-name = "vdd_ldo13";
> > +					regulator-min-microvolt =<2800000>;
> > +					regulator-max-microvolt =<2800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo15_reg: LDO15 {
> > +					regulator-name = "vdd_ldo15";
> > +					regulator-min-microvolt =<3100000>;
> > +					regulator-max-microvolt =<3100000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo16_reg: LDO16 {
> > +					regulator-name = "vdd_ldo16";
> > +					regulator-min-microvolt =<2200000>;
> > +					regulator-max-microvolt =<2200000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo17_reg: LDO17 {
> > +					regulator-name = "tsp_avdd";
> > +					regulator-min-microvolt =<3300000>;
> > +					regulator-max-microvolt =<3300000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo19_reg: LDO19 {
> > +					regulator-name = "vdd_sd";
> > +					regulator-min-microvolt =<2800000>;
> > +					regulator-max-microvolt =<2800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				ldo24_reg: LDO24 {
> > +					regulator-name = "tsp_io";
> > +					regulator-min-microvolt =<2800000>;
> > +					regulator-max-microvolt =<2800000>;
> > +					regulator-always-on;
> > +				};
> > +
> > +				buck1_reg: BUCK1 {
> > +					regulator-name = "vdd_mif";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1300000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck2_reg: BUCK2 {
> > +					regulator-name = "vdd_arm";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1500000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck3_reg: BUCK3 {
> > +					regulator-name = "vdd_int";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1400000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck4_reg: BUCK4 {
> > +					regulator-name = "vdd_g3d";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1400000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck5_reg: BUCK5 {
> > +					regulator-name = "vdd_mem";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1400000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck6_reg: BUCK6 {
> > +					regulator-name = "vdd_kfc";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1500000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck7_reg: BUCK7 {
> > +					regulator-name = "vdd_1.0v_ldo";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1500000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck8_reg: BUCK8 {
> > +					regulator-name = "vdd_1.8v_ldo";
> > +					regulator-min-microvolt =<800000>;
> > +					regulator-max-microvolt =<1500000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck9_reg: BUCK9 {
> > +					regulator-name = "vdd_2.8v_ldo";
> > +					regulator-min-microvolt =<3000000>;
> > +					regulator-max-microvolt =<3750000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +
> > +				buck10_reg: BUCK10 {
> > +					regulator-name = "vdd_vmem";
> > +					regulator-min-microvolt =<2850000>;
> > +					regulator-max-microvolt =<2850000>;
> > +					regulator-always-on;
> > +					regulator-boot-on;
> > +				};
> > +			};
> > +		};
> > +	};
> >   };
> 
> just + DT ml.

AFAIK, current recommendation after the KS is that changes touching only
DTS files don't have to be posted on DT ML. Just changes related to DT
bindings such as Documentation/devicetree and DT parsing code. Full series
doing both still need to be posted there, though.

Best regards,
Tomasz

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

end of thread, other threads:[~2013-12-18 15:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1386236664-27763-1-git-send-email-sachin.kamat@linaro.org>
2013-12-09 21:27 ` [PATCH v2] ARM: dts: Add regulator entries to Exynos5420 SMDK board Kukjin Kim
2013-12-18 15:18   ` Tomasz Figa

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox