Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751933AbdF3HPP (ORCPT ); Fri, 30 Jun 2017 03:15:15 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:56739 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751896AbdF3HPN (ORCPT ); Fri, 30 Jun 2017 03:15:13 -0400 From: Gabriel FERNANDEZ To: Stephen Boyd CC: Rob Herring , Mark Rutland , Russell King , Maxime Coquelin , Alexandre TORGUE , Michael Turquette , Nicolas Pitre , Arnd Bergmann , "daniel.thompson@linaro.org" , "andrea.merello@gmail.com" , "radoslaw.pietrzyk@gmail.com" , Lee Jones , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-clk@vger.kernel.org" , Ludovic BARRE , "Olivier BIDEAU" , Amelie DELAUNAY , "gabriel.fernandez.st@gmail.com" Subject: Re: [RESEND PATCH v4] clk: stm32h7: Add stm32h743 clock driver Thread-Topic: [RESEND PATCH v4] clk: stm32h7: Add stm32h743 clock driver Thread-Index: AQHS31uuzVmODfuWeEOg6i19U5EFNaIv1S6AgAjJOICAAdCHAIABa4kAgACyjQCAAHO8AA== Date: Fri, 30 Jun 2017 07:14:23 +0000 Message-ID: References: <1496818794-14771-1-git-send-email-gabriel.fernandez@st.com> <20170621220703.GI4493@codeaurora.org> <110cf67f-e1d6-1f99-953c-6b9639bb4910@st.com> <20170628155956.GA5316@codeaurora.org> <7c0c3861-cf61-a9e2-cd5d-2ae30b8f32fa@st.com> <20170630002008.GD22780@codeaurora.org> In-Reply-To: <20170630002008.GD22780@codeaurora.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.49] Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-06-30_05:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v5U7FOG9025062 Content-Length: 1110 Lines: 31 On 06/30/2017 02:20 AM, Stephen Boyd wrote: > On 06/29, Gabriel FERNANDEZ wrote: >> >> On 06/28/2017 05:59 PM, Stephen Boyd wrote: >>> On 06/27, Gabriel FERNANDEZ wrote: >>>> On 06/22/2017 12:07 AM, Stephen Boyd wrote: >>>>> readl_poll_timeout? >>>>> >>>> if i use readl_poll_timeout (wich use 'ktime_get()') it can be >>>> operational only after the selection of clocksource ? (device_initcall). >>>> And then if a driver turn on a clock before, it could blocked the linux >>>> console ? >>>> >>> Ok. I wonder if we could add some sort of starting check to >>> readl_poll_timeout() that tests system_state for booting vs. >>> scheduling? That should be sufficient to handle this case? >>> >> Oops i think i understood my problem... >> i used readl_poll_timeout in atomic context. >> I have to move my code in the .prepare ops. >> >> If you are ok with that i will send a v5 >> > There's readl_poll_timeout_atomic() for those modes. > yes it's exactly the test i made (use 'readl_poll_timeout()_atomic' in .enable ops) but i'm blocked. if i do the same in .prepare ops with 'readl_poll_timeout()' it's ok.