Hi,

Sebastian Reichel [off-list ref] hat am 23. Februar 2015 um 16:38
geschrieben:


On Fri, Feb 20, 2015 at 11:57:41AM +0100, Stefan Wahren wrote:

quoted

quoted

power: power at 80044000 {
compatible = "fsl,imx28-power"; /* handled by mxs_power.c */
#address-cells = <1>;
#size-cells = <1>;
reg = <0x80044000 0x2000>;
interrupts = <6>;
switching-frequency = <24000000>; /* new property */
ranges;

reg_vddd: regulator at 80044040 {
reg = <0x80044040 0x10>,
<0x80044010 0x10>,
<0x800440c0 0x10>;
reg-names = "base-address",
"v5ctrl-address",
"status-address";
compatible = "fsl,imx28-vddd"; /* handled by mxs-regulator.c */
regulator-name = "vddd";
regulator-min-microvolt = <1350000>;
regulator-max-microvolt = <1550000>;
vddd-supply = <&reg_vdda>;
};

reg_vdda: regulator at 80044050 {
reg = <0x80044050 0x10>,
<0x80044010 0x10>,
<0x800440c0 0x10>;
reg-names = "base-address",
"v5ctrl-address",
"status-address";
compatible = "fsl,imx28-vdda"; /* handled by mxs-regulator.c */
regulator-name = "vdda";
regulator-min-microvolt = <1725000>;
regulator-max-microvolt = <1950000>;
vdda-supply = <&reg_vddio>;
};

reg_vddio: regulator at 80044060 {
reg = <0x80044060 0x10>,
<0x80044010 0x10>,
<0x800440c0 0x10>;
reg-names = "base-address",
"v5ctrl-address",
"status-address";
compatible = "fsl,imx28-vddio"; /* handled by mxs-regulator.c */
regulator-name = "vddio";
regulator-min-microvolt = <3000000>;
regulator-max-microvolt = <3575000>;
regulator-microvolt-offset = <80000>;
};
};

Please correct me if i'm wrong.

Stefan

Gentle Ping.

The binding looks ok to me. It would be nice to get an ACK of a DT
binding maintainer, though.

any objections about implementing DC-DC switching frequency as a DT property
instead of module parameter?

Stefan