On Tue, May 28, 2013 at 04:05:22PM +0530, Padmavathi Venna wrote:
> + avdd2:fixed-regulator@0 {
> + compatible = "regulator-fixed";
> + regulator-name = "avdd2-supply";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-always-on;
> + };
> + cpvdd:fixed-regulator@1 {
> + compatible = "regulator-fixed";
> + regulator-name = "cpvdd-supply";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-always-on;
> + };
Several of these look like the same supply (especially the speaker ones)
so it seems like it'd be sensible to define a single regulator for each
of them. Not much practical difference but it might be neater.
> +
> + dbvdd:fixed-regulator@2 {
> + compatible = "regulator-fixed";
> + regulator-name = "dbvdd-supply";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + spkvdd1:fixed-regulator@3 {
> + compatible = "regulator-fixed";
> + regulator-name = "spkvdd1-supply";
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5000000>;
> + regulator-always-on;
> + };
> +
> + spkvdd2:fixed-regulator@4 {
> + compatible = "regulator-fixed";
> + regulator-name = "spkvdd2-supply";
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5000000>;
> + regulator-always-on;
> + };
> +
> i2c@12C70000 {
> samsung,i2c-sda-delay = <100>;
> samsung,i2c-max-bus-freq = <20000>;
> @@ -47,8 +87,17 @@
> };
>
> wm8994: wm8994@1a {
> - compatible = "wlf,wm8994";
> - reg = <0x1a>;
> + compatible = "wlf,wm8994";
> + reg = <0x1a>;
> +
> + gpio-controller;
> + #gpio-cells = <2>;
> +
> + AVDD2-supply = <&avdd2>;
> + CPVDD-supply = <&cpvdd>;
> + DBVDD-supply = <&dbvdd>;
> + SPKVDD1-supply = <&spkvdd1>;
> + SPKVDD2-supply = <&spkvdd2>;
> };
> };
>
> --
> 1.7.4.4
>
>
signature.asc
Description: Digital signature
