Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1491300 > unrolled thread

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

Started byFabio Estevam <festevam@gmail.com>
First post2016-09-26 15:50 +0200
Last post2016-09-26 16:00 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual  initial support Fabio Estevam <festevam@gmail.com> - 2016-09-26 15:50 +0200
    Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual  initial support Fabio Estevam <festevam@gmail.com> - 2016-09-26 16:00 +0200
      Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual  initial support Michael Trimarchi <michael@amarulasolutions.com> - 2016-09-26 16:00 +0200
        Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual  initial support Jagan Teki <jagan@amarulasolutions.com> - 2016-09-26 16:30 +0200
    Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual  initial support Michael Trimarchi <michael@amarulasolutions.com> - 2016-09-26 16:00 +0200

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

FromFabio Estevam <festevam@gmail.com>
Date2016-09-26 15:50 +0200
SubjectRe: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support
Message-ID<slEs9-4qU-13@gated-at.bofh.it>
Hi Matteo,

On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi <matteo.lisi@engicam.com> 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 don't you simply do like this?

       reg_3p3v: regulator-3p3v {
               compatible = "regulator-fixed";
               regulator-name = "3P3V";
               regulator-min-microvolt = <3300000>;
               regulator-max-microvolt = <3300000>;
       };

[toc] | [next] | [standalone]

#1491304

FromFabio Estevam <festevam@gmail.com>
Date2016-09-26 16:00 +0200
Message-ID<slEBP-4u5-9@gated-at.bofh.it>
In reply to#1491300
On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi
<michael@amarulasolutions.com> wrote:

>> Why don't you simply do like this?
>>
>>        reg_3p3v: regulator-3p3v {
>>                compatible = "regulator-fixed";
>>                regulator-name = "3P3V";
>>                regulator-min-microvolt = <3300000>;
>>                regulator-max-microvolt = <3300000>;
>>        };
>
> Should dts rispect the schematic connection?

Yes

[toc] | [prev] | [next] | [standalone]

#1491307

FromMichael Trimarchi <michael@amarulasolutions.com>
Date2016-09-26 16:00 +0200
Message-ID<slEBQ-4u5-13@gated-at.bofh.it>
In reply to#1491304
Hi

On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi
> <michael@amarulasolutions.com> wrote:
>
>>> Why don't you simply do like this?
>>>
>>>        reg_3p3v: regulator-3p3v {
>>>                compatible = "regulator-fixed";
>>>                regulator-name = "3P3V";
>>>                regulator-min-microvolt = <3300000>;
>>>                regulator-max-microvolt = <3300000>;
>>>        };
>>
>> Should dts rispect the schematic connection?
>
> Yes

So boot on and always on are used to declare their status and their
behavionr. I don't
see the point to not specify them in this way

Michael


-- 
| Michael Nazzareno Trimarchi                     Amarula Solutions BV |
| COO  -  Founder                                      Cruquiuskade 47 |
| +31(0)851119172                                 Amsterdam 1018 AM NL |
|                  [`as] http://www.amarulasolutions.com               |

[toc] | [prev] | [next] | [standalone]

#1491322

FromJagan Teki <jagan@amarulasolutions.com>
Date2016-09-26 16:30 +0200
Message-ID<slF4S-4Wz-21@gated-at.bofh.it>
In reply to#1491307
Hi Fabio,

On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi
<michael@amarulasolutions.com> wrote:
> Hi
>
> On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi
>> <michael@amarulasolutions.com> wrote:
>>
>>>> Why don't you simply do like this?
>>>>
>>>>        reg_3p3v: regulator-3p3v {
>>>>                compatible = "regulator-fixed";
>>>>                regulator-name = "3P3V";
>>>>                regulator-min-microvolt = <3300000>;
>>>>                regulator-max-microvolt = <3300000>;
>>>>        };
>>>
>>> Should dts rispect the schematic connection?
>>
>> Yes
>
> So boot on and always on are used to declare their status and their
> behavionr. I don't
> see the point to not specify them in this way

Please try to understand that due to the case where we used fixed
voltage regulators on SOM as mentioned By Metteo and Michael dt nodes
should have regulator-always-on and regulator-boot-on properties. And
even we added one of our module which is already in ML[1]

Hope this is helpful conclude this thread.

[1] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a58e4e608bc88735581fdd9861676e522fa3c20d

thanks!
-- 
Jagan Teki
Free Software Engineer | Amarula Solutions
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

[toc] | [prev] | [next] | [standalone]

#1491305

FromMichael Trimarchi <michael@amarulasolutions.com>
Date2016-09-26 16:00 +0200
Message-ID<slEBP-4u5-11@gated-at.bofh.it>
In reply to#1491300
Hi

On Mon, Sep 26, 2016 at 3:47 PM, Fabio Estevam <festevam@gmail.com> wrote:
> Hi Matteo,
>
> On Mon, Sep 26, 2016 at 10:44 AM, Matteo Lisi <matteo.lisi@engicam.com> 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 don't you simply do like this?
>
>        reg_3p3v: regulator-3p3v {
>                compatible = "regulator-fixed";
>                regulator-name = "3P3V";
>                regulator-min-microvolt = <3300000>;
>                regulator-max-microvolt = <3300000>;
>        };

Should dts rispect the schematic connection?


Michael


-- 
| Michael Nazzareno Trimarchi                     Amarula Solutions BV |
| COO  -  Founder                                      Cruquiuskade 47 |
| +31(0)851119172                                 Amsterdam 1018 AM NL |
|                  [`as] http://www.amarulasolutions.com               |

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web