| Message ID | 1430346747-28728-5-git-send-email-stefan.wahren@i2se.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Stefan, for the i.MX23 the regulator daisy chain looks a little bit different: +------------+ VDD5V +---------+ VDDIO | DC-DC | +--------> LINREG +--------->+ | | | VDDIO | | | | +---------+ | | DCDC_VDDIO +->-+-----------------------------+ | | | | | | VDDIO +---------+ VDDMEM | | +--------> LINREG +---------> | | | | VDDMEM | | | | +---------+ | | | VDDIO +---------+ VDDA | | +--------> LINREG +--------->+ | | | VDDA | | | | +---------+ | | DCDC_VDDA +->-+-----------------------------+ | | | VDDA +---------+ VDDD | | +--------> LINREG +--------->+ | | | VDDD | | | | +---------+ | | DCDC_VDDD +->-------------------------------+ +------------+ | v Regards, Juergen
Hello Juergen, > Jürgen Borleis <jbe@pengutronix.de> hat am 4. Mai 2015 um 15:51 geschrieben: > > > Hi Stefan, > > for the i.MX23 the regulator daisy chain looks a little bit different: > > +------------+ VDD5V +---------+ VDDIO > | DC-DC | +--------> LINREG +--------->+ > | | | VDDIO | | > | | +---------+ | > | DCDC_VDDIO +->-+-----------------------------+ > | | | > | | | VDDIO +---------+ VDDMEM > | | +--------> LINREG +---------> > | | | | VDDMEM | > | | | +---------+ > | | | VDDIO +---------+ VDDA > | | +--------> LINREG +--------->+ > | | | VDDA | | > | | +---------+ | > | DCDC_VDDA +->-+-----------------------------+ > | | | VDDA +---------+ VDDD > | | +--------> LINREG +--------->+ > | | | VDDD | | > | | +---------+ | > | DCDC_VDDD +->-------------------------------+ > +------------+ | > v damn! I didn't noticed that VDDMEM is powered by VDDIO and VDD1P5 is powered by VDDA. Thank you very much. From my understanding VDDMEM and VDD1P5 describe the same regulator function (provides power to external memory such as SDRAM and mDDR which doesn't fit to VDDD, VDDIO or VDDA). But the usage of the name is not consistent across the documentation: | i.MX23 | i.MX28 ------------------------------------ Application note | VDDMEM | VDD1P5 Reference manual | VDDMEM | VDDMEM Data sheet | - | VDD1P5 I would suggest to use VDDMEM, because the registers are always called HW_VDDMEM (for both SoCs). I think we have 3 options here: a) add the regulator daisy chain for the i.MX23 too b) remove VDD1P5 from the i.MX28 diagram so it would fit for both again c) remove the diagram complete I would prefer option b. What's your opinion? > > Regards, > Juergen Best regards Stefan
Hi Stefan, On Monday 04 May 2015 22:20:50 Stefan Wahren wrote: > [...] > > > > +------------+ VDD5V +---------+ VDDIO > > | DC-DC | +--------> LINREG +--------->+ > > | | | VDDIO | | > > | | +---------+ | > > | | | > > | DCDC_VDDIO +->-+-----------------------------+ > > | | | > > | | | VDDIO +---------+ VDDMEM > > | | +--------> LINREG +---------> > > | | | | VDDMEM | > > | | | +---------+ > > | | | VDDIO +---------+ VDDA > > | | +--------> LINREG +--------->+ > > | | | VDDA | | > > | | +---------+ | > > | DCDC_VDDA +->-+-----------------------------+ > > | | | > > | | | VDDA +---------+ VDDD > > | | +--------> LINREG +--------->+ > > | | | VDDD | | > > | | +---------+ | > > | DCDC_VDDD +->-------------------------------+ > > +------------+ | > > v > > damn! I didn't noticed that VDDMEM is powered by VDDIO and VDD1P5 is > powered by VDDA. Thank you very much. > > From my understanding VDDMEM and VDD1P5 describe the same regulator > function (provides power to external memory such as SDRAM and mDDR which > doesn't fit to VDDD, VDDIO or VDDA). But the usage of > the name is not consistent across the documentation: > | i.MX23 | i.MX28 > ------------------------------------ > Application note | VDDMEM | VDD1P5 > Reference manual | VDDMEM | VDDMEM > Data sheet | - | VDD1P5 > > I would suggest to use VDDMEM, because the registers are always called > HW_VDDMEM (for both SoCs). Ack. > I think we have 3 options here: > > a) add the regulator daisy chain for the i.MX23 too > b) remove VDD1P5 from the i.MX28 diagram so it would fit for both again > c) remove the diagram complete > > I would prefer option b. What's your opinion? I prefer a) because once the details are known, we should keep them. Regards, Juergen
Hi Juergen, > Jürgen Borleis <jbe@pengutronix.de> hat am 5. Mai 2015 um 10:21 geschrieben: > > > Hi Stefan, > > On Monday 04 May 2015 22:20:50 Stefan Wahren wrote: > > [...] > > > > I think we have 3 options here: > > > > a) add the regulator daisy chain for the i.MX23 too > > b) remove VDD1P5 from the i.MX28 diagram so it would fit for both again > > c) remove the diagram complete > > > > I would prefer option b. What's your opinion? > > I prefer a) because once the details are known, we should keep them. beside the point this could be to much diagrams for a binding documentation, i don't have any objections about option a). > > Regards, > Juergen Regards Stefan
diff --git a/Documentation/devicetree/bindings/regulator/mxs-regulator.txt b/Documentation/devicetree/bindings/regulator/mxs-regulator.txt new file mode 100644 index 0000000..e7092fc --- /dev/null +++ b/Documentation/devicetree/bindings/regulator/mxs-regulator.txt @@ -0,0 +1,65 @@ +Regulators on Freescale i.MX23/i.MX28 + +On-chip linear regulator daisy chain: + ++------------+ VDD5V +---------+ VDDIO +| DC-DC | +--------> LINREG +--------->+ +| | | VDDIO | | +| | +---------+ | +| DCDC_VDDIO +->-+-----------------------------+ +| | | +| | | VDDIO +---------+ VDDA +| | +--------> LINREG +--------->+ +| | | VDDA | | +| | +---------+ | +| DCDC_VDDA +->-+-----------------------------+ +| | | +| | | VDDA +---------+ VDD1P5 +| | +--------> LINREG +---------> +| | | | VDD1P5 | +| | | +---------+ +| | | VDDA +---------+ VDDD +| | +--------> LINREG +--------->+ +| | | VDDD | | +| | +---------+ | +| DCDC_VDDD +->-------------------------------+ ++------------+ | + v + +Required properties: +- compatible (imx23 and imx28 supported): + - "fsl,<soc>-dcdc" for DC-DC converter + - "fsl,<soc>-vddd" for VDDD linear regulator + - "fsl,<soc>-vdda" for VDDA linear regulator + - "fsl,<soc>-vddio" for VDDIO linear regulator + +Optional properties: +- switching-frequency: switching frequency for the DC-DC converter in Hz. + Possible values are <1200000>, <1250000> or <1500000> (default). + +Any regulator property defined as part of the core regulator +binding, defined in regulator.txt, can also be used. + +Example for i.MX28: + + power: power@80044000 { + compatible = "fsl,imx28-power", "syscon"; + reg = <0x80044000 0x2000>; + interrupts = <6>; + + dcdc: regulator@1 { + compatible = "fsl,imx28-dcdc"; + regulator-name = "dcdc"; + regulator-boot-on; + regulator-always-on; + switching-frequency = <1250000>; + }; + + reg_vddd: regulator@2 { + compatible = "fsl,imx28-vddd"; + regulator-name = "vddd"; + regulator-min-microvolt = <1350000>; + regulator-max-microvolt = <1550000>; + }; + }; +
This patch adds the device tree bindings for the Freescale MXS on-chip regulators. Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> --- .../bindings/regulator/mxs-regulator.txt | 65 ++++++++++++++++++++ 1 file changed, 65 insertions(+) create mode 100644 Documentation/devicetree/bindings/regulator/mxs-regulator.txt