From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751504AbcF3Ux7 (ORCPT ); Thu, 30 Jun 2016 16:53:59 -0400 Received: from mail-wm0-f44.google.com ([74.125.82.44]:37002 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751068AbcF3Ux5 (ORCPT ); Thu, 30 Jun 2016 16:53:57 -0400 Date: Thu, 30 Jun 2016 21:54:08 +0100 From: Lee Jones To: Stephen Boyd Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@stlinux.com, maxime.coquelin@st.com, patrice.chotard@st.com, mturquette@baylibre.com, broonie@kernel.org, linux-clk@vger.kernel.org, linux-spi@vger.kernel.org Subject: Re: [PATCH 0/6] clk: sti: Add support for critical clocks Message-ID: <20160630205408.GG1707@dell> References: <20160607111930.10375-1-lee.jones@linaro.org> <20160630192337.GN1521@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160630192337.GN1521@codeaurora.org> User-Agent: Mutt/1.6.1 (2016-04-27) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 30 Jun 2016, Stephen Boyd wrote: > On 06/07, Lee Jones wrote: > > Describe the platform's critical clocks in DT, then search for them > > from the framework. This set allows us to remove the clk_ignore_unused > > kernel command line flag which has been required to achieve a successful > > boot to prompt until this point. > > > > This may not be the definitive list. There may be other clocks which > > need to be marked as critical, but we can add those as we go along. > > > > I applied the three clk ones to an immutable branch named > 'clk-st-critical' in the clk tree and merged this into clk-next. > I'm not comfortable taking the spi patch as I suspect that will > break something unless the dts patches are applied. > > Please use this branch to base your dts and spi patches on so it > can be merged through arm-soc without introducing some bisection > hole. > > The following changes since commit 1a695a905c18548062509178b98bc91e67510864: > > Linux 4.7-rc1 (2016-05-29 09:29:24 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-st-critical > > for you to fetch changes up to 6ca59e6e1fc3a8d7ccbf85ff036bd6ff40847c1a: > > clk: st: clkgen-pll: Detect critical clocks (2016-06-30 12:17:11 -0700) > > ---------------------------------------------------------------- > Lee Jones (3): > clk: st: clk-flexgen: Detect critical clocks > clk: st: clkgen-fsyn: Detect critical clocks > clk: st: clkgen-pll: Detect critical clocks > > drivers/clk/st/clk-flexgen.c | 4 +++- > drivers/clk/st/clkgen-fsyn.c | 10 +++++++--- > drivers/clk/st/clkgen-pll.c | 27 +++++++++++++++++---------- > 3 files changed, 27 insertions(+), 14 deletions(-) Ideal, thanks Stephen. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 0/6] clk: sti: Add support for critical clocks Date: Thu, 30 Jun 2016 21:54:08 +0100 Message-ID: <20160630205408.GG1707@dell> References: <20160607111930.10375-1-lee.jones@linaro.org> <20160630192337.GN1521@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: kernel@stlinux.com, mturquette@baylibre.com, patrice.chotard@st.com, linux-kernel@vger.kernel.org, broonie@kernel.org, linux-spi@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, maxime.coquelin@st.com To: Stephen Boyd Return-path: Content-Disposition: inline In-Reply-To: <20160630192337.GN1521@codeaurora.org> 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 List-Id: linux-spi.vger.kernel.org T24gVGh1LCAzMCBKdW4gMjAxNiwgU3RlcGhlbiBCb3lkIHdyb3RlOgo+IE9uIDA2LzA3LCBMZWUg Sm9uZXMgd3JvdGU6Cj4gPiBEZXNjcmliZSB0aGUgcGxhdGZvcm0ncyBjcml0aWNhbCBjbG9ja3Mg aW4gRFQsIHRoZW4gc2VhcmNoIGZvciB0aGVtCj4gPiBmcm9tIHRoZSBmcmFtZXdvcmsuICBUaGlz IHNldCBhbGxvd3MgdXMgdG8gcmVtb3ZlIHRoZSBjbGtfaWdub3JlX3VudXNlZAo+ID4ga2VybmVs IGNvbW1hbmQgbGluZSBmbGFnIHdoaWNoIGhhcyBiZWVuIHJlcXVpcmVkIHRvIGFjaGlldmUgYSBz dWNjZXNzZnVsCj4gPiBib290IHRvIHByb21wdCB1bnRpbCB0aGlzIHBvaW50Lgo+ID4gCj4gPiBU aGlzIG1heSBub3QgYmUgdGhlIGRlZmluaXRpdmUgbGlzdC4gIFRoZXJlIG1heSBiZSBvdGhlciBj bG9ja3Mgd2hpY2gKPiA+IG5lZWQgdG8gYmUgbWFya2VkIGFzIGNyaXRpY2FsLCBidXQgd2UgY2Fu IGFkZCB0aG9zZSBhcyB3ZSBnbyBhbG9uZy4KPiA+IAo+IAo+IEkgYXBwbGllZCB0aGUgdGhyZWUg Y2xrIG9uZXMgdG8gYW4gaW1tdXRhYmxlIGJyYW5jaCBuYW1lZAo+ICdjbGstc3QtY3JpdGljYWwn IGluIHRoZSBjbGsgdHJlZSBhbmQgbWVyZ2VkIHRoaXMgaW50byBjbGstbmV4dC4KPiBJJ20gbm90 IGNvbWZvcnRhYmxlIHRha2luZyB0aGUgc3BpIHBhdGNoIGFzIEkgc3VzcGVjdCB0aGF0IHdpbGwK PiBicmVhayBzb21ldGhpbmcgdW5sZXNzIHRoZSBkdHMgcGF0Y2hlcyBhcmUgYXBwbGllZC4KPiAK PiBQbGVhc2UgdXNlIHRoaXMgYnJhbmNoIHRvIGJhc2UgeW91ciBkdHMgYW5kIHNwaSBwYXRjaGVz IG9uIHNvIGl0Cj4gY2FuIGJlIG1lcmdlZCB0aHJvdWdoIGFybS1zb2Mgd2l0aG91dCBpbnRyb2R1 Y2luZyBzb21lIGJpc2VjdGlvbgo+IGhvbGUuCj4gCj4gVGhlIGZvbGxvd2luZyBjaGFuZ2VzIHNp bmNlIGNvbW1pdCAxYTY5NWE5MDVjMTg1NDgwNjI1MDkxNzhiOThiYzkxZTY3NTEwODY0Ogo+IAo+ ICAgTGludXggNC43LXJjMSAoMjAxNi0wNS0yOSAwOToyOToyNCAtMDcwMCkKPiAKPiBhcmUgYXZh aWxhYmxlIGluIHRoZSBnaXQgcmVwb3NpdG9yeSBhdDoKPiAKPiAgIGdpdDovL2dpdC5rZXJuZWwu b3JnL3B1Yi9zY20vbGludXgva2VybmVsL2dpdC9jbGsvbGludXguZ2l0IGNsay1zdC1jcml0aWNh bAo+IAo+IGZvciB5b3UgdG8gZmV0Y2ggY2hhbmdlcyB1cCB0byA2Y2E1OWU2ZTFmYzNhOGQ3Y2Ni Zjg1ZmYwMzZiZDZmZjQwODQ3YzFhOgo+IAo+ICAgY2xrOiBzdDogY2xrZ2VuLXBsbDogRGV0ZWN0 IGNyaXRpY2FsIGNsb2NrcyAoMjAxNi0wNi0zMCAxMjoxNzoxMSAtMDcwMCkKPiAKPiAtLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t Cj4gTGVlIEpvbmVzICgzKToKPiAgICAgICBjbGs6IHN0OiBjbGstZmxleGdlbjogRGV0ZWN0IGNy aXRpY2FsIGNsb2Nrcwo+ICAgICAgIGNsazogc3Q6IGNsa2dlbi1mc3luOiBEZXRlY3QgY3JpdGlj YWwgY2xvY2tzCj4gICAgICAgY2xrOiBzdDogY2xrZ2VuLXBsbDogRGV0ZWN0IGNyaXRpY2FsIGNs b2Nrcwo+IAo+ICBkcml2ZXJzL2Nsay9zdC9jbGstZmxleGdlbi5jIHwgIDQgKysrLQo+ICBkcml2 ZXJzL2Nsay9zdC9jbGtnZW4tZnN5bi5jIHwgMTAgKysrKysrKy0tLQo+ICBkcml2ZXJzL2Nsay9z dC9jbGtnZW4tcGxsLmMgIHwgMjcgKysrKysrKysrKysrKysrKystLS0tLS0tLS0tCj4gIDMgZmls ZXMgY2hhbmdlZCwgMjcgaW5zZXJ0aW9ucygrKSwgMTQgZGVsZXRpb25zKC0pCgpJZGVhbCwgdGhh bmtzIFN0ZXBoZW4uCgotLSAKTGVlIEpvbmVzCkxpbmFybyBTVE1pY3JvZWxlY3Ryb25pY3MgTGFu ZGluZyBUZWFtIExlYWQKTGluYXJvLm9yZyDilIIgT3BlbiBzb3VyY2Ugc29mdHdhcmUgZm9yIEFS TSBTb0NzCkZvbGxvdyBMaW5hcm86IEZhY2Vib29rIHwgVHdpdHRlciB8IEJsb2cKCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwg bWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8v bGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Thu, 30 Jun 2016 21:54:08 +0100 Subject: [PATCH 0/6] clk: sti: Add support for critical clocks In-Reply-To: <20160630192337.GN1521@codeaurora.org> References: <20160607111930.10375-1-lee.jones@linaro.org> <20160630192337.GN1521@codeaurora.org> Message-ID: <20160630205408.GG1707@dell> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 30 Jun 2016, Stephen Boyd wrote: > On 06/07, Lee Jones wrote: > > Describe the platform's critical clocks in DT, then search for them > > from the framework. This set allows us to remove the clk_ignore_unused > > kernel command line flag which has been required to achieve a successful > > boot to prompt until this point. > > > > This may not be the definitive list. There may be other clocks which > > need to be marked as critical, but we can add those as we go along. > > > > I applied the three clk ones to an immutable branch named > 'clk-st-critical' in the clk tree and merged this into clk-next. > I'm not comfortable taking the spi patch as I suspect that will > break something unless the dts patches are applied. > > Please use this branch to base your dts and spi patches on so it > can be merged through arm-soc without introducing some bisection > hole. > > The following changes since commit 1a695a905c18548062509178b98bc91e67510864: > > Linux 4.7-rc1 (2016-05-29 09:29:24 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-st-critical > > for you to fetch changes up to 6ca59e6e1fc3a8d7ccbf85ff036bd6ff40847c1a: > > clk: st: clkgen-pll: Detect critical clocks (2016-06-30 12:17:11 -0700) > > ---------------------------------------------------------------- > Lee Jones (3): > clk: st: clk-flexgen: Detect critical clocks > clk: st: clkgen-fsyn: Detect critical clocks > clk: st: clkgen-pll: Detect critical clocks > > drivers/clk/st/clk-flexgen.c | 4 +++- > drivers/clk/st/clkgen-fsyn.c | 10 +++++++--- > drivers/clk/st/clkgen-pll.c | 27 +++++++++++++++++---------- > 3 files changed, 27 insertions(+), 14 deletions(-) Ideal, thanks Stephen. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog