From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?S=c3=a9bastien_Szymanski?= Subject: Re: [PATCH 2/2] ARM: dts: imx6ull: add operating points Date: Thu, 19 Apr 2018 09:27:03 +0200 Message-ID: <265f4635-bf67-48a9-4ad5-caaba752be8d@armadeus.com> References: <20180418145230.10249-1-sebastien.szymanski@armadeus.com> <20180418145230.10249-2-sebastien.szymanski@armadeus.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Stefan Agner Cc: Fabio Estevam , Shawn Guo , "Rafael J . Wysocki" , linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org List-Id: linux-pm@vger.kernel.org SGkgU3RlZmFuLAoKT24gMDQvMTgvMjAxOCAwNTozNSBQTSwgU3RlZmFuIEFnbmVyIHdyb3RlOgo+ IE9uIDE4LjA0LjIwMTggMTY6NTIsIFPDqWJhc3RpZW4gU3p5bWFuc2tpIHdyb3RlOgo+PiBpLk1Y NlVMTCBoYXMgZGlmZmVyZW50IG9wZXJhdGluZyByYW5nZXMgdGhhbiBpLk1YNlVMIHNvIGFkZCB0 aGUKPj4gb3BlcmF0aW5nIHBvaW50cyBmb3IgdGhlIGkuTVg2VUxMLiBBIDI1bVYgb2Zmc2V0IGlz IGFkZGVkIHRvIHRoZSBtaW5pbXVtCj4+IGFsbG93ZWQgdmFsdWVzIGxpa2UgZm9yIHRoZSBpLk1Y NlVMLgo+IAo+IEhpIFPDqWJhc3RpZW4sCj4gCj4gTGFzdCB0aW1lIEkgY2hlY2tlZCB0aGVyZSB3 YXMgYSBEQyBjb250cm9sbGVyIG1pc3NpbmcgZm9yIHRoZSBFVkssIHNvIEkKPiB0aGluayBpdCB3 YXMgbm90IHBvc3NpYmxlIHRvIGdvIHRvIHRoZSBoaWdoZXN0IG9wZXJhdGluZyBwb2ludHMgeWV0 Lgo+IEhhdmUgeW91IHRlc3RlZCB0aGlzIG9uIHRoZSBFVks/CgpObywgSSBoYXZlIHRlc3RlZCBv biBhbiBPUE9TNlVMRGV2IGJvYXJkLgoKSSd2ZSBqdXN0IG5vdGljZWQgdGhhdCB0aGUgc29jLW9w ZXJhdGluZy1wb2ludHMgdmFsdWVzIGFyZSB3cm9uZyBpbiB0aGlzCnBhdGNoLCBJJ3ZlIHNlbnQg YSB2Mi4KClJlZ2FyZHMsCgo+IAo+IC0tCj4gU3RlZmFuCj4gCj4+Cj4+IFNpZ25lZC1vZmYtYnk6 IFPDqWJhc3RpZW4gU3p5bWFuc2tpIDxzZWJhc3RpZW4uc3p5bWFuc2tpQGFybWFkZXVzLmNvbT4K Pj4gLS0tCj4+ICBhcmNoL2FybS9ib290L2R0cy9pbXg2dWxsLmR0c2kgfCAxOSArKysrKysrKysr KysrKysrKysrCj4+ICAxIGZpbGUgY2hhbmdlZCwgMTkgaW5zZXJ0aW9ucygrKQo+Pgo+PiBkaWZm IC0tZ2l0IGEvYXJjaC9hcm0vYm9vdC9kdHMvaW14NnVsbC5kdHNpIGIvYXJjaC9hcm0vYm9vdC9k dHMvaW14NnVsbC5kdHNpCj4+IGluZGV4IDU3MWRkZDcxY2RiYS4uN2U2MjczYmUwMjkxIDEwMDY0 NAo+PiAtLS0gYS9hcmNoL2FybS9ib290L2R0cy9pbXg2dWxsLmR0c2kKPj4gKysrIGIvYXJjaC9h cm0vYm9vdC9kdHMvaW14NnVsbC5kdHNpCj4+IEBAIC00Niw2ICs0NiwyNSBAQAo+PiAgLyogRGVs ZXRlIFVBUlQ4IGluIEFJUFMtMSAoaS5NWDZVTCBzcGVjaWZpYykgKi8KPj4gIC9kZWxldGUtbm9k ZS8gJnVhcnQ4Owo+PiAgCj4+ICsmY3B1MCB7Cj4+ICsJb3BlcmF0aW5nLXBvaW50cyA9IDwKPj4g KwkJLyoga0h6CXVWICovCj4+ICsJCTkwMDAwMAkxMjc1MDAwCj4+ICsJCTc5MjAwMAkxMjI1MDAw Cj4+ICsJCTUyODAwMAkxMTc1MDAwCj4+ICsJCTM5NjAwMAkxMDI1MDAwCj4+ICsJCTE5ODAwMAk5 NTAwMDAKPj4gKwk+Owo+PiArCWZzbCxzb2Mtb3BlcmF0aW5nLXBvaW50cyA9IDwKPj4gKwkJLyog S0h6CXVWICovCj4+ICsJCTkwMDAwMAkxNDAwMDAwCj4+ICsJCTc5MjAwMAkxMzUwMDAwCj4+ICsJ CTUyODAwMAkxMTc1MDAwCj4+ICsJCTM5NjAwMAkxMTc1MDAwCj4+ICsJCTE5ODAwMAkxMTc1MDAw Cj4+ICsJPjsKPj4gK307Cj4+ICsKPj4gIC8gewo+PiAgCXNvYyB7Cj4+ICAJCWFpcHMzOiBhaXBz LWJ1c0AyMjAwMDAwIHsKCgotLSAKU8OpYmFzdGllbiBTenltYW5za2kKU29mdHdhcmUgZW5naW5l ZXIsIEFybWFkZXVzIFN5c3RlbXMKVGVsOiArMzMgKDApOSA3MiAyOSA0MSA0NApGYXg6ICszMyAo MCk5IDcyIDI4IDc5IDI2CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxp c3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0 aW5mby9saW51eC1hcm0ta2VybmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: sebastien.szymanski@armadeus.com (=?UTF-8?Q?S=c3=a9bastien_Szymanski?=) Date: Thu, 19 Apr 2018 09:27:03 +0200 Subject: [PATCH 2/2] ARM: dts: imx6ull: add operating points In-Reply-To: References: <20180418145230.10249-1-sebastien.szymanski@armadeus.com> <20180418145230.10249-2-sebastien.szymanski@armadeus.com> Message-ID: <265f4635-bf67-48a9-4ad5-caaba752be8d@armadeus.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 >> --- >> 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 at 2200000 { -- S?bastien Szymanski Software engineer, Armadeus Systems Tel: +33 (0)9 72 29 41 44 Fax: +33 (0)9 72 28 79 26