Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Jagan Teki
Hi Fabio, On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi wrote: > Hi > > On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: >> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi >> wrote: >> Why don't

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Jagan Teki
Hi Fabio, On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi wrote: > Hi > > On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: >> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi >> wrote: >> Why don't you simply do like this? reg_3p3v: regulator-3p3v {

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Michael Trimarchi
Hi On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: > On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi > wrote: > >>> Why don't you simply do like this? >>> >>>reg_3p3v: regulator-3p3v { >>>compatible =

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Michael Trimarchi
Hi On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: > On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi > wrote: > >>> Why don't you simply do like this? >>> >>>reg_3p3v: regulator-3p3v { >>>compatible = "regulator-fixed"; >>>regulator-name =

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Fabio Estevam
On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi wrote: >> Why don't you simply do like this? >> >>reg_3p3v: regulator-3p3v { >>compatible = "regulator-fixed"; >>regulator-name = "3P3V"; >>

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Fabio Estevam
On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi wrote: >> Why don't you simply do like this? >> >>reg_3p3v: regulator-3p3v { >>compatible = "regulator-fixed"; >>regulator-name = "3P3V"; >>regulator-min-microvolt = <330>; >>

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Michael Trimarchi
Hi On Mon, Sep 26, 2016 at 3:47 PM, Fabio Estevam wrote: > Hi Matteo, > > On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi wrote: >> Hi Fabio, >> >> Our SOM doesn't use external PMIC. >> >> We powered all devices with a fix voltage regulators that

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Michael Trimarchi
Hi On Mon, Sep 26, 2016 at 3:47 PM, Fabio Estevam wrote: > Hi Matteo, > > On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi wrote: >> Hi Fabio, >> >> Our SOM doesn't use external PMIC. >> >> We powered all devices with a fix voltage regulators that cannot be drive >> directly from SoC so >> any

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Matteo Lisi
Hi Fabio, Our SOM doesn't use external PMIC. We powered all devices with a fix voltage regulators that cannot be drive directly from SoC so any external power rail can be driven by drivers. Regards Matteo Il 21/09/2016 18:46, Fabio Estevam ha scritto: Hi Michael, On Tue, Sep 20, 2016

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Matteo Lisi
Hi Fabio, Our SOM doesn't use external PMIC. We powered all devices with a fix voltage regulators that cannot be drive directly from SoC so any external power rail can be driven by drivers. Regards Matteo Il 21/09/2016 18:46, Fabio Estevam ha scritto: Hi Michael, On Tue, Sep 20, 2016

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Fabio Estevam
Hi Matteo, On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi wrote: > Hi Fabio, > > Our SOM doesn't use external PMIC. > > We powered all devices with a fix voltage regulators that cannot be drive > directly from SoC so > any external power rail can be driven by drivers.

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-26 Thread Fabio Estevam
Hi Matteo, On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi wrote: > Hi Fabio, > > Our SOM doesn't use external PMIC. > > We powered all devices with a fix voltage regulators that cannot be drive > directly from SoC so > any external power rail can be driven by drivers. Sure, this is fine. Why

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-21 Thread Fabio Estevam
Hi Michael, On Tue, Sep 20, 2016 at 10:23 AM, Michael Trimarchi wrote: > Engicam use fixed regulator always on and on on boot. Their board does > not have any > external pmu. Is this answer to your comment? All I am saying is that "regulator-boot-on" and

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-21 Thread Fabio Estevam
Hi Michael, On Tue, Sep 20, 2016 at 10:23 AM, Michael Trimarchi wrote: > Engicam use fixed regulator always on and on on boot. Their board does > not have any > external pmu. Is this answer to your comment? All I am saying is that "regulator-boot-on" and "regulator-always-on" properties

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-20 Thread Michael Trimarchi
Hi Fabio On Wed, Sep 14, 2016 at 8:00 PM, Fabio Estevam wrote: > On Wed, Sep 14, 2016 at 2:43 PM, Jagan Teki > wrote: > >> OK, but where it showed on the processor manual? I only find LDO_2P5 > > It is not a SoC regulator. It is the regulator

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-20 Thread Michael Trimarchi
Hi Fabio On Wed, Sep 14, 2016 at 8:00 PM, Fabio Estevam wrote: > On Wed, Sep 14, 2016 at 2:43 PM, Jagan Teki > wrote: > >> OK, but where it showed on the processor manual? I only find LDO_2P5 > > It is not a SoC regulator. It is the regulator that powers the CAN > transceiver. Engicam use

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-14 Thread Fabio Estevam
On Wed, Sep 14, 2016 at 2:43 PM, Jagan Teki wrote: > OK, but where it showed on the processor manual? I only find LDO_2P5 It is not a SoC regulator. It is the regulator that powers the CAN transceiver.

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-14 Thread Fabio Estevam
On Wed, Sep 14, 2016 at 2:43 PM, Jagan Teki wrote: > OK, but where it showed on the processor manual? I only find LDO_2P5 It is not a SoC regulator. It is the regulator that powers the CAN transceiver.

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-14 Thread Jagan Teki
On Wed, Sep 14, 2016 at 4:06 AM, Fabio Estevam wrote: > On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > >> + reg_3p3v: regulator-3p3v { >> + compatible = "regulator-fixed"; >> + regulator-name = "3P3V"; >> +

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-14 Thread Jagan Teki
On Wed, Sep 14, 2016 at 4:06 AM, Fabio Estevam wrote: > On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > >> + reg_3p3v: regulator-3p3v { >> + compatible = "regulator-fixed"; >> + regulator-name = "3P3V"; >> + regulator-min-microvolt =

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-13 Thread Fabio Estevam
On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > + { > + pinctrl-names = "default"; > + pinctrl-0 = <_gpmi_nand>; > + fsl,legacy-bch-geometry; I don't see this property documented nor used anywhere.

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-13 Thread Fabio Estevam
On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > + { > + pinctrl-names = "default"; > + pinctrl-0 = <_gpmi_nand>; > + fsl,legacy-bch-geometry; I don't see this property documented nor used anywhere.

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-13 Thread Fabio Estevam
On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > + reg_3p3v: regulator-3p3v { > + compatible = "regulator-fixed"; > + regulator-name = "3P3V"; > + regulator-min-microvolt = <330>; > +

Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-13 Thread Fabio Estevam
On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki wrote: > + reg_3p3v: regulator-3p3v { > + compatible = "regulator-fixed"; > + regulator-name = "3P3V"; > + regulator-min-microvolt = <330>; > + regulator-max-microvolt = <330>; > +

[PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-11 Thread Jagan Teki
i.CoreM6 Quad/Dual modules are system on module solutions manufactured by Engicam with following characteristics: CPU NXP i.MX6 DQ, 800MHz RAM 1GB, 32, 64 bit, DDR3-800/1066 NAND SLC,512MB Power supply Single 5V MAX LCD RES FULLHD and more info at

[PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-09-11 Thread Jagan Teki
i.CoreM6 Quad/Dual modules are system on module solutions manufactured by Engicam with following characteristics: CPU NXP i.MX6 DQ, 800MHz RAM 1GB, 32, 64 bit, DDR3-800/1066 NAND SLC,512MB Power supply Single 5V MAX LCD RES FULLHD and more info at