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

[Jagan Teki]

On Wed, Sep 14, 2016 at 4:06 AM, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> On Sun, Sep 11, 2016 at 3:30 PM, Jagan Teki <jagannadh.teki@xxxxxxxxx> wrote:
>
>> +       reg_3p3v: regulator-3p3v {
>> +               compatible = "regulator-fixed";
>> +               regulator-name = "3P3V";
>> +               regulator-min-microvolt = <3300000>;
>> +               regulator-max-microvolt = <3300000>;
>> +               regulator-boot-on;
>> +               regulator-always-on;
>
> You should remove " regulator-boot-on" and "regulator-always-on" as
> the reg_3p3v regulator is controlled by the can nodes.

OK, but where it showed on the processor manual? I only find LDO_2P5
regulator which is connected to high speed devices, HDMI, SATA etc on
Figure 50-1: Power system overview

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