RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-07 Thread Anson Huang
ark Rutland <mark.rutl...@arm.com>; > Russell King - ARM Linux <li...@armlinux.org.uk>; Viresh Kumar > <viresh.ku...@linaro.org>; Jacky Bai <ping....@nxp.com>; A.s. Dong > <aisheng.d...@nxp.com> > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz oper

RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-07 Thread Anson Huang
kernel.org>; Linux Kernel Mailing List ker...@vger.kernel.org>; Shawn Guo ; Sascha Hauer > ; Fabio Estevam ; Rob > Herring ; Mark Rutland ; > Russell King - ARM Linux ; Viresh Kumar > ; Jacky Bai ; A.s. Dong > > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-07 Thread Rafael J. Wysocki
ocki.net>; Viresh Kumar <viresh.ku...@linaro.org>; Jacky Bai > > <ping@nxp.com>; A.s. Dong <aisheng.d...@nxp.com> > > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for > > i.mx6ul > > > > On Tue, Jan 2, 2018 at 6:0

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-07 Thread Rafael J. Wysocki
gt; > Russell King - ARM Linux ; Rafael J. Wysocki > > ; Viresh Kumar ; Jacky Bai > > ; A.s. Dong > > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for > > i.mx6ul > > > > On Tue, Jan 2, 2018 at 6:07 PM, Anson Huang wrote: > > >

RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-05 Thread Anson Huang
org>; Mark Rutland <mark.rutl...@arm.com>; > Russell King - ARM Linux <li...@armlinux.org.uk>; Rafael J. Wysocki > <r...@rjwysocki.net>; Viresh Kumar <viresh.ku...@linaro.org>; Jacky Bai > <ping@nxp.com>; A.s. Dong <aisheng.d...@nxp.com> > Sub

RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-05 Thread Anson Huang
Linux > PM ; Linux Kernel Mailing List ker...@vger.kernel.org>; Shawn Guo ; Sascha Hauer > ; Fabio Estevam ; Rob > Herring ; Mark Rutland ; > Russell King - ARM Linux ; Rafael J. Wysocki > ; Viresh Kumar ; Jacky Bai > ; A.s. Dong > Subject: Re: [PATCH 2/2] cpufreq: imx6q

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-05 Thread Rafael J. Wysocki
On Tue, Jan 2, 2018 at 6:07 PM, Anson Huang wrote: > Add 696MHz operating point for i.MX6UL, only for those > parts with speed grading fuse set to 2b'10 supports > 696MHz operating point, so, speed grading check is also > added for i.MX6UL in this patch, the clock tree for

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-05 Thread Rafael J. Wysocki
On Tue, Jan 2, 2018 at 6:07 PM, Anson Huang wrote: > Add 696MHz operating point for i.MX6UL, only for those > parts with speed grading fuse set to 2b'10 supports > 696MHz operating point, so, speed grading check is also > added for i.MX6UL in this patch, the clock tree for each > operating point

RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Anson Huang
rnel.org>; Sascha Hauer <ker...@pengutronix.de>; > Fabio Estevam <fabio.este...@nxp.com>; Shawn Guo > <shawn...@kernel.org> > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for > i.mx6ul > > On Tue, Jan 2, 2018 at 1:17 PM, Anson Huang <

RE: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Anson Huang
cki.net; Russell King - ARM Linux ; Rob > Herring ; Sascha Hauer ; > Fabio Estevam ; Shawn Guo > > Subject: Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for > i.mx6ul > > On Tue, Jan 2, 2018 at 1:17 PM, Anson Huang wrote: > > > This change i

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
On Tue, Jan 2, 2018 at 1:17 PM, Anson Huang wrote: > This change is only valid for mx6ul and mx6ull, other SoCs like 6q/dl/qp are > NOT impacted. Thanks for the clarification: Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
On Tue, Jan 2, 2018 at 1:17 PM, Anson Huang wrote: > This change is only valid for mx6ul and mx6ull, other SoCs like 6q/dl/qp are > NOT impacted. Thanks for the clarification: Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
Hi Anson, On Tue, Jan 2, 2018 at 1:05 PM, Anson Huang wrote: > This change is to support 696MHz operating point, both the speed grading > check and pll rate change are necessary for 696MHz support, do you think > they should be in different patch? I thought this could

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
Hi Anson, On Tue, Jan 2, 2018 at 1:05 PM, Anson Huang wrote: > This change is to support 696MHz operating point, both the speed grading > check and pll rate change are necessary for 696MHz support, do you think > they should be in different patch? I thought this could also change the

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
Hi Anson, On Tue, Jan 2, 2018 at 3:07 PM, Anson Huang wrote: > diff --git a/drivers/cpufreq/imx6q-cpufreq.c b/drivers/cpufreq/imx6q-cpufreq.c > index d9b2c2d..cbda0cc 100644 > --- a/drivers/cpufreq/imx6q-cpufreq.c > +++ b/drivers/cpufreq/imx6q-cpufreq.c > @@ -120,6 +120,10

Re: [PATCH 2/2] cpufreq: imx6q: add 696MHz operating point for i.mx6ul

2018-01-02 Thread Fabio Estevam
Hi Anson, On Tue, Jan 2, 2018 at 3:07 PM, Anson Huang wrote: > diff --git a/drivers/cpufreq/imx6q-cpufreq.c b/drivers/cpufreq/imx6q-cpufreq.c > index d9b2c2d..cbda0cc 100644 > --- a/drivers/cpufreq/imx6q-cpufreq.c > +++ b/drivers/cpufreq/imx6q-cpufreq.c > @@ -120,6 +120,10 @@ static int