RE: [PATCH 1/2] ARM: dts: imx6ul: add 696MHz operating point

[Anson Huang]

Post this discussion mail to kernel mail list, since last mail is rejected due to incorrect format, sorry for confusion. 

Best Regards!
Anson Huang


> -----Original Message-----
> From: Fabio Estevam [mailto:festevam@xxxxxxxxx]
> Sent: 2018-01-02 11:32 PM
> To: Anson Huang <anson.huang@xxxxxxx>
> Cc: moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE <linux-
> arm-kernel@xxxxxxxxxxxxxxxxxxx>; open list:OPEN FIRMWARE AND FLATTENED
> DEVICE TREE BINDINGS <devicetree@xxxxxxxxxxxxxxx>; linux-
> pm@xxxxxxxxxxxxxxx; linux-kernel <linux-kernel@xxxxxxxxxxxxxxx>; Mark
> Rutland <mark.rutland@xxxxxxx>; A.s. Dong <aisheng.dong@xxxxxxx>; Jacky
> Bai <ping.bai@xxxxxxx>; viresh kumar <viresh.kumar@xxxxxxxxxx>;
> rjw@xxxxxxxxxxxxx; Russell King - ARM Linux <linux@xxxxxxxxxxxxxxx>; Rob
> Herring <robh+dt@xxxxxxxxxx>; Sascha Hauer <kernel@xxxxxxxxxxxxxx>;
> Fabio Estevam <fabio.estevam@xxxxxxx>; Shawn Guo
> <shawnguo@xxxxxxxxxx>
> Subject: Re: [PATCH 1/2] ARM: dts: imx6ul: add 696MHz operating point
> 
> On Tue, Jan 2, 2018 at 1:12 PM, Anson Huang <anson.huang@xxxxxxx> wrote:
> 
> > There is a comment in VDD_ARM, VDD_SOC must NOT lower than VDD_ARM.
> >
> > Output voltage must be set to the following rules:
> > â VDD_ARM_CAP <= VDD_SOC_CAP
> > â VDD_SOC_CAP - VDD_ARM_CAP < 330 mV
> 
> Thanks for the clarifcation.
> 
> Reviewed-by: Fabio Estevam <fabio.estevam@xxxxxxx>