| Message ID | 20180418145230.10249-2-sebastien.szymanski@armadeus.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/2] cpufreq: imx6q: check speed grades for i.MX6ULL | expand |
On 18.04.2018 16:52, Sébastien Szymanski wrote: > i.MX6ULL has different operating ranges than i.MX6UL so add the > operating points for the i.MX6ULL. A 25mV offset is added to the minimum > allowed values like for the i.MX6UL. Hi Sébastien, Last time I checked there was a DC controller missing for the EVK, so I think it was not possible to go to the highest operating points yet. Have you tested this on the EVK? -- Stefan > > Signed-off-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com> > --- > arch/arm/boot/dts/imx6ull.dtsi | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi > index 571ddd71cdba..7e6273be0291 100644 > --- a/arch/arm/boot/dts/imx6ull.dtsi > +++ b/arch/arm/boot/dts/imx6ull.dtsi > @@ -46,6 +46,25 @@ > /* Delete UART8 in AIPS-1 (i.MX6UL specific) */ > /delete-node/ &uart8; > > +&cpu0 { > + operating-points = < > + /* kHz uV */ > + 900000 1275000 > + 792000 1225000 > + 528000 1175000 > + 396000 1025000 > + 198000 950000 > + >; > + fsl,soc-operating-points = < > + /* KHz uV */ > + 900000 1400000 > + 792000 1350000 > + 528000 1175000 > + 396000 1175000 > + 198000 1175000 > + >; > +}; > + > / { > soc { > aips3: aips-bus@2200000 {
Hi Stefan, On 04/18/2018 05:35 PM, Stefan Agner wrote: > On 18.04.2018 16:52, Sébastien Szymanski wrote: >> i.MX6ULL has different operating ranges than i.MX6UL so add the >> operating points for the i.MX6ULL. A 25mV offset is added to the minimum >> allowed values like for the i.MX6UL. > > Hi Sébastien, > > Last time I checked there was a DC controller missing for the EVK, so I > think it was not possible to go to the highest operating points yet. > Have you tested this on the EVK? No, I have tested on an OPOS6ULDev board. I've just noticed that the soc-operating-points values are wrong in this patch, I've sent a v2. Regards, > > -- > Stefan > >> >> Signed-off-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com> >> --- >> arch/arm/boot/dts/imx6ull.dtsi | 19 +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> >> diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi >> index 571ddd71cdba..7e6273be0291 100644 >> --- a/arch/arm/boot/dts/imx6ull.dtsi >> +++ b/arch/arm/boot/dts/imx6ull.dtsi >> @@ -46,6 +46,25 @@ >> /* Delete UART8 in AIPS-1 (i.MX6UL specific) */ >> /delete-node/ &uart8; >> >> +&cpu0 { >> + operating-points = < >> + /* kHz uV */ >> + 900000 1275000 >> + 792000 1225000 >> + 528000 1175000 >> + 396000 1025000 >> + 198000 950000 >> + >; >> + fsl,soc-operating-points = < >> + /* KHz uV */ >> + 900000 1400000 >> + 792000 1350000 >> + 528000 1175000 >> + 396000 1175000 >> + 198000 1175000 >> + >; >> +}; >> + >> / { >> soc { >> aips3: aips-bus@2200000 {
diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi index 571ddd71cdba..7e6273be0291 100644 --- a/arch/arm/boot/dts/imx6ull.dtsi +++ b/arch/arm/boot/dts/imx6ull.dtsi @@ -46,6 +46,25 @@ /* Delete UART8 in AIPS-1 (i.MX6UL specific) */ /delete-node/ &uart8; +&cpu0 { + operating-points = < + /* kHz uV */ + 900000 1275000 + 792000 1225000 + 528000 1175000 + 396000 1025000 + 198000 950000 + >; + fsl,soc-operating-points = < + /* KHz uV */ + 900000 1400000 + 792000 1350000 + 528000 1175000 + 396000 1175000 + 198000 1175000 + >; +}; + / { soc { aips3: aips-bus@2200000 {
i.MX6ULL has different operating ranges than i.MX6UL so add the operating points for the i.MX6ULL. A 25mV offset is added to the minimum allowed values like for the i.MX6UL. Signed-off-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com> --- arch/arm/boot/dts/imx6ull.dtsi | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)