Hi Sylwester, Andrzej,
On Monday 01 of July 2013 17:22:34 Sylwester Nawrocki wrote:
> From: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
>
> Add MAX8998 LDO12 and fixed voltage regulator nodes. While at it,
> all fixed voltage regulator nodes are grouped in a 'regulators' node.
>
> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> ---
> arch/arm/boot/dts/exynos4210-trats.dts | 80
> +++++++++++++++++++++++++------- 1 file changed, 64 insertions(+), 16
> deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210-trats.dts
> b/arch/arm/boot/dts/exynos4210-trats.dts index 6b1568e..f62e299 100644
> --- a/arch/arm/boot/dts/exynos4210-trats.dts
> +++ b/arch/arm/boot/dts/exynos4210-trats.dts
> @@ -30,13 +30,64 @@
> bootargs = "console=ttySAC2,115200N8 root=/dev/mmcblk0p5
rootwait
> earlyprintk panic=5"; };
>
> - vemmc_reg: voltage-regulator@0 {
> - compatible = "regulator-fixed";
> - regulator-name = "VMEM_VDD_2.8V";
> - regulator-min-microvolt = <2800000>;
> - regulator-max-microvolt = <2800000>;
> - gpio = <&gpk0 2 0>;
> - enable-active-high;
> + regulators {
> + compatible = "simple-bus";
> + #address-cells = <1>;
> + #size-cells = <0>;
I don't think any addressing is needed for these regulators, so I'd
suggest removing those #properties and replacing @N with -N suffix.
Otherwise looks good.
Best regards,
Tomasz
> +
> + vemmc_reg: regulator@0 {
> + compatible = "regulator-fixed";
> + regulator-name = "VMEM_VDD_2.8V";
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> + gpio = <&gpk0 2 0>;
> + enable-active-high;
> + };
> +
> + tsp_reg: regulator@1 {
> + compatible = "regulator-fixed";
> + regulator-name = "TSP_FIXED_VOLTAGES";
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> + gpio = <&gpl0 3 0>;
> + enable-active-high;
> + };
> +
> + cam_af_28v_reg: regulator@2 {
> + compatible = "regulator-fixed";
> + regulator-name = "8M_AF_2.8V_EN";
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> + gpio = <&gpk1 1 0>;
> + enable-active-high;
> + };
> +
> + cam_io_en_reg: regulator@3 {
> + compatible = "regulator-fixed";
> + regulator-name = "CAM_IO_EN";
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> + gpio = <&gpe2 1 0>;
> + enable-active-high;
> + };
> +
> + cam_io_12v_reg: regulator@4 {
> + compatible = "regulator-fixed";
> + regulator-name = "8M_1.2V_EN";
> + regulator-min-microvolt = <1200000>;
> + regulator-max-microvolt = <1200000>;
> + gpio = <&gpe2 5 0>;
> + enable-active-high;
> + };
> +
> + vt_core_15v_reg: regulator@5 {
> + compatible = "regulator-fixed";
> + regulator-name = "VT_CORE_1.5V";
> + regulator-min-microvolt = <1500000>;
> + regulator-max-microvolt = <1500000>;
> + gpio = <&gpe2 2 0>;
> + enable-active-high;
> + };
> };
>
> sdhci_emmc: sdhci@12510000 {
> @@ -97,15 +148,6 @@
> };
> };
>
> - tsp_reg: voltage-regulator {
> - compatible = "regulator-fixed";
> - regulator-name = "TSP_FIXED_VOLTAGES";
> - regulator-min-microvolt = <2800000>;
> - regulator-max-microvolt = <2800000>;
> - gpio = <&gpl0 3 0>;
> - enable-active-high;
> - };
> -
> i2c@13890000 {
> samsung,i2c-sda-delay = <100>;
> samsung,i2c-slave-addr = <0x10>;
> @@ -218,6 +260,12 @@
> regulator-always-on;
> };
>
> + vtcam_reg: LDO12 {
> + regulator-name = "VT_CAM_1.8V";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + };
> +
> vcclcd_reg: LDO13 {
> regulator-name = "VCC_3.3V_LCD";
> regulator-min-microvolt = <3300000>;
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
